* [PATCH BlueZ 1/3] device: Refactor device_discover_services function
@ 2026-06-08 10:39 Simon Mikuda
2026-06-08 10:39 ` [PATCH BlueZ 2/3] device: Rename start_discovery function Simon Mikuda
` (4 more replies)
0 siblings, 5 replies; 13+ messages in thread
From: Simon Mikuda @ 2026-06-08 10:39 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Simon Mikuda
After refactoring we can reuse function once more in function
void device_bonding_complete(...)
---
profiles/input/server.c | 2 +-
src/adapter.c | 2 +-
src/device.c | 24 +++++++-----------------
src/device.h | 3 ++-
4 files changed, 11 insertions(+), 20 deletions(-)
diff --git a/profiles/input/server.c b/profiles/input/server.c
index 97579152e..93c73f8d2 100644
--- a/profiles/input/server.c
+++ b/profiles/input/server.c
@@ -102,7 +102,7 @@ static void sixaxis_browse_sdp(const bdaddr_t *src, const bdaddr_t *dst,
data->psm = psm;
if (psm == L2CAP_PSM_HIDP_CTRL)
- device_discover_services(device);
+ device_discover_services(device, BDADDR_BREDR, NULL);
device_wait_for_svc_complete(device, sixaxis_sdp_cb, data);
}
diff --git a/src/adapter.c b/src/adapter.c
index 03593d962..cc753fbd6 100644
--- a/src/adapter.c
+++ b/src/adapter.c
@@ -3717,7 +3717,7 @@ static void device_connect_cb(GIOChannel *io, GError *gerr, gpointer user_data)
device_attach_att(device, io);
}
- device_discover_services(device);
+ device_discover_services(device, data->dst_type, NULL);
g_io_channel_unref(io);
dbus_message_unref(data->msg);
diff --git a/src/device.c b/src/device.c
index bea893301..0c2ee7d8d 100644
--- a/src/device.c
+++ b/src/device.c
@@ -2812,10 +2812,7 @@ DBusMessage *device_connect_profiles(struct btd_device *dev,
resolve_services:
DBG("Resolving services for %s", dev->path);
- if (bdaddr_type == BDADDR_BREDR)
- err = device_browse_sdp(dev, msg);
- else
- err = device_browse_gatt(dev, msg);
+ err = device_discover_services(dev, bdaddr_type, msg);
if (err < 0) {
return btd_error_failed(msg, bdaddr_type == BDADDR_BREDR ?
ERR_BREDR_CONN_SDP_SEARCH : ERR_LE_CONN_GATT_BROWSE);
@@ -6834,14 +6831,15 @@ void device_cancel_browse(struct btd_device *device, uint8_t bdaddr_type)
browse_request_free(device->browse);
}
-int device_discover_services(struct btd_device *device)
+int device_discover_services(struct btd_device *device,
+ uint8_t bdaddr_type, DBusMessage *msg)
{
int err;
- if (device->bredr)
- err = device_browse_sdp(device, NULL);
+ if (bdaddr_type == BDADDR_BREDR)
+ err = device_browse_sdp(device, msg);
else
- err = device_browse_gatt(device, NULL);
+ err = device_browse_gatt(device, msg);
if (err == 0 && device->discov_timer) {
timeout_remove(device->discov_timer);
@@ -7354,15 +7352,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type,
DBG("Proceeding with service discovery");
/* If we are initiators remove any discovery timer and just
* start discovering services directly */
- if (device->discov_timer) {
- timeout_remove(device->discov_timer);
- device->discov_timer = 0;
- }
-
- if (bdaddr_type == BDADDR_BREDR)
- device_browse_sdp(device, bonding->msg);
- else
- device_browse_gatt(device, bonding->msg);
+ device_discover_services(device, bdaddr_type, bonding->msg);
bonding_request_free(bonding);
} else if (!state->svc_resolved) {
diff --git a/src/device.h b/src/device.h
index b36f9af66..b890f23d4 100644
--- a/src/device.h
+++ b/src/device.h
@@ -207,7 +207,8 @@ bool device_remove_svc_complete_callback(struct btd_device *dev,
struct btd_service *btd_device_get_service(struct btd_device *dev,
const char *remote_uuid);
-int device_discover_services(struct btd_device *device);
+int device_discover_services(struct btd_device *device,
+ uint8_t bdaddr_type, DBusMessage *msg);
int btd_device_connect_services(struct btd_device *dev, GSList *services);
bool btd_device_flags_enabled(struct btd_device *dev, uint32_t flags);
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH BlueZ 2/3] device: Rename start_discovery function 2026-06-08 10:39 [PATCH BlueZ 1/3] device: Refactor device_discover_services function Simon Mikuda @ 2026-06-08 10:39 ` Simon Mikuda 2026-06-08 10:39 ` [PATCH BlueZ 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda ` (3 subsequent siblings) 4 siblings, 0 replies; 13+ messages in thread From: Simon Mikuda @ 2026-06-08 10:39 UTC (permalink / raw) To: linux-bluetooth; +Cc: Simon Mikuda Rename it to start_discovery_cb to indicate that it is callback function from timer. --- src/device.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/device.c b/src/device.c index 0c2ee7d8d..da70f9cc0 100644 --- a/src/device.c +++ b/src/device.c @@ -7185,7 +7185,7 @@ bool device_is_connectable(struct btd_device *device) return state->connectable; } -static bool start_discovery(gpointer user_data) +static bool start_discovery_cb(gpointer user_data) { struct btd_device *device = user_data; @@ -7364,7 +7364,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, DBG("setting timer for reverse service discovery"); device->discov_timer = timeout_add_seconds( DISCOVERY_TIMER, - start_discovery, + start_discovery_cb, device, NULL); } } @@ -7409,7 +7409,7 @@ unsigned int device_wait_for_svc_complete(struct btd_device *dev, dev->discov_timer = timeout_add_seconds( 0, - start_discovery, + start_discovery_cb, dev, NULL); } -- 2.43.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH BlueZ 3/3] device: Fix returning discovery error for Device.Pair 2026-06-08 10:39 [PATCH BlueZ 1/3] device: Refactor device_discover_services function Simon Mikuda 2026-06-08 10:39 ` [PATCH BlueZ 2/3] device: Rename start_discovery function Simon Mikuda @ 2026-06-08 10:39 ` Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 1/3] device: Refactor device_discover_services function Simon Mikuda ` (2 subsequent siblings) 4 siblings, 0 replies; 13+ messages in thread From: Simon Mikuda @ 2026-06-08 10:39 UTC (permalink / raw) To: linux-bluetooth; +Cc: Simon Mikuda If discovery was requesed from pair request we will report successfull pairing even if there was an error during discovery. --- src/device.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/src/device.c b/src/device.c index da70f9cc0..429fc5e6d 100644 --- a/src/device.c +++ b/src/device.c @@ -7294,6 +7294,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, struct bonding_req *bonding = device->bonding; struct authentication_req *auth = device->authr; struct bearer_state *state = get_state(device, bdaddr_type); + int err; DBG("bonding %p status 0x%02x", bonding, status); @@ -7352,8 +7353,16 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, DBG("Proceeding with service discovery"); /* If we are initiators remove any discovery timer and just * start discovering services directly */ - device_discover_services(device, bdaddr_type, bonding->msg); - + err = device_discover_services(device, bdaddr_type, bonding->msg); + if (err) { + if (device->pending_paired) { + g_dbus_emit_property_changed(dbus_conn, device->path, + DEVICE_INTERFACE, "Paired"); + device->pending_paired = false; + } + /* Disregard browse errors in case of Pair */ + g_dbus_send_reply(dbus_conn, bonding->msg, DBUS_TYPE_INVALID); + } bonding_request_free(bonding); } else if (!state->svc_resolved) { if (!device->browse && !device->discov_timer && -- 2.43.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH BlueZ v2 1/3] device: Refactor device_discover_services function 2026-06-08 10:39 [PATCH BlueZ 1/3] device: Refactor device_discover_services function Simon Mikuda 2026-06-08 10:39 ` [PATCH BlueZ 2/3] device: Rename start_discovery function Simon Mikuda 2026-06-08 10:39 ` [PATCH BlueZ 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda @ 2026-06-08 10:59 ` Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 2/3] device: Rename start_discovery function Simon Mikuda ` (2 more replies) 2026-06-08 11:07 ` [PATCH BlueZ v3 1/3] " Simon Mikuda 2026-06-08 11:40 ` [BlueZ,1/3] " bluez.test.bot 4 siblings, 3 replies; 13+ messages in thread From: Simon Mikuda @ 2026-06-08 10:59 UTC (permalink / raw) To: linux-bluetooth; +Cc: Simon Mikuda After refactoring we can reuse function once more in function void device_bonding_complete(...) --- profiles/input/server.c | 2 +- src/adapter.c | 2 +- src/device.c | 24 +++++++----------------- src/device.h | 3 ++- 4 files changed, 11 insertions(+), 20 deletions(-) diff --git a/profiles/input/server.c b/profiles/input/server.c index 97579152e..93c73f8d2 100644 --- a/profiles/input/server.c +++ b/profiles/input/server.c @@ -102,7 +102,7 @@ static void sixaxis_browse_sdp(const bdaddr_t *src, const bdaddr_t *dst, data->psm = psm; if (psm == L2CAP_PSM_HIDP_CTRL) - device_discover_services(device); + device_discover_services(device, BDADDR_BREDR, NULL); device_wait_for_svc_complete(device, sixaxis_sdp_cb, data); } diff --git a/src/adapter.c b/src/adapter.c index 03593d962..cc753fbd6 100644 --- a/src/adapter.c +++ b/src/adapter.c @@ -3717,7 +3717,7 @@ static void device_connect_cb(GIOChannel *io, GError *gerr, gpointer user_data) device_attach_att(device, io); } - device_discover_services(device); + device_discover_services(device, data->dst_type, NULL); g_io_channel_unref(io); dbus_message_unref(data->msg); diff --git a/src/device.c b/src/device.c index bea893301..0c2ee7d8d 100644 --- a/src/device.c +++ b/src/device.c @@ -2812,10 +2812,7 @@ DBusMessage *device_connect_profiles(struct btd_device *dev, resolve_services: DBG("Resolving services for %s", dev->path); - if (bdaddr_type == BDADDR_BREDR) - err = device_browse_sdp(dev, msg); - else - err = device_browse_gatt(dev, msg); + err = device_discover_services(dev, bdaddr_type, msg); if (err < 0) { return btd_error_failed(msg, bdaddr_type == BDADDR_BREDR ? ERR_BREDR_CONN_SDP_SEARCH : ERR_LE_CONN_GATT_BROWSE); @@ -6834,14 +6831,15 @@ void device_cancel_browse(struct btd_device *device, uint8_t bdaddr_type) browse_request_free(device->browse); } -int device_discover_services(struct btd_device *device) +int device_discover_services(struct btd_device *device, + uint8_t bdaddr_type, DBusMessage *msg) { int err; - if (device->bredr) - err = device_browse_sdp(device, NULL); + if (bdaddr_type == BDADDR_BREDR) + err = device_browse_sdp(device, msg); else - err = device_browse_gatt(device, NULL); + err = device_browse_gatt(device, msg); if (err == 0 && device->discov_timer) { timeout_remove(device->discov_timer); @@ -7354,15 +7352,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, DBG("Proceeding with service discovery"); /* If we are initiators remove any discovery timer and just * start discovering services directly */ - if (device->discov_timer) { - timeout_remove(device->discov_timer); - device->discov_timer = 0; - } - - if (bdaddr_type == BDADDR_BREDR) - device_browse_sdp(device, bonding->msg); - else - device_browse_gatt(device, bonding->msg); + device_discover_services(device, bdaddr_type, bonding->msg); bonding_request_free(bonding); } else if (!state->svc_resolved) { diff --git a/src/device.h b/src/device.h index b36f9af66..b890f23d4 100644 --- a/src/device.h +++ b/src/device.h @@ -207,7 +207,8 @@ bool device_remove_svc_complete_callback(struct btd_device *dev, struct btd_service *btd_device_get_service(struct btd_device *dev, const char *remote_uuid); -int device_discover_services(struct btd_device *device); +int device_discover_services(struct btd_device *device, + uint8_t bdaddr_type, DBusMessage *msg); int btd_device_connect_services(struct btd_device *dev, GSList *services); bool btd_device_flags_enabled(struct btd_device *dev, uint32_t flags); -- 2.43.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH BlueZ v2 2/3] device: Rename start_discovery function 2026-06-08 10:59 ` [PATCH BlueZ v2 1/3] device: Refactor device_discover_services function Simon Mikuda @ 2026-06-08 10:59 ` Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda 2026-06-08 11:41 ` [BlueZ,v2,1/3] device: Refactor device_discover_services function bluez.test.bot 2 siblings, 0 replies; 13+ messages in thread From: Simon Mikuda @ 2026-06-08 10:59 UTC (permalink / raw) To: linux-bluetooth; +Cc: Simon Mikuda Rename it to start_discovery_cb to indicate that it is callback function from timer. --- src/device.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/device.c b/src/device.c index 0c2ee7d8d..da70f9cc0 100644 --- a/src/device.c +++ b/src/device.c @@ -7185,7 +7185,7 @@ bool device_is_connectable(struct btd_device *device) return state->connectable; } -static bool start_discovery(gpointer user_data) +static bool start_discovery_cb(gpointer user_data) { struct btd_device *device = user_data; @@ -7364,7 +7364,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, DBG("setting timer for reverse service discovery"); device->discov_timer = timeout_add_seconds( DISCOVERY_TIMER, - start_discovery, + start_discovery_cb, device, NULL); } } @@ -7409,7 +7409,7 @@ unsigned int device_wait_for_svc_complete(struct btd_device *dev, dev->discov_timer = timeout_add_seconds( 0, - start_discovery, + start_discovery_cb, dev, NULL); } -- 2.43.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH BlueZ v2 3/3] device: Fix returning discovery error for Device.Pair 2026-06-08 10:59 ` [PATCH BlueZ v2 1/3] device: Refactor device_discover_services function Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 2/3] device: Rename start_discovery function Simon Mikuda @ 2026-06-08 10:59 ` Simon Mikuda 2026-06-08 11:41 ` [BlueZ,v2,1/3] device: Refactor device_discover_services function bluez.test.bot 2 siblings, 0 replies; 13+ messages in thread From: Simon Mikuda @ 2026-06-08 10:59 UTC (permalink / raw) To: linux-bluetooth; +Cc: Simon Mikuda If discovery was requesed from pair request we will report successfull pairing even if there was an error during discovery. --- src/device.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/src/device.c b/src/device.c index da70f9cc0..429fc5e6d 100644 --- a/src/device.c +++ b/src/device.c @@ -7294,6 +7294,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, struct bonding_req *bonding = device->bonding; struct authentication_req *auth = device->authr; struct bearer_state *state = get_state(device, bdaddr_type); + int err; DBG("bonding %p status 0x%02x", bonding, status); @@ -7352,8 +7353,16 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, DBG("Proceeding with service discovery"); /* If we are initiators remove any discovery timer and just * start discovering services directly */ - device_discover_services(device, bdaddr_type, bonding->msg); - + err = device_discover_services(device, bdaddr_type, bonding->msg); + if (err) { + if (device->pending_paired) { + g_dbus_emit_property_changed(dbus_conn, device->path, + DEVICE_INTERFACE, "Paired"); + device->pending_paired = false; + } + /* Disregard browse errors in case of Pair */ + g_dbus_send_reply(dbus_conn, bonding->msg, DBUS_TYPE_INVALID); + } bonding_request_free(bonding); } else if (!state->svc_resolved) { if (!device->browse && !device->discov_timer && -- 2.43.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* RE: [BlueZ,v2,1/3] device: Refactor device_discover_services function 2026-06-08 10:59 ` [PATCH BlueZ v2 1/3] device: Refactor device_discover_services function Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 2/3] device: Rename start_discovery function Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda @ 2026-06-08 11:41 ` bluez.test.bot 2 siblings, 0 replies; 13+ messages in thread From: bluez.test.bot @ 2026-06-08 11:41 UTC (permalink / raw) To: linux-bluetooth, simon.mikuda [-- Attachment #1: Type: text/plain, Size: 32452 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=1107770 ---Test result--- Test Summary: CheckPatch FAIL 1.42 seconds GitLint PASS 0.97 seconds BuildEll PASS 20.01 seconds BluezMake FAIL 56.35 seconds MakeCheck FAIL 152.19 seconds MakeDistcheck FAIL 46.47 seconds CheckValgrind FAIL 45.46 seconds CheckSmatch FAIL 105.09 seconds bluezmakeextell FAIL 28.23 seconds IncrementalBuild FAIL 56.54 seconds ScanBuild FAIL 135.62 seconds Details ############################## Test: CheckPatch - FAIL Desc: Run checkpatch.pl script Output: [BlueZ,v2,3/3] device: Fix returning discovery error for Device.Pair WARNING:TYPO_SPELLING: 'successfull' may be misspelled - perhaps 'successful'? #99: If discovery was requesed from pair request we will report successfull ^^^^^^^^^^^ WARNING:LONG_LINE: line length of 82 exceeds 80 columns #123: FILE: src/device.c:7356: + err = device_discover_services(device, bdaddr_type, bonding->msg); WARNING:LONG_LINE: line length of 85 exceeds 80 columns #126: FILE: src/device.c:7359: + g_dbus_emit_property_changed(dbus_conn, device->path, WARNING:LONG_LINE: line length of 86 exceeds 80 columns #131: FILE: src/device.c:7364: + g_dbus_send_reply(dbus_conn, bonding->msg, DBUS_TYPE_INVALID); /github/workspace/src/patch/14616510.patch total: 0 errors, 4 warnings, 25 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. /github/workspace/src/patch/14616510.patch has style problems, please review. NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. ############################## Test: BluezMake - FAIL Desc: Build BlueZ Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... tools/mgmt-tester.c: In function ‘main’: tools/mgmt-tester.c:12990:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without 12990 | int main(int argc, char *argv[]) | ^~~~ make: *** [Makefile:4172: all] Error 2 ############################## Test: MakeCheck - FAIL Desc: Run Bluez Make Check Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make: *** [Makefile:10820: check] Error 2 ############################## Test: MakeDistcheck - FAIL Desc: Run Bluez Make Distcheck Output: ../../src/device.c: In function ‘btd_device_connect_services’: ../../src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from ../../src/device.c:54: ../../src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[2]: *** [Makefile:9539: src/bluetoothd-device.o] Error 1 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [Makefile:4172: all] Error 2 make: *** [Makefile:10741: distcheck] Error 1 ############################## Test: CheckValgrind - FAIL Desc: Run Bluez Make Check with Valgrind Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... tools/mgmt-tester.c: In function ‘main’: tools/mgmt-tester.c:12990:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without 12990 | int main(int argc, char *argv[]) | ^~~~ make: *** [Makefile:10820: check] Error 2 ############################## Test: CheckSmatch - FAIL Desc: Run smatch tool with source Output: src/shared/crypto.c:271:21: warning: Variable length array is used. src/shared/crypto.c:272:23: warning: Variable length array is used. src/shared/gatt-helpers.c:764:31: warning: Variable length array is used. src/shared/gatt-helpers.c:842:31: warning: Variable length array is used. src/shared/gatt-helpers.c:1335:31: warning: Variable length array is used. src/shared/gatt-helpers.c:1366:23: warning: Variable length array is used. src/shared/gatt-server.c:271:25: warning: Variable length array is used. src/shared/gatt-server.c:614:25: warning: Variable length array is used. src/shared/gatt-server.c:712:25: warning: Variable length array is used. src/shared/bap.c:312:25: warning: array of flexible structures src/shared/bap.c: note: in included file: ./src/shared/ascs.h:88:25: warning: array of flexible structures src/shared/shell.c: note: in included file (through /usr/include/readline/readline.h): /usr/include/readline/rltypedefs.h:35:23: warning: non-ANSI function declaration of function 'Function' /usr/include/readline/rltypedefs.h:36:25: warning: non-ANSI function declaration of function 'VFunction' /usr/include/readline/rltypedefs.h:37:27: warning: non-ANSI function declaration of function 'CPFunction' /usr/include/readline/rltypedefs.h:38:29: warning: non-ANSI function declaration of function 'CPPFunction' src/shared/crypto.c:271:21: warning: Variable length array is used. src/shared/crypto.c:272:23: warning: Variable length array is used. src/shared/gatt-helpers.c:764:31: warning: Variable length array is used. src/shared/gatt-helpers.c:842:31: warning: Variable length array is used. src/shared/gatt-helpers.c:1335:31: warning: Variable length array is used. src/shared/gatt-helpers.c:1366:23: warning: Variable length array is used. src/shared/gatt-server.c:271:25: warning: Variable length array is used. src/shared/gatt-server.c:614:25: warning: Variable length array is used. src/shared/gatt-server.c:712:25: warning: Variable length array is used. src/shared/bap.c:312:25: warning: array of flexible structures src/shared/bap.c: note: in included file: ./src/shared/ascs.h:88:25: warning: array of flexible structures src/shared/shell.c: note: in included file (through /usr/include/readline/readline.h): /usr/include/readline/rltypedefs.h:35:23: warning: non-ANSI function declaration of function 'Function' /usr/include/readline/rltypedefs.h:36:25: warning: non-ANSI function declaration of function 'VFunction' /usr/include/readline/rltypedefs.h:37:27: warning: non-ANSI function declaration of function 'CPFunction' /usr/include/readline/rltypedefs.h:38:29: warning: non-ANSI function declaration of function 'CPPFunction' tools/mesh-cfgtest.c:1453:17: warning: unknown escape sequence: '\%' tools/sco-tester.c: note: in included file: ./lib/bluetooth/bluetooth.h:232:15: warning: array of flexible structures ./lib/bluetooth/bluetooth.h:237:31: warning: array of flexible structures tools/bneptest.c:634:39: warning: unknown escape sequence: '\%' tools/seq2bseq.c:57:26: warning: Variable length array is used. tools/obex-client-tool.c: note: in included file (through /usr/include/readline/readline.h): /usr/include/readline/rltypedefs.h:35:23: warning: non-ANSI function declaration of function 'Function' /usr/include/readline/rltypedefs.h:36:25: warning: non-ANSI function declaration of function 'VFunction' /usr/include/readline/rltypedefs.h:37:27: warning: non-ANSI function declaration of function 'CPFunction' /usr/include/readline/rltypedefs.h:38:29: warning: non-ANSI function declaration of function 'CPPFunction' client/btpclient/gatt.c: note: in included file: ./src/shared/btp.h:335:41: warning: array of flexible structures ./src/shared/btp.h:340:55: warning: array of flexible structures ./src/shared/btp.h:363:47: warning: array of flexible structures ./src/shared/btp.h:392:42: warning: array of flexible structures src/advertising.c: note: in included file: ./src/shared/mgmt.h:95:25: error: redefinition of unsigned int enum mgmt_io_capability src/agent.c: note: in included file: src/shared/queue.h:19:20: error: redefinition of struct queue_entry src/adv_monitor.c: note: in included file: ./src/shared/mgmt.h:95:25: error: redefinition of unsigned int enum mgmt_io_capability src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:4172: all] Error 2 ############################## Test: bluezmakeextell - FAIL Desc: Build Bluez with External ELL Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:4172: all] Error 2 ############################## Test: IncrementalBuild - FAIL Desc: Incremental build with the patches in the series Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... tools/mgmt-tester.c: In function ‘main’: tools/mgmt-tester.c:12990:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without 12990 | int main(int argc, char *argv[]) | ^~~~ make: *** [Makefile:4172: all] Error 2 [BlueZ,v2,1/3] device: Refactor device_discover_services function src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... tools/mgmt-tester.c: In function ‘main’: tools/mgmt-tester.c:12990:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without 12990 | int main(int argc, char *argv[]) | ^~~~ make: *** [Makefile:4172: all] Error 2 ############################## Test: ScanBuild - FAIL Desc: Run Scan Build Output: src/shared/gatt-client.c:447:21: warning: Use of memory after it is freed gatt_db_unregister(op->client->db, op->db_id); ^~~~~~~~~~ src/shared/gatt-client.c:692:2: warning: Use of memory after it is freed discovery_op_complete(op, false, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:992:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1098:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1292:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1357:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1632:6: warning: Use of memory after it is freed if (read_db_hash(op)) { ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1637:2: warning: Use of memory after it is freed discover_all(op); ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1693:56: warning: Use of memory after it is freed notify_data->chrc->ccc_write_id = notify_data->att_id = att_id; ~~~~~~~~~~~~~~~~~~~ ^ src/shared/gatt-client.c:2146:6: warning: Use of memory after it is freed if (read_db_hash(op)) { ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:2154:8: warning: Use of memory after it is freed discovery_op_ref(op), ^~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3332:2: warning: Use of memory after it is freed complete_write_long_op(req, success, 0, false); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3354:2: warning: Use of memory after it is freed request_unref(req); ^~~~~~~~~~~~~~~~~~ 13 warnings generated. src/shared/bap.c:1529:8: warning: Use of memory after it is freed bap = bt_bap_ref_safe(bap); ^~~~~~~~~~~~~~~~~~~~ src/shared/bap.c:2340:20: warning: Use of memory after it is freed return queue_find(stream->bap->streams, NULL, stream); ^~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/shared/gatt-client.c:447:21: warning: Use of memory after it is freed gatt_db_unregister(op->client->db, op->db_id); ^~~~~~~~~~ src/shared/gatt-client.c:692:2: warning: Use of memory after it is freed discovery_op_complete(op, false, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:992:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1098:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1292:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1357:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1632:6: warning: Use of memory after it is freed if (read_db_hash(op)) { ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1637:2: warning: Use of memory after it is freed discover_all(op); ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1693:56: warning: Use of memory after it is freed notify_data->chrc->ccc_write_id = notify_data->att_id = att_id; ~~~~~~~~~~~~~~~~~~~ ^ src/shared/gatt-client.c:2146:6: warning: Use of memory after it is freed if (read_db_hash(op)) { ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:2154:8: warning: Use of memory after it is freed discovery_op_ref(op), ^~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3332:2: warning: Use of memory after it is freed complete_write_long_op(req, success, 0, false); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3354:2: warning: Use of memory after it is freed request_unref(req); ^~~~~~~~~~~~~~~~~~ 13 warnings generated. tools/hciattach.c:817:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 10)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:865:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 4)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:887:8: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 10)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:909:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 4)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:930:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 4)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:974:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 6)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ 6 warnings generated. src/shared/bap.c:1529:8: warning: Use of memory after it is freed bap = bt_bap_ref_safe(bap); ^~~~~~~~~~~~~~~~~~~~ src/shared/bap.c:2340:20: warning: Use of memory after it is freed return queue_find(stream->bap->streams, NULL, stream); ^~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/oui.c:50:2: warning: Value stored to 'hwdb' is never read hwdb = udev_hwdb_unref(hwdb); ^ ~~~~~~~~~~~~~~~~~~~~~ src/oui.c:53:2: warning: Value stored to 'udev' is never read udev = udev_unref(udev); ^ ~~~~~~~~~~~~~~~~ 2 warnings generated. tools/rfcomm.c:234:3: warning: Value stored to 'i' is never read i = execvp(cmdargv[0], cmdargv); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:234:7: warning: Null pointer passed to 1st parameter expecting 'nonnull' i = execvp(cmdargv[0], cmdargv); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:354:8: warning: Although the value stored to 'fd' is used in the enclosing expression, the value is never actually read from 'fd' if ((fd = open(devname, O_RDONLY | O_NOCTTY)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:497:14: warning: Assigned value is garbage or undefined req.channel = raddr.rc_channel; ^ ~~~~~~~~~~~~~~~~ tools/rfcomm.c:515:8: warning: Although the value stored to 'fd' is used in the enclosing expression, the value is never actually read from 'fd' if ((fd = open(devname, O_RDONLY | O_NOCTTY)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 5 warnings generated. tools/ciptool.c:351:7: warning: 5th function call argument is an uninitialized value sk = do_connect(ctl, dev_id, &src, &dst, psm, (1 << CMTP_LOOPBACK)); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. src/sdp-xml.c:126:10: warning: Assigned value is garbage or undefined buf[1] = data[i + 1]; ^ ~~~~~~~~~~~ src/sdp-xml.c:306:11: warning: Assigned value is garbage or undefined buf[1] = data[i + 1]; ^ ~~~~~~~~~~~ src/sdp-xml.c:344:11: warning: Assigned value is garbage or undefined buf[1] = data[i + 1]; ^ ~~~~~~~~~~~ 3 warnings generated. tools/sdptool.c:941:26: warning: Result of 'malloc' is converted to a pointer of type 'uint32_t', which is incompatible with sizeof operand type 'int' uint32_t *value_int = malloc(sizeof(int)); ~~~~~~~~~~ ^~~~~~ ~~~~~~~~~~~ tools/sdptool.c:980:4: warning: 1st function call argument is an uninitialized value free(allocArray[i]); ^~~~~~~~~~~~~~~~~~~ tools/sdptool.c:3777:2: warning: Potential leak of memory pointed to by 'si.name' return add_service(0, &si); ^~~~~~~~~~~~~~~~~~~~~~~~~~ tools/sdptool.c:4112:4: warning: Potential leak of memory pointed to by 'context.svc' return -1; ^~~~~~~~~ 4 warnings generated. tools/avtest.c:243:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:253:5: warning: Value stored to 'len' is never read len = write(sk, buf, 4); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:262:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:276:5: warning: Value stored to 'len' is never read len = write(sk, buf, ^ ~~~~~~~~~~~~~~ tools/avtest.c:283:5: warning: Value stored to 'len' is never read len = write(sk, buf, ^ ~~~~~~~~~~~~~~ tools/avtest.c:290:5: warning: Value stored to 'len' is never read len = write(sk, buf, ^ ~~~~~~~~~~~~~~ tools/avtest.c:297:5: warning: Value stored to 'len' is never read len = write(sk, buf, ^ ~~~~~~~~~~~~~~ tools/avtest.c:309:5: warning: Value stored to 'len' is never read len = write(sk, buf, 4); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:313:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:322:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:326:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:335:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:342:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:364:5: warning: Value stored to 'len' is never read len = write(sk, buf, 4); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:368:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:377:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:381:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:394:5: warning: Value stored to 'len' is never read len = write(sk, buf, 4); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:398:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:405:4: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:415:4: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:580:3: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:588:3: warning: Value stored to 'len' is never read len = write(sk, buf, invalid ? 2 : 3); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:602:3: warning: Value stored to 'len' is never read len = write(sk, buf, 4 + media_transport_size); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:615:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:625:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:637:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:652:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:664:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:673:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:680:3: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:716:2: warning: Value stored to 'len' is never read len = write(sk, buf, AVCTP_HEADER_LENGTH + sizeof(play_pressed)); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 32 warnings generated. tools/btproxy.c:836:15: warning: Null pointer passed to 1st parameter expecting 'nonnull' tcp_port = atoi(optarg); ^~~~~~~~~~~~ tools/btproxy.c:839:8: warning: Null pointer passed to 1st parameter expecting 'nonnull' if (strlen(optarg) > 3 && !strncmp(optarg, "hci", 3)) ^~~~~~~~~~~~~~ 2 warnings generated. tools/create-image.c:76:3: warning: Value stored to 'fd' is never read fd = -1; ^ ~~ tools/create-image.c:84:3: warning: Value stored to 'fd' is never read fd = -1; ^ ~~ tools/create-image.c:92:3: warning: Value stored to 'fd' is never read fd = -1; ^ ~~ tools/create-image.c:105:2: warning: Value stored to 'fd' is never read fd = -1; ^ ~~ 4 warnings generated. tools/btgatt-client.c:1822:2: warning: Value stored to 'argv' is never read argv += optind; ^ ~~~~~~ 1 warning generated. tools/check-selftest.c:42:3: warning: Value stored to 'ptr' is never read ptr = fgets(result, sizeof(result), fp); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. tools/btgatt-server.c:1204:2: warning: Value stored to 'argv' is never read argv -= optind; ^ ~~~~~~ 1 warning generated. tools/gatt-service.c:294:2: warning: 2nd function call argument is an uninitialized value chr_write(chr, value, len); ^~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. tools/obex-server-tool.c:133:13: warning: Null pointer passed to 1st parameter expecting 'nonnull' data->fd = open(name, O_WRONLY | O_CREAT | O_NOCTTY, 0600); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/obex-server-tool.c:192:13: warning: Null pointer passed to 1st parameter expecting 'nonnull' data->fd = open(name, O_RDONLY | O_NOCTTY, 0); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 warnings generated. tools/test-runner.c:1370:2: warning: Address of stack memory associated with local variable 'kernel_path' is still referred to by the global variable 'kernel_image' upon returning to the caller. This will be a dangling reference return EXIT_SUCCESS; ^~~~~~~~~~~~~~~~~~~ 1 warning generated. client/btpclient/btpclientctl.c:402:3: warning: Value stored to 'bit' is never read bit = 0; ^ ~ client/btpclient/btpclientctl.c:1655:2: warning: Null pointer passed to 2nd parameter expecting 'nonnull' memcpy(cp->data, ad_data, ad_len); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/sdp-client.c:353:14: warning: Access to field 'cb' results in a dereference of a null pointer (*ctxt)->cb = cb; ~~~~~~~~~~~~^~~~ 1 warning generated. src/sdpd-request.c:209:13: warning: Result of 'malloc' is converted to a pointer of type 'char', which is incompatible with sizeof operand type 'uint16_t' pElem = malloc(sizeof(uint16_t)); ^~~~~~ ~~~~~~~~~~~~~~~~ src/sdpd-request.c:237:13: warning: Result of 'malloc' is converted to a pointer of type 'char', which is incompatible with sizeof operand type 'uint32_t' pElem = malloc(sizeof(uint32_t)); ^~~~~~ ~~~~~~~~~~~~~~~~ 2 warnings generated. src/gatt-database.c:1171:10: warning: Value stored to 'bits' during its initialization is never read uint8_t bits[] = { BT_GATT_CHRC_CLI_FEAT_ROBUST_CACHING, ^~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... src/gatt-client.c:1569:2: warning: Use of memory after it is freed notify_client_unref(client); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. make: *** [Makefile:4172: all] Error 2 https://github.com/bluez/bluez/pull/2186 --- Regards, Linux Bluetooth ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH BlueZ v3 1/3] device: Refactor device_discover_services function 2026-06-08 10:39 [PATCH BlueZ 1/3] device: Refactor device_discover_services function Simon Mikuda ` (2 preceding siblings ...) 2026-06-08 10:59 ` [PATCH BlueZ v2 1/3] device: Refactor device_discover_services function Simon Mikuda @ 2026-06-08 11:07 ` Simon Mikuda 2026-06-08 11:07 ` [PATCH BlueZ v3 2/3] device: Rename start_discovery function Simon Mikuda ` (3 more replies) 2026-06-08 11:40 ` [BlueZ,1/3] " bluez.test.bot 4 siblings, 4 replies; 13+ messages in thread From: Simon Mikuda @ 2026-06-08 11:07 UTC (permalink / raw) To: linux-bluetooth; +Cc: Simon Mikuda After refactoring we can reuse function once more in function void device_bonding_complete(...) --- profiles/input/server.c | 2 +- src/adapter.c | 2 +- src/device.c | 26 ++++++++------------------ src/device.h | 3 ++- 4 files changed, 12 insertions(+), 21 deletions(-) diff --git a/profiles/input/server.c b/profiles/input/server.c index 97579152e..93c73f8d2 100644 --- a/profiles/input/server.c +++ b/profiles/input/server.c @@ -102,7 +102,7 @@ static void sixaxis_browse_sdp(const bdaddr_t *src, const bdaddr_t *dst, data->psm = psm; if (psm == L2CAP_PSM_HIDP_CTRL) - device_discover_services(device); + device_discover_services(device, BDADDR_BREDR, NULL); device_wait_for_svc_complete(device, sixaxis_sdp_cb, data); } diff --git a/src/adapter.c b/src/adapter.c index 03593d962..cc753fbd6 100644 --- a/src/adapter.c +++ b/src/adapter.c @@ -3717,7 +3717,7 @@ static void device_connect_cb(GIOChannel *io, GError *gerr, gpointer user_data) device_attach_att(device, io); } - device_discover_services(device); + device_discover_services(device, data->dst_type, NULL); g_io_channel_unref(io); dbus_message_unref(data->msg); diff --git a/src/device.c b/src/device.c index bea893301..223e25102 100644 --- a/src/device.c +++ b/src/device.c @@ -2735,7 +2735,7 @@ int btd_device_connect_services(struct btd_device *dev, GSList *services) if (!dev->bredr_state.svc_resolved) { int err; - err = device_discover_services(dev); + err = device_discover_services(dev, BDADDR_BREDR, NULL); if (err) return err; @@ -2812,10 +2812,7 @@ DBusMessage *device_connect_profiles(struct btd_device *dev, resolve_services: DBG("Resolving services for %s", dev->path); - if (bdaddr_type == BDADDR_BREDR) - err = device_browse_sdp(dev, msg); - else - err = device_browse_gatt(dev, msg); + err = device_discover_services(dev, bdaddr_type, msg); if (err < 0) { return btd_error_failed(msg, bdaddr_type == BDADDR_BREDR ? ERR_BREDR_CONN_SDP_SEARCH : ERR_LE_CONN_GATT_BROWSE); @@ -6834,14 +6831,15 @@ void device_cancel_browse(struct btd_device *device, uint8_t bdaddr_type) browse_request_free(device->browse); } -int device_discover_services(struct btd_device *device) +int device_discover_services(struct btd_device *device, + uint8_t bdaddr_type, DBusMessage *msg) { int err; - if (device->bredr) - err = device_browse_sdp(device, NULL); + if (bdaddr_type == BDADDR_BREDR) + err = device_browse_sdp(device, msg); else - err = device_browse_gatt(device, NULL); + err = device_browse_gatt(device, msg); if (err == 0 && device->discov_timer) { timeout_remove(device->discov_timer); @@ -7354,15 +7352,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, DBG("Proceeding with service discovery"); /* If we are initiators remove any discovery timer and just * start discovering services directly */ - if (device->discov_timer) { - timeout_remove(device->discov_timer); - device->discov_timer = 0; - } - - if (bdaddr_type == BDADDR_BREDR) - device_browse_sdp(device, bonding->msg); - else - device_browse_gatt(device, bonding->msg); + device_discover_services(device, bdaddr_type, bonding->msg); bonding_request_free(bonding); } else if (!state->svc_resolved) { diff --git a/src/device.h b/src/device.h index b36f9af66..b890f23d4 100644 --- a/src/device.h +++ b/src/device.h @@ -207,7 +207,8 @@ bool device_remove_svc_complete_callback(struct btd_device *dev, struct btd_service *btd_device_get_service(struct btd_device *dev, const char *remote_uuid); -int device_discover_services(struct btd_device *device); +int device_discover_services(struct btd_device *device, + uint8_t bdaddr_type, DBusMessage *msg); int btd_device_connect_services(struct btd_device *dev, GSList *services); bool btd_device_flags_enabled(struct btd_device *dev, uint32_t flags); -- 2.43.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH BlueZ v3 2/3] device: Rename start_discovery function 2026-06-08 11:07 ` [PATCH BlueZ v3 1/3] " Simon Mikuda @ 2026-06-08 11:07 ` Simon Mikuda 2026-06-08 11:07 ` [PATCH BlueZ v3 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda ` (2 subsequent siblings) 3 siblings, 0 replies; 13+ messages in thread From: Simon Mikuda @ 2026-06-08 11:07 UTC (permalink / raw) To: linux-bluetooth; +Cc: Simon Mikuda Rename it to start_discovery_cb to indicate that it is callback function from timer. --- src/device.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/device.c b/src/device.c index 223e25102..b0c1960ff 100644 --- a/src/device.c +++ b/src/device.c @@ -7185,7 +7185,7 @@ bool device_is_connectable(struct btd_device *device) return state->connectable; } -static bool start_discovery(gpointer user_data) +static bool start_discovery_cb(gpointer user_data) { struct btd_device *device = user_data; @@ -7364,7 +7364,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, DBG("setting timer for reverse service discovery"); device->discov_timer = timeout_add_seconds( DISCOVERY_TIMER, - start_discovery, + start_discovery_cb, device, NULL); } } @@ -7409,7 +7409,7 @@ unsigned int device_wait_for_svc_complete(struct btd_device *dev, dev->discov_timer = timeout_add_seconds( 0, - start_discovery, + start_discovery_cb, dev, NULL); } -- 2.43.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH BlueZ v3 3/3] device: Fix returning discovery error for Device.Pair 2026-06-08 11:07 ` [PATCH BlueZ v3 1/3] " Simon Mikuda 2026-06-08 11:07 ` [PATCH BlueZ v3 2/3] device: Rename start_discovery function Simon Mikuda @ 2026-06-08 11:07 ` Simon Mikuda 2026-06-08 12:25 ` [BlueZ,v3,1/3] device: Refactor device_discover_services function bluez.test.bot 2026-06-08 18:20 ` [PATCH BlueZ v3 1/3] " patchwork-bot+bluetooth 3 siblings, 0 replies; 13+ messages in thread From: Simon Mikuda @ 2026-06-08 11:07 UTC (permalink / raw) To: linux-bluetooth; +Cc: Simon Mikuda If discovery was requesed from pair request we will report successfull pairing even if there was an error during discovery. --- src/device.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/src/device.c b/src/device.c index b0c1960ff..5f9f7ca99 100644 --- a/src/device.c +++ b/src/device.c @@ -7294,6 +7294,7 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, struct bonding_req *bonding = device->bonding; struct authentication_req *auth = device->authr; struct bearer_state *state = get_state(device, bdaddr_type); + int err; DBG("bonding %p status 0x%02x", bonding, status); @@ -7352,8 +7353,16 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, DBG("Proceeding with service discovery"); /* If we are initiators remove any discovery timer and just * start discovering services directly */ - device_discover_services(device, bdaddr_type, bonding->msg); - + err = device_discover_services(device, bdaddr_type, bonding->msg); + if (err) { + if (device->pending_paired) { + g_dbus_emit_property_changed(dbus_conn, device->path, + DEVICE_INTERFACE, "Paired"); + device->pending_paired = false; + } + /* Disregard browse errors in case of Pair */ + g_dbus_send_reply(dbus_conn, bonding->msg, DBUS_TYPE_INVALID); + } bonding_request_free(bonding); } else if (!state->svc_resolved) { if (!device->browse && !device->discov_timer && -- 2.43.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* RE: [BlueZ,v3,1/3] device: Refactor device_discover_services function 2026-06-08 11:07 ` [PATCH BlueZ v3 1/3] " Simon Mikuda 2026-06-08 11:07 ` [PATCH BlueZ v3 2/3] device: Rename start_discovery function Simon Mikuda 2026-06-08 11:07 ` [PATCH BlueZ v3 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda @ 2026-06-08 12:25 ` bluez.test.bot 2026-06-08 18:20 ` [PATCH BlueZ v3 1/3] " patchwork-bot+bluetooth 3 siblings, 0 replies; 13+ messages in thread From: bluez.test.bot @ 2026-06-08 12:25 UTC (permalink / raw) To: linux-bluetooth, simon.mikuda [-- Attachment #1: Type: text/plain, Size: 2454 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=1107774 ---Test result--- Test Summary: CheckPatch FAIL 1.45 seconds GitLint PASS 0.99 seconds BuildEll PASS 20.15 seconds BluezMake PASS 611.00 seconds MakeCheck PASS 1.18 seconds MakeDistcheck PASS 235.33 seconds CheckValgrind PASS 201.99 seconds CheckSmatch PASS 324.63 seconds bluezmakeextell PASS 166.14 seconds IncrementalBuild PASS 619.73 seconds ScanBuild PASS 928.74 seconds Details ############################## Test: CheckPatch - FAIL Desc: Run checkpatch.pl script Output: [BlueZ,v3,3/3] device: Fix returning discovery error for Device.Pair WARNING:TYPO_SPELLING: 'successfull' may be misspelled - perhaps 'successful'? #99: If discovery was requesed from pair request we will report successfull ^^^^^^^^^^^ WARNING:LONG_LINE: line length of 82 exceeds 80 columns #123: FILE: src/device.c:7356: + err = device_discover_services(device, bdaddr_type, bonding->msg); WARNING:LONG_LINE: line length of 85 exceeds 80 columns #126: FILE: src/device.c:7359: + g_dbus_emit_property_changed(dbus_conn, device->path, WARNING:LONG_LINE: line length of 86 exceeds 80 columns #131: FILE: src/device.c:7364: + g_dbus_send_reply(dbus_conn, bonding->msg, DBUS_TYPE_INVALID); /github/workspace/src/patch/14616520.patch total: 0 errors, 4 warnings, 25 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. /github/workspace/src/patch/14616520.patch has style problems, please review. NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. https://github.com/bluez/bluez/pull/2187 --- Regards, Linux Bluetooth ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH BlueZ v3 1/3] device: Refactor device_discover_services function 2026-06-08 11:07 ` [PATCH BlueZ v3 1/3] " Simon Mikuda ` (2 preceding siblings ...) 2026-06-08 12:25 ` [BlueZ,v3,1/3] device: Refactor device_discover_services function bluez.test.bot @ 2026-06-08 18:20 ` patchwork-bot+bluetooth 3 siblings, 0 replies; 13+ messages in thread From: patchwork-bot+bluetooth @ 2026-06-08 18:20 UTC (permalink / raw) To: Simon Mikuda; +Cc: linux-bluetooth Hello: This series was applied to bluetooth/bluez.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Mon, 8 Jun 2026 13:07:41 +0200 you wrote: > After refactoring we can reuse function once more in function > void device_bonding_complete(...) > --- > profiles/input/server.c | 2 +- > src/adapter.c | 2 +- > src/device.c | 26 ++++++++------------------ > src/device.h | 3 ++- > 4 files changed, 12 insertions(+), 21 deletions(-) Here is the summary with links: - [BlueZ,v3,1/3] device: Refactor device_discover_services function https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=56835e9061de - [BlueZ,v3,2/3] device: Rename start_discovery function https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=415955bb70ba - [BlueZ,v3,3/3] device: Fix returning discovery error for Device.Pair (no matching commit) You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html ^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [BlueZ,1/3] device: Refactor device_discover_services function 2026-06-08 10:39 [PATCH BlueZ 1/3] device: Refactor device_discover_services function Simon Mikuda ` (3 preceding siblings ...) 2026-06-08 11:07 ` [PATCH BlueZ v3 1/3] " Simon Mikuda @ 2026-06-08 11:40 ` bluez.test.bot 4 siblings, 0 replies; 13+ messages in thread From: bluez.test.bot @ 2026-06-08 11:40 UTC (permalink / raw) To: linux-bluetooth, simon.mikuda [-- Attachment #1: Type: text/plain, Size: 32446 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=1107759 ---Test result--- Test Summary: CheckPatch FAIL 1.45 seconds GitLint PASS 0.99 seconds BuildEll PASS 20.46 seconds BluezMake FAIL 54.23 seconds MakeCheck FAIL 160.82 seconds MakeDistcheck FAIL 48.37 seconds CheckValgrind FAIL 45.48 seconds CheckSmatch FAIL 100.92 seconds bluezmakeextell FAIL 28.02 seconds IncrementalBuild FAIL 55.13 seconds ScanBuild FAIL 126.20 seconds Details ############################## Test: CheckPatch - FAIL Desc: Run checkpatch.pl script Output: [BlueZ,3/3] device: Fix returning discovery error for Device.Pair WARNING:TYPO_SPELLING: 'successfull' may be misspelled - perhaps 'successful'? #98: If discovery was requesed from pair request we will report successfull ^^^^^^^^^^^ WARNING:LONG_LINE: line length of 82 exceeds 80 columns #122: FILE: src/device.c:7356: + err = device_discover_services(device, bdaddr_type, bonding->msg); WARNING:LONG_LINE: line length of 85 exceeds 80 columns #125: FILE: src/device.c:7359: + g_dbus_emit_property_changed(dbus_conn, device->path, WARNING:LONG_LINE: line length of 86 exceeds 80 columns #130: FILE: src/device.c:7364: + g_dbus_send_reply(dbus_conn, bonding->msg, DBUS_TYPE_INVALID); /github/workspace/src/patch/14616455.patch total: 0 errors, 4 warnings, 25 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. /github/workspace/src/patch/14616455.patch has style problems, please review. NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. ############################## Test: BluezMake - FAIL Desc: Build BlueZ Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... tools/mgmt-tester.c: In function ‘main’: tools/mgmt-tester.c:12990:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without 12990 | int main(int argc, char *argv[]) | ^~~~ make: *** [Makefile:4172: all] Error 2 ############################## Test: MakeCheck - FAIL Desc: Run Bluez Make Check Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make: *** [Makefile:10820: check] Error 2 ############################## Test: MakeDistcheck - FAIL Desc: Run Bluez Make Distcheck Output: ../../src/device.c: In function ‘btd_device_connect_services’: ../../src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from ../../src/device.c:54: ../../src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[2]: *** [Makefile:9539: src/bluetoothd-device.o] Error 1 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [Makefile:4172: all] Error 2 make: *** [Makefile:10741: distcheck] Error 1 ############################## Test: CheckValgrind - FAIL Desc: Run Bluez Make Check with Valgrind Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... tools/mgmt-tester.c: In function ‘main’: tools/mgmt-tester.c:12990:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without 12990 | int main(int argc, char *argv[]) | ^~~~ make: *** [Makefile:10820: check] Error 2 ############################## Test: CheckSmatch - FAIL Desc: Run smatch tool with source Output: src/shared/crypto.c:271:21: warning: Variable length array is used. src/shared/crypto.c:272:23: warning: Variable length array is used. src/shared/gatt-helpers.c:764:31: warning: Variable length array is used. src/shared/gatt-helpers.c:842:31: warning: Variable length array is used. src/shared/gatt-helpers.c:1335:31: warning: Variable length array is used. src/shared/gatt-helpers.c:1366:23: warning: Variable length array is used. src/shared/gatt-server.c:271:25: warning: Variable length array is used. src/shared/gatt-server.c:614:25: warning: Variable length array is used. src/shared/gatt-server.c:712:25: warning: Variable length array is used. src/shared/bap.c:312:25: warning: array of flexible structures src/shared/bap.c: note: in included file: ./src/shared/ascs.h:88:25: warning: array of flexible structures src/shared/shell.c: note: in included file (through /usr/include/readline/readline.h): /usr/include/readline/rltypedefs.h:35:23: warning: non-ANSI function declaration of function 'Function' /usr/include/readline/rltypedefs.h:36:25: warning: non-ANSI function declaration of function 'VFunction' /usr/include/readline/rltypedefs.h:37:27: warning: non-ANSI function declaration of function 'CPFunction' /usr/include/readline/rltypedefs.h:38:29: warning: non-ANSI function declaration of function 'CPPFunction' src/shared/crypto.c:271:21: warning: Variable length array is used. src/shared/crypto.c:272:23: warning: Variable length array is used. src/shared/gatt-helpers.c:764:31: warning: Variable length array is used. src/shared/gatt-helpers.c:842:31: warning: Variable length array is used. src/shared/gatt-helpers.c:1335:31: warning: Variable length array is used. src/shared/gatt-helpers.c:1366:23: warning: Variable length array is used. src/shared/gatt-server.c:271:25: warning: Variable length array is used. src/shared/gatt-server.c:614:25: warning: Variable length array is used. src/shared/gatt-server.c:712:25: warning: Variable length array is used. src/shared/bap.c:312:25: warning: array of flexible structures src/shared/bap.c: note: in included file: ./src/shared/ascs.h:88:25: warning: array of flexible structures src/shared/shell.c: note: in included file (through /usr/include/readline/readline.h): /usr/include/readline/rltypedefs.h:35:23: warning: non-ANSI function declaration of function 'Function' /usr/include/readline/rltypedefs.h:36:25: warning: non-ANSI function declaration of function 'VFunction' /usr/include/readline/rltypedefs.h:37:27: warning: non-ANSI function declaration of function 'CPFunction' /usr/include/readline/rltypedefs.h:38:29: warning: non-ANSI function declaration of function 'CPPFunction' tools/mesh-cfgtest.c:1453:17: warning: unknown escape sequence: '\%' tools/sco-tester.c: note: in included file: ./lib/bluetooth/bluetooth.h:232:15: warning: array of flexible structures ./lib/bluetooth/bluetooth.h:237:31: warning: array of flexible structures tools/bneptest.c:634:39: warning: unknown escape sequence: '\%' tools/seq2bseq.c:57:26: warning: Variable length array is used. tools/obex-client-tool.c: note: in included file (through /usr/include/readline/readline.h): /usr/include/readline/rltypedefs.h:35:23: warning: non-ANSI function declaration of function 'Function' /usr/include/readline/rltypedefs.h:36:25: warning: non-ANSI function declaration of function 'VFunction' /usr/include/readline/rltypedefs.h:37:27: warning: non-ANSI function declaration of function 'CPFunction' /usr/include/readline/rltypedefs.h:38:29: warning: non-ANSI function declaration of function 'CPPFunction' client/btpclient/gatt.c: note: in included file: ./src/shared/btp.h:335:41: warning: array of flexible structures ./src/shared/btp.h:340:55: warning: array of flexible structures ./src/shared/btp.h:363:47: warning: array of flexible structures ./src/shared/btp.h:392:42: warning: array of flexible structures src/advertising.c: note: in included file: ./src/shared/mgmt.h:95:25: error: redefinition of unsigned int enum mgmt_io_capability src/agent.c: note: in included file: src/shared/queue.h:19:20: error: redefinition of struct queue_entry src/adv_monitor.c: note: in included file: ./src/shared/mgmt.h:95:25: error: redefinition of unsigned int enum mgmt_io_capability src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:4172: all] Error 2 ############################## Test: bluezmakeextell - FAIL Desc: Build Bluez with External ELL Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:4172: all] Error 2 ############################## Test: IncrementalBuild - FAIL Desc: Incremental build with the patches in the series Output: src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... tools/mgmt-tester.c: In function ‘main’: tools/mgmt-tester.c:12990:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without 12990 | int main(int argc, char *argv[]) | ^~~~ make: *** [Makefile:4172: all] Error 2 [BlueZ,1/3] device: Refactor device_discover_services function src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... tools/mgmt-tester.c: In function ‘main’: tools/mgmt-tester.c:12990:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without 12990 | int main(int argc, char *argv[]) | ^~~~ make: *** [Makefile:4172: all] Error 2 ############################## Test: ScanBuild - FAIL Desc: Run Scan Build Output: src/shared/gatt-client.c:447:21: warning: Use of memory after it is freed gatt_db_unregister(op->client->db, op->db_id); ^~~~~~~~~~ src/shared/gatt-client.c:692:2: warning: Use of memory after it is freed discovery_op_complete(op, false, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:992:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1098:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1292:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1357:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1632:6: warning: Use of memory after it is freed if (read_db_hash(op)) { ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1637:2: warning: Use of memory after it is freed discover_all(op); ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1693:56: warning: Use of memory after it is freed notify_data->chrc->ccc_write_id = notify_data->att_id = att_id; ~~~~~~~~~~~~~~~~~~~ ^ src/shared/gatt-client.c:2146:6: warning: Use of memory after it is freed if (read_db_hash(op)) { ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:2154:8: warning: Use of memory after it is freed discovery_op_ref(op), ^~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3332:2: warning: Use of memory after it is freed complete_write_long_op(req, success, 0, false); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3354:2: warning: Use of memory after it is freed request_unref(req); ^~~~~~~~~~~~~~~~~~ 13 warnings generated. src/shared/bap.c:1529:8: warning: Use of memory after it is freed bap = bt_bap_ref_safe(bap); ^~~~~~~~~~~~~~~~~~~~ src/shared/bap.c:2340:20: warning: Use of memory after it is freed return queue_find(stream->bap->streams, NULL, stream); ^~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/shared/gatt-client.c:447:21: warning: Use of memory after it is freed gatt_db_unregister(op->client->db, op->db_id); ^~~~~~~~~~ src/shared/gatt-client.c:692:2: warning: Use of memory after it is freed discovery_op_complete(op, false, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:992:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1098:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1292:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1357:2: warning: Use of memory after it is freed discovery_op_complete(op, success, att_ecode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1632:6: warning: Use of memory after it is freed if (read_db_hash(op)) { ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1637:2: warning: Use of memory after it is freed discover_all(op); ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1693:56: warning: Use of memory after it is freed notify_data->chrc->ccc_write_id = notify_data->att_id = att_id; ~~~~~~~~~~~~~~~~~~~ ^ src/shared/gatt-client.c:2146:6: warning: Use of memory after it is freed if (read_db_hash(op)) { ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:2154:8: warning: Use of memory after it is freed discovery_op_ref(op), ^~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3332:2: warning: Use of memory after it is freed complete_write_long_op(req, success, 0, false); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3354:2: warning: Use of memory after it is freed request_unref(req); ^~~~~~~~~~~~~~~~~~ 13 warnings generated. tools/hciattach.c:817:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 10)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:865:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 4)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:887:8: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 10)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:909:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 4)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:930:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 4)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/hciattach.c:974:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' if ((n = read_hci_event(fd, resp, 6)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ 6 warnings generated. src/shared/bap.c:1529:8: warning: Use of memory after it is freed bap = bt_bap_ref_safe(bap); ^~~~~~~~~~~~~~~~~~~~ src/shared/bap.c:2340:20: warning: Use of memory after it is freed return queue_find(stream->bap->streams, NULL, stream); ^~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/oui.c:50:2: warning: Value stored to 'hwdb' is never read hwdb = udev_hwdb_unref(hwdb); ^ ~~~~~~~~~~~~~~~~~~~~~ src/oui.c:53:2: warning: Value stored to 'udev' is never read udev = udev_unref(udev); ^ ~~~~~~~~~~~~~~~~ 2 warnings generated. tools/rfcomm.c:234:3: warning: Value stored to 'i' is never read i = execvp(cmdargv[0], cmdargv); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:234:7: warning: Null pointer passed to 1st parameter expecting 'nonnull' i = execvp(cmdargv[0], cmdargv); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:354:8: warning: Although the value stored to 'fd' is used in the enclosing expression, the value is never actually read from 'fd' if ((fd = open(devname, O_RDONLY | O_NOCTTY)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:497:14: warning: Assigned value is garbage or undefined req.channel = raddr.rc_channel; ^ ~~~~~~~~~~~~~~~~ tools/rfcomm.c:515:8: warning: Although the value stored to 'fd' is used in the enclosing expression, the value is never actually read from 'fd' if ((fd = open(devname, O_RDONLY | O_NOCTTY)) < 0) { ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 5 warnings generated. tools/ciptool.c:351:7: warning: 5th function call argument is an uninitialized value sk = do_connect(ctl, dev_id, &src, &dst, psm, (1 << CMTP_LOOPBACK)); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. src/sdp-xml.c:126:10: warning: Assigned value is garbage or undefined buf[1] = data[i + 1]; ^ ~~~~~~~~~~~ src/sdp-xml.c:306:11: warning: Assigned value is garbage or undefined buf[1] = data[i + 1]; ^ ~~~~~~~~~~~ src/sdp-xml.c:344:11: warning: Assigned value is garbage or undefined buf[1] = data[i + 1]; ^ ~~~~~~~~~~~ 3 warnings generated. tools/sdptool.c:941:26: warning: Result of 'malloc' is converted to a pointer of type 'uint32_t', which is incompatible with sizeof operand type 'int' uint32_t *value_int = malloc(sizeof(int)); ~~~~~~~~~~ ^~~~~~ ~~~~~~~~~~~ tools/sdptool.c:980:4: warning: 1st function call argument is an uninitialized value free(allocArray[i]); ^~~~~~~~~~~~~~~~~~~ tools/sdptool.c:3777:2: warning: Potential leak of memory pointed to by 'si.name' return add_service(0, &si); ^~~~~~~~~~~~~~~~~~~~~~~~~~ tools/sdptool.c:4112:4: warning: Potential leak of memory pointed to by 'context.svc' return -1; ^~~~~~~~~ 4 warnings generated. tools/avtest.c:243:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:253:5: warning: Value stored to 'len' is never read len = write(sk, buf, 4); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:262:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:276:5: warning: Value stored to 'len' is never read len = write(sk, buf, ^ ~~~~~~~~~~~~~~ tools/avtest.c:283:5: warning: Value stored to 'len' is never read len = write(sk, buf, ^ ~~~~~~~~~~~~~~ tools/avtest.c:290:5: warning: Value stored to 'len' is never read len = write(sk, buf, ^ ~~~~~~~~~~~~~~ tools/avtest.c:297:5: warning: Value stored to 'len' is never read len = write(sk, buf, ^ ~~~~~~~~~~~~~~ tools/avtest.c:309:5: warning: Value stored to 'len' is never read len = write(sk, buf, 4); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:313:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:322:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:326:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:335:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:342:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:364:5: warning: Value stored to 'len' is never read len = write(sk, buf, 4); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:368:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:377:5: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:381:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:394:5: warning: Value stored to 'len' is never read len = write(sk, buf, 4); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:398:5: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:405:4: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:415:4: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:580:3: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:588:3: warning: Value stored to 'len' is never read len = write(sk, buf, invalid ? 2 : 3); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:602:3: warning: Value stored to 'len' is never read len = write(sk, buf, 4 + media_transport_size); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:615:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:625:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:637:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:652:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:664:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:673:3: warning: Value stored to 'len' is never read len = write(sk, buf, 3); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:680:3: warning: Value stored to 'len' is never read len = write(sk, buf, 2); ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:716:2: warning: Value stored to 'len' is never read len = write(sk, buf, AVCTP_HEADER_LENGTH + sizeof(play_pressed)); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 32 warnings generated. tools/btproxy.c:836:15: warning: Null pointer passed to 1st parameter expecting 'nonnull' tcp_port = atoi(optarg); ^~~~~~~~~~~~ tools/btproxy.c:839:8: warning: Null pointer passed to 1st parameter expecting 'nonnull' if (strlen(optarg) > 3 && !strncmp(optarg, "hci", 3)) ^~~~~~~~~~~~~~ 2 warnings generated. tools/create-image.c:76:3: warning: Value stored to 'fd' is never read fd = -1; ^ ~~ tools/create-image.c:84:3: warning: Value stored to 'fd' is never read fd = -1; ^ ~~ tools/create-image.c:92:3: warning: Value stored to 'fd' is never read fd = -1; ^ ~~ tools/create-image.c:105:2: warning: Value stored to 'fd' is never read fd = -1; ^ ~~ 4 warnings generated. tools/btgatt-client.c:1822:2: warning: Value stored to 'argv' is never read argv += optind; ^ ~~~~~~ 1 warning generated. tools/btgatt-server.c:1204:2: warning: Value stored to 'argv' is never read argv -= optind; ^ ~~~~~~ 1 warning generated. tools/check-selftest.c:42:3: warning: Value stored to 'ptr' is never read ptr = fgets(result, sizeof(result), fp); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. tools/gatt-service.c:294:2: warning: 2nd function call argument is an uninitialized value chr_write(chr, value, len); ^~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. tools/obex-server-tool.c:133:13: warning: Null pointer passed to 1st parameter expecting 'nonnull' data->fd = open(name, O_WRONLY | O_CREAT | O_NOCTTY, 0600); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/obex-server-tool.c:192:13: warning: Null pointer passed to 1st parameter expecting 'nonnull' data->fd = open(name, O_RDONLY | O_NOCTTY, 0); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 warnings generated. tools/test-runner.c:1370:2: warning: Address of stack memory associated with local variable 'kernel_path' is still referred to by the global variable 'kernel_image' upon returning to the caller. This will be a dangling reference return EXIT_SUCCESS; ^~~~~~~~~~~~~~~~~~~ 1 warning generated. client/btpclient/btpclientctl.c:402:3: warning: Value stored to 'bit' is never read bit = 0; ^ ~ client/btpclient/btpclientctl.c:1655:2: warning: Null pointer passed to 2nd parameter expecting 'nonnull' memcpy(cp->data, ad_data, ad_len); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/sdp-client.c:353:14: warning: Access to field 'cb' results in a dereference of a null pointer (*ctxt)->cb = cb; ~~~~~~~~~~~~^~~~ 1 warning generated. src/sdpd-request.c:209:13: warning: Result of 'malloc' is converted to a pointer of type 'char', which is incompatible with sizeof operand type 'uint16_t' pElem = malloc(sizeof(uint16_t)); ^~~~~~ ~~~~~~~~~~~~~~~~ src/sdpd-request.c:237:13: warning: Result of 'malloc' is converted to a pointer of type 'char', which is incompatible with sizeof operand type 'uint32_t' pElem = malloc(sizeof(uint32_t)); ^~~~~~ ~~~~~~~~~~~~~~~~ 2 warnings generated. src/gatt-database.c:1171:10: warning: Value stored to 'bits' during its initialization is never read uint8_t bits[] = { BT_GATT_CHRC_CLI_FEAT_ROBUST_CACHING, ^~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. src/device.c: In function ‘btd_device_connect_services’: src/device.c:2738:9: error: too few arguments to function ‘device_discover_services’ 2738 | err = device_discover_services(dev); | ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from src/device.c:54: src/device.h:210:5: note: declared here 210 | int device_discover_services(struct btd_device *device, | ^~~~~~~~~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7058: src/device.o] Error 1 make[1]: *** Waiting for unfinished jobs.... src/gatt-client.c:1569:2: warning: Use of memory after it is freed notify_client_unref(client); ^~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. make: *** [Makefile:4172: all] Error 2 https://github.com/bluez/bluez/pull/2185 --- Regards, Linux Bluetooth ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2026-06-08 18:20 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-06-08 10:39 [PATCH BlueZ 1/3] device: Refactor device_discover_services function Simon Mikuda 2026-06-08 10:39 ` [PATCH BlueZ 2/3] device: Rename start_discovery function Simon Mikuda 2026-06-08 10:39 ` [PATCH BlueZ 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 1/3] device: Refactor device_discover_services function Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 2/3] device: Rename start_discovery function Simon Mikuda 2026-06-08 10:59 ` [PATCH BlueZ v2 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda 2026-06-08 11:41 ` [BlueZ,v2,1/3] device: Refactor device_discover_services function bluez.test.bot 2026-06-08 11:07 ` [PATCH BlueZ v3 1/3] " Simon Mikuda 2026-06-08 11:07 ` [PATCH BlueZ v3 2/3] device: Rename start_discovery function Simon Mikuda 2026-06-08 11:07 ` [PATCH BlueZ v3 3/3] device: Fix returning discovery error for Device.Pair Simon Mikuda 2026-06-08 12:25 ` [BlueZ,v3,1/3] device: Refactor device_discover_services function bluez.test.bot 2026-06-08 18:20 ` [PATCH BlueZ v3 1/3] " patchwork-bot+bluetooth 2026-06-08 11:40 ` [BlueZ,1/3] " bluez.test.bot
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox