mbox series

[BlueZ,0/3] Fix crash in dbus caused by Unicode characters

Message ID 20250318131431.124750-1-r.smirnov@omp.ru
Headers show
Series Fix crash in dbus caused by Unicode characters | expand

Message

Roman Smirnov March 18, 2025, 1:14 p.m. UTC 
A crash was found during fuzzing testing:

dbus[5324]: arguments to dbus_message_iter_append_basic() were
incorrect, assertion "_dbus_check_is_valid_utf8 (*string_p)"
failed in file .../dbus-message.c line 2765.

Fix it with additional checks.

Roman Smirnov (3):
  shared/util: implement argsisutf8()
  client: replace validate_input() with argsisutf8()
  shared/shell: add check to shell_exec()

 client/main.c      | 17 ++---------------
 src/shared/shell.c |  3 +++
 src/shared/util.c  | 12 ++++++++++++
 src/shared/util.h  |  1 +
 4 files changed, 18 insertions(+), 15 deletions(-)

Comments

patchwork-bot+bluetooth@kernel.org March 18, 2025, 6 p.m. UTC | #1 
Hello:

This series was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:

On Tue, 18 Mar 2025 16:14:28 +0300 you wrote:
> A crash was found during fuzzing testing:
> 
> dbus[5324]: arguments to dbus_message_iter_append_basic() were
> incorrect, assertion "_dbus_check_is_valid_utf8 (*string_p)"
> failed in file .../dbus-message.c line 2765.
> 
> Fix it with additional checks.
> 
> [...]

Here is the summary with links:
  - [BlueZ,1/3] shared/util: implement argsisutf8()
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=0205edbd29fc
  - [BlueZ,2/3] client: replace validate_input() with argsisutf8()
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=eb1dd2bc1c4b
  - [BlueZ,3/3] shared/shell: add check to shell_exec()
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=1edffc22a343

You are awesome, thank you!