It is necessary to prevent dereferencing of a NULL pointer. Found with the SVACE static analysis tool. --- src/shared/csip.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/shared/csip.c b/src/shared/csip.c index 87b4590d9..a9c57c9bd 100644 --- a/src/shared/csip.c +++ b/src/shared/csip.c @@ -611,6 +611,9 @@ static void foreach_csis_service(struct gatt_db_attribute *attr, struct bt_csip *csip = user_data; struct bt_csis *csis = csip_get_csis(csip); + if (!csis) + return; + csis->service = attr; gatt_db_service_set_claimed(attr, true); -- 2.34.1