mbox series

[BlueZ,v1,0/1] mgmt: Fix crash after pair command

Message ID 20231214110339.1763-1-vinit.mehta@nxp.com (mailing list archive)
Headers show
Series mgmt: Fix crash after pair command | expand

Message

Vinit Mehta Dec. 14, 2023, 11:03 a.m. UTC
Hello Maintainers

This patch fixes assertion in bluetoothctl client process

After pair command, if the user doesn't provide any input on bluetoothctl
CLI interface after receiving the prompt(yes/no), than subsequent CLI 
command will trigger a call to DBUS library function 
(dbus_message_get_no_reply) with a NULL message pointer which triggers
assertion in DBUS library causing the bluetoothctl process to crash. 
The change below will ensure in case if no input is given by the user, 
a conditional check is added to handle this scenario and a default 
character ('N') will be passed so as to avoid the assertion.

Thank you in advance for your review.

Warm Regards
Vinit Mehta

Vinit Mehta (1):
  mgmt: Fix crash after pair command

 client/mgmt.c | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)