Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-652-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Vinit Mehta <vinit.mehta@nxp.com>
To: linux-bluetooth@vger.kernel.org
Cc: vinit.mehta@nxp.com,
	devyani.godbole@nxp.com,
	nitin.jadhav@nxp.com
Subject: [PATCH BlueZ v2 0/1] mgmt: Fix crash after pair command
Date: Tue, 19 Dec 2023 11:58:00 +0530
Message-ID: <20231219062801.307-1-vinit.mehta@nxp.com>
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?/75+18nvzqcofnQvonDKdJZ+JvLmtEIo0ZIpAJWma2dKHtzLaQC/Wp2wRy3i?=
 =?us-ascii?Q?WzpKUEIUjde8C/qHojDoCRk0thKOFw4JYOh8eNxEG1RBsoKgpJMyJ6ZwIMi9?=
 =?us-ascii?Q?5GfRUvcvZYciSJswhOFlOIS9DljFCC6LNEbdQlpcixofKZPYOlFPzuHFZdRn?=
 =?us-ascii?Q?5Ost8S4lSMZShmmsSnpq5g9jaMAC7zpHir0woif3cAgnohFZJuEUgm4j6EZR?=
 =?us-ascii?Q?VNx+lSLa6hu/p6SpxAxWcK5FcIXVIJWrHmMn7KOJwTQRJcfp/K6lLd+bRp/o?=
 =?us-ascii?Q?kbaVc9CS1mHxBraPgXVB6WTaiu6wTwFRyuEfm2Pi/mPPqLpmCRPNDjeZE0Fg?=
 =?us-ascii?Q?R1Q3UKbYxzNiaqIj68M05x6LujRAdS9Gg2LP43i8CMsMafZ0nFo+Z5+dF+jO?=
 =?us-ascii?Q?yJdbpueZ1UzXomrXcMlJOLh1KN+vuFzvZe1G3Pt4jnoCxu/GkQo6DuWHtt7U?=
 =?us-ascii?Q?+4b+v3+v0t24iMlTy5XiSvnyqNWilTpZCpHxlecwQBsYTJ52Ma0kqdFM0U9a?=
 =?us-ascii?Q?1YOyzBJEeybxK5xJQb7orQ7mtqmMZ2rCLk4gufx7JqCorv1IDXXfdjXDcc5v?=
 =?us-ascii?Q?wxOO32wARS4GJL9uQ3Whvz+ErNZwGND59bIoohjkQ4TcXZDv0vbR41FK2B+n?=
 =?us-ascii?Q?gFMY3kTCwQ4UatjY8xxjzoXujUJmJYpLiJiroP+lXP7k/IoEab2uDXYMT0Vp?=
 =?us-ascii?Q?mJ5Sn1upAEDiAmRykQdaWsE/bC5GEOWE+9Yzb83xLqnz5BzqQzwTEC5uwBHw?=
 =?us-ascii?Q?vzdOiJLFSrqH6czR0N7gF3Il+56rFtKQCAx9sLrzfBkp5LNI+AktRDyTnWVF?=
 =?us-ascii?Q?vyJVnOd5oTof6bQIknE03jh4RrBA2e1fCYo2oLSuDXdXOj754sPEkP1Qr1Wi?=
 =?us-ascii?Q?L1UljT39/h9gAfRil7jH3Z10n/fAJeORzvQvGM4OOGIsxsyiFTNvA6031Ym5?=
 =?us-ascii?Q?PEsHMWQvwP4aXF1ez/4UUtHLovNswsmXIYoTXqjbQX7ksN4QwkSTjinYocLg?=
 =?us-ascii?Q?coAiFzoe/zo30LhPA8Jet0+jbF824NQce0IDgmAbjjPaHDIwFqIHzcr65pMa?=
 =?us-ascii?Q?+Rybceeff1fiVaLUITPLswaQsO78FJ+iahzehHevEkeJL99B1bmyP6HycEf4?=
 =?us-ascii?Q?+lddMYaqd3jnCtJKF/SpbS7DKSTeSM7RlV6TweOhar1mfYmYRoOXMVIGuFsy?=
 =?us-ascii?Q?47buIfBMXUMxsjRycNGlzS9FokHRWUw5s0SwceRYJ3uBMxNK8bLISHj/0Grc?=
 =?us-ascii?Q?s0X/aXjADt6BUj8p3/0VwYyZb2MM9eCI98zM/JF1UQi0nQNddPMr8y89wsm6?=
 =?us-ascii?Q?QEGkEa0ygO5zucOkvhIC5drLya6ZlHHA1MaHQwP0TMD98b7RoT9ku4GM8RZx?=
 =?us-ascii?Q?V4RZ2bzJEojBnztx9iFF6jOPPXniI/a01QIvZ9cbeck8i4wZBGLGKR6Ritjv?=
 =?us-ascii?Q?hJxO5P3oAF7SmfjDFfCV5mktPvlk+KQcOR5H01ocjl6acxz9C3lJvL97soSJ?=
 =?us-ascii?Q?77nM6lqJ2SPbaJDj/GLLNCyoMMVUc9vlzaRrjp3kKSzSdbT1TSQfmO7UzyBV?=
 =?us-ascii?Q?KXVn8BGdCrDX/Yf4jgfVqvGIHS/KZ1h9Wdn3DDfJ?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b0504c1a-11c3-48f5-c392-08dc005bbbcb
X-MS-Exchange-CrossTenant-AuthSource: PA4PR04MB7709.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Dec 2023 06:28:37.5648
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: fH0afNC2K8uQJmRz5sZuM+fTBAPcSTJrz1+JLmFFJe7G1OPoW5KEFeRf6B7zLRBG7WDUn/cZo/W9zJAl22GnBA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB7717

Hello Maintainers=0D
=0D
This patch fixes assertion in bluetoothctl client process=0D
=0D
After pair command, if the user doesn't provide any input on bluetoothctl=0D
CLI interface after receiving the prompt(yes/no), than subsequent CLI =0D
command will trigger a call to DBUS library function =0D
(dbus_message_get_no_reply) with a NULL message pointer which triggers=0D
assertion in DBUS library causing the bluetoothctl process to crash. =0D
The change is done in confirm_response callback to trigger a DBUS=0D
API error call only if the pending_message pointer is not NULL so =0D
as to avoid the assertion.=0D
=0D
Thank you in advance for your review.=0D
=0D
Warm Regards=0D
Vinit Mehta

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

 client/agent.c | 15 +++++++++------
 client/mgmt.c  | 12 +++++++++---
 2 files changed, 18 insertions(+), 9 deletions(-)

--=20
2.42.0.windows.2