diff mbox series

[BlueZ,v3,resend,6/6] gatt-client: add NULL check to discover_secondary_cb()

Message ID 20240710141621.64394-7-r.smirnov@omp.ru (mailing list archive)
State Accepted
Commit 7a45038dc1e505afbaa49f8dd64fd41dab627f23
Headers show
Series fix errors found by SVACE static analyzer #1 | expand

Commit Message

Roman Smirnov July 10, 2024, 2:16 p.m. UTC 
It is necessary to prevent dereferencing of a NULL pointer.

Found with the SVACE static analysis tool.
---
 V1 -> V3: the patch name has been shortened
 src/shared/gatt-client.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/src/shared/gatt-client.c b/src/shared/gatt-client.c
index b48d739fc..9db3f5211 100644
--- a/src/shared/gatt-client.c
+++ b/src/shared/gatt-client.c
@@ -1276,7 +1276,9 @@  next:
 
 	range = queue_peek_head(op->discov_ranges);
 
-	client->discovery_req = bt_gatt_discover_included_services(client->att,
+	if (range)
+		client->discovery_req = bt_gatt_discover_included_services(
+							client->att,
 							range->start,
 							range->end,
 							discover_incl_cb,