* [PATCH BlueZ] gatt: Fix crash when a device is removed
@ 2021-01-19 19:35 Luiz Augusto von Dentz
2021-01-19 20:27 ` [BlueZ] " bluez.test.bot
0 siblings, 1 reply; 3+ messages in thread
From: Luiz Augusto von Dentz @ 2021-01-19 19:35 UTC (permalink / raw)
To: linux-bluetooth
From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
If a device is removed with notifications enabled that would lead to
device_state being freed while att_disconnected has not been called
yet.
gh-issue: https://github.com/bluez/bluez/issues/82
---
src/gatt-database.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/gatt-database.c b/src/gatt-database.c
index d99604826..d635c3214 100644
--- a/src/gatt-database.c
+++ b/src/gatt-database.c
@@ -1350,11 +1350,17 @@ static void send_notification_to_device(void *data, void *user_data)
if (!ccc->value || (notify->conf && !(ccc->value & 0x0002)))
return;
- device = btd_adapter_get_device(notify->database->adapter,
+ device = btd_adapter_find_device(notify->database->adapter,
&device_state->bdaddr,
device_state->bdaddr_type);
- if (!device)
+ if (!device) {
+ /* If ATT has not disconnect yet don't remove the state as it
+ * will eventually be removed when att_disconnected is called.
+ */
+ if (device_state->disc_id)
+ return;
goto remove;
+ }
server = btd_device_get_gatt_server(device);
if (!server) {
--
2.26.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* RE: [BlueZ] gatt: Fix crash when a device is removed
2021-01-19 19:35 [PATCH BlueZ] gatt: Fix crash when a device is removed Luiz Augusto von Dentz
@ 2021-01-19 20:27 ` bluez.test.bot
2021-01-20 22:06 ` Luiz Augusto von Dentz
0 siblings, 1 reply; 3+ messages in thread
From: bluez.test.bot @ 2021-01-19 20:27 UTC (permalink / raw)
To: linux-bluetooth, luiz.dentz
[-- Attachment #1: Type: text/plain, Size: 557 bytes --]
This is automated email and please do not reply to this email!
Dear submitter,
Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=417643
---Test result---
##############################
Test: CheckPatch - PASS
##############################
Test: CheckGitLint - PASS
##############################
Test: CheckBuild - PASS
##############################
Test: MakeCheck - PASS
---
Regards,
Linux Bluetooth
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [BlueZ] gatt: Fix crash when a device is removed
2021-01-19 20:27 ` [BlueZ] " bluez.test.bot
@ 2021-01-20 22:06 ` Luiz Augusto von Dentz
0 siblings, 0 replies; 3+ messages in thread
From: Luiz Augusto von Dentz @ 2021-01-20 22:06 UTC (permalink / raw)
To: linux-bluetooth
Hi,
On Tue, Jan 19, 2021 at 12:27 PM <bluez.test.bot@gmail.com> wrote:
>
> This is automated email and please do not reply to this email!
>
> Dear submitter,
>
> Thank you for submitting the patches to the linux bluetooth mailing list.
> This is a CI test results with your patch series:
> PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=417643
>
> ---Test result---
>
> ##############################
> Test: CheckPatch - PASS
>
> ##############################
> Test: CheckGitLint - PASS
>
> ##############################
> Test: CheckBuild - PASS
>
> ##############################
> Test: MakeCheck - PASS
>
>
>
> ---
> Regards,
> Linux Bluetooth
Pushed.
--
Luiz Augusto von Dentz
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-01-20 23:42 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-19 19:35 [PATCH BlueZ] gatt: Fix crash when a device is removed Luiz Augusto von Dentz
2021-01-19 20:27 ` [BlueZ] " bluez.test.bot
2021-01-20 22:06 ` Luiz Augusto von Dentz
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.