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/series/1175561/ ---Test result--- Test Summary: CheckPatch FAIL 15.02 seconds GitLint FAIL 8.34 seconds BuildEll PASS 19.16 seconds BluezMake PASS 368.71 seconds MakeCheck PASS 14.32 seconds MakeDistcheck PASS 137.33 seconds CheckValgrind PASS 228.40 seconds CheckSmatch PASS 288.22 seconds bluezmakeextell PASS 92.65 seconds TestFunctional PASS 1112.78 seconds IncrementalBuild FAIL 748.63 seconds ScanBuild FAIL 150.39 seconds Details ############################## Test: CheckPatch - FAIL Desc: Run checkpatch.pl script Output: [BlueZ,v6,09/23] client/agent: Fix crash on Cancel with no pending request WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?) #108: #6 _dbus_warn_check_failed () from /usr/lib/x86_64-linux-gnu/libdbus-1.so.3 /home/runner/work/bluez/bluez/src/patch/14853402.patch total: 0 errors, 1 warnings, 16 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. /home/runner/work/bluez/bluez/src/patch/14853402.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. [BlueZ,v6,23/23] attrib: Remove directory WARNING:LINE_SPACING: Missing a blank line after declarations #1911: FILE: src/device.c:6488: + sdp_list_t *p; + for (p = list->data; p; p = p->next) { WARNING:LINE_SPACING: Missing a blank line after declarations #1913: FILE: src/device.c:6490: + sdp_data_t *seq = p->data; + if (seq && seq->dtd == SDP_UUID16 && /home/runner/work/bluez/bluez/src/patch/14853415.patch total: 0 errors, 2 warnings, 278 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. /home/runner/work/bluez/bluez/src/patch/14853415.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: GitLint - FAIL Desc: Run gitlint Output: [BlueZ,v6,01/23] shared/gatt-client: Fix calling destroy after unregistering notify 1: T1 Title exceeds max length (83>80): "[BlueZ,v6,01/23] shared/gatt-client: Fix calling destroy after unregistering notify" [BlueZ,v6,13/23] shared/gatt-client: Fix calling idle callbacks again while notifying 1: T1 Title exceeds max length (85>80): "[BlueZ,v6,13/23] shared/gatt-client: Fix calling idle callbacks again while notifying" [BlueZ,v6,17/23] test: functional: change the HoG SCI mode with the HID Control Point 1: T1 Title exceeds max length (85>80): "[BlueZ,v6,17/23] test: functional: change the HoG SCI mode with the HID Control Point" ############################## Test: IncrementalBuild - FAIL Desc: Incremental build with the patches in the series Output: In file included from src/adapter.c:63: ./attrib/gatt.h:61:29: error: unknown type name ‘GAttrib’ 61 | guint gatt_discover_primary(GAttrib *attrib, bt_uuid_t *uuid, gatt_cb_t func, | ^~~~~~~ ./attrib/gatt.h:64:33: error: unknown type name ‘GAttrib’ 64 | unsigned int gatt_find_included(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:67:26: error: unknown type name ‘GAttrib’ 67 | guint gatt_discover_char(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:71:22: error: unknown type name ‘GAttrib’ 71 | guint gatt_read_char(GAttrib *attrib, uint16_t handle, GAttribResultFunc func, | ^~~~~~~ ./attrib/gatt.h:71:56: error: unknown type name ‘GAttribResultFunc’ 71 | guint gatt_read_char(GAttrib *attrib, uint16_t handle, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:74:23: error: unknown type name ‘GAttrib’ 74 | guint gatt_write_char(GAttrib *attrib, uint16_t handle, const uint8_t *value, | ^~~~~~~ ./attrib/gatt.h:75:54: error: unknown type name ‘GAttribResultFunc’ 75 | size_t vlen, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:78:26: error: unknown type name ‘GAttrib’ 78 | guint gatt_discover_desc(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:82:32: error: unknown type name ‘GAttrib’ 82 | guint gatt_reliable_write_char(GAttrib *attrib, uint16_t handle, | ^~~~~~~ ./attrib/gatt.h:84:41: error: unknown type name ‘GAttribResultFunc’ 84 | GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:87:26: error: unknown type name ‘GAttrib’ 87 | guint gatt_execute_write(GAttrib *attrib, uint8_t flags, | ^~~~~~~ ./attrib/gatt.h:88:33: error: unknown type name ‘GAttribResultFunc’ 88 | GAttribResultFunc func, gpointer user_data); | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:90:22: error: unknown type name ‘GAttrib’ 90 | guint gatt_write_cmd(GAttrib *attrib, uint16_t handle, const uint8_t *value, | ^~~~~~~ ./attrib/gatt.h:93:29: error: unknown type name ‘GAttrib’ 93 | guint gatt_signed_write_cmd(GAttrib *attrib, uint16_t handle, | ^~~~~~~ ./attrib/gatt.h:100:30: error: unknown type name ‘GAttrib’ 100 | guint gatt_read_char_by_uuid(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:101:50: error: unknown type name ‘GAttribResultFunc’ 101 | bt_uuid_t *uuid, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:104:25: error: unknown type name ‘GAttrib’ 104 | guint gatt_exchange_mtu(GAttrib *attrib, uint16_t mtu, GAttribResultFunc func, | ^~~~~~~ ./attrib/gatt.h:104:56: error: unknown type name ‘GAttribResultFunc’ 104 | guint gatt_exchange_mtu(GAttrib *attrib, uint16_t mtu, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7232: src/adapter.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:4303: all] Error 2 [BlueZ,v6,21/23] device: Use bt_att instead of GAttrib In file included from src/adapter.c:63: ./attrib/gatt.h:61:29: error: unknown type name ‘GAttrib’ 61 | guint gatt_discover_primary(GAttrib *attrib, bt_uuid_t *uuid, gatt_cb_t func, | ^~~~~~~ ./attrib/gatt.h:64:33: error: unknown type name ‘GAttrib’ 64 | unsigned int gatt_find_included(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:67:26: error: unknown type name ‘GAttrib’ 67 | guint gatt_discover_char(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:71:22: error: unknown type name ‘GAttrib’ 71 | guint gatt_read_char(GAttrib *attrib, uint16_t handle, GAttribResultFunc func, | ^~~~~~~ ./attrib/gatt.h:71:56: error: unknown type name ‘GAttribResultFunc’ 71 | guint gatt_read_char(GAttrib *attrib, uint16_t handle, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:74:23: error: unknown type name ‘GAttrib’ 74 | guint gatt_write_char(GAttrib *attrib, uint16_t handle, const uint8_t *value, | ^~~~~~~ ./attrib/gatt.h:75:54: error: unknown type name ‘GAttribResultFunc’ 75 | size_t vlen, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:78:26: error: unknown type name ‘GAttrib’ 78 | guint gatt_discover_desc(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:82:32: error: unknown type name ‘GAttrib’ 82 | guint gatt_reliable_write_char(GAttrib *attrib, uint16_t handle, | ^~~~~~~ ./attrib/gatt.h:84:41: error: unknown type name ‘GAttribResultFunc’ 84 | GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:87:26: error: unknown type name ‘GAttrib’ 87 | guint gatt_execute_write(GAttrib *attrib, uint8_t flags, | ^~~~~~~ ./attrib/gatt.h:88:33: error: unknown type name ‘GAttribResultFunc’ 88 | GAttribResultFunc func, gpointer user_data); | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:90:22: error: unknown type name ‘GAttrib’ 90 | guint gatt_write_cmd(GAttrib *attrib, uint16_t handle, const uint8_t *value, | ^~~~~~~ ./attrib/gatt.h:93:29: error: unknown type name ‘GAttrib’ 93 | guint gatt_signed_write_cmd(GAttrib *attrib, uint16_t handle, | ^~~~~~~ ./attrib/gatt.h:100:30: error: unknown type name ‘GAttrib’ 100 | guint gatt_read_char_by_uuid(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:101:50: error: unknown type name ‘GAttribResultFunc’ 101 | bt_uuid_t *uuid, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:104:25: error: unknown type name ‘GAttrib’ 104 | guint gatt_exchange_mtu(GAttrib *attrib, uint16_t mtu, GAttribResultFunc func, | ^~~~~~~ ./attrib/gatt.h:104:56: error: unknown type name ‘GAttribResultFunc’ 104 | guint gatt_exchange_mtu(GAttrib *attrib, uint16_t mtu, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7232: src/adapter.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:4303: all] Error 2 ############################## Test: ScanBuild - FAIL Desc: Run Scan Build Output: src/shared/gatt-client.c:457:21: warning: Use of memory after it is freed [unix.Malloc] 457 | gatt_db_unregister(op->client->db, op->db_id); | ^~~~~~~~~~ src/shared/gatt-client.c:702:2: warning: Use of memory after it is freed [unix.Malloc] 702 | discovery_op_complete(op, false, att_ecode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1108:2: warning: Use of memory after it is freed [unix.Malloc] 1108 | discovery_op_complete(op, success, att_ecode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1302:2: warning: Use of memory after it is freed [unix.Malloc] 1302 | discovery_op_complete(op, success, att_ecode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1367:2: warning: Use of memory after it is freed [unix.Malloc] 1367 | discovery_op_complete(op, success, att_ecode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1642:6: warning: Use of memory after it is freed [unix.Malloc] 1642 | if (read_db_hash(op)) { | ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1647:2: warning: Use of memory after it is freed [unix.Malloc] 1647 | discover_all(op); | ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1703:56: warning: Use of memory after it is freed [unix.Malloc] 1703 | notify_data->chrc->ccc_write_id = notify_data->att_id = att_id; | ~~~~~~~~~~~~~~~~~~~ ^ src/shared/gatt-client.c:2156:6: warning: Use of memory after it is freed [unix.Malloc] 2156 | if (read_db_hash(op)) { | ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:2164:8: warning: Use of memory after it is freed [unix.Malloc] 2164 | discovery_op_ref(op), | ^~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3350:2: warning: Use of memory after it is freed [unix.Malloc] 3350 | complete_write_long_op(req, success, 0, false); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3372:2: warning: Use of memory after it is freed [unix.Malloc] 3372 | request_unref(req); | ^~~~~~~~~~~~~~~~~~ 12 warnings generated. src/shared/rap.c:3322:30: warning: Assigned value is uninitialized [core.uninitialized.Assign] 3322 | out->tone_pct[k].i_sample = tone_pct_i[k]; | ^ ~~~~~~~~~~~~~ src/shared/rap.c:4269:13: warning: Use of memory after it is freed [unix.Malloc] 4269 | attached = queue_find(sessions, NULL, rap); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/shared/bap.c:1543:8: warning: Use of memory after it is freed [unix.Malloc] 1543 | bap = bt_bap_ref_safe(bap); | ^~~~~~~~~~~~~~~~~~~~ src/shared/bap.c:2359:20: warning: Use of memory after it is freed [unix.Malloc] 2359 | return queue_find(stream->bap->streams, NULL, stream); | ^~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/shared/gatt-client.c:457:21: warning: Use of memory after it is freed [unix.Malloc] 457 | gatt_db_unregister(op->client->db, op->db_id); | ^~~~~~~~~~ src/shared/gatt-client.c:702:2: warning: Use of memory after it is freed [unix.Malloc] 702 | discovery_op_complete(op, false, att_ecode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1108:2: warning: Use of memory after it is freed [unix.Malloc] 1108 | discovery_op_complete(op, success, att_ecode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1302:2: warning: Use of memory after it is freed [unix.Malloc] 1302 | discovery_op_complete(op, success, att_ecode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1367:2: warning: Use of memory after it is freed [unix.Malloc] 1367 | discovery_op_complete(op, success, att_ecode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1642:6: warning: Use of memory after it is freed [unix.Malloc] 1642 | if (read_db_hash(op)) { | ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1647:2: warning: Use of memory after it is freed [unix.Malloc] 1647 | discover_all(op); | ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:1703:56: warning: Use of memory after it is freed [unix.Malloc] 1703 | notify_data->chrc->ccc_write_id = notify_data->att_id = att_id; | ~~~~~~~~~~~~~~~~~~~ ^ src/shared/gatt-client.c:2156:6: warning: Use of memory after it is freed [unix.Malloc] 2156 | if (read_db_hash(op)) { | ^~~~~~~~~~~~~~~~ src/shared/gatt-client.c:2164:8: warning: Use of memory after it is freed [unix.Malloc] 2164 | discovery_op_ref(op), | ^~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3350:2: warning: Use of memory after it is freed [unix.Malloc] 3350 | complete_write_long_op(req, success, 0, false); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/shared/gatt-client.c:3372:2: warning: Use of memory after it is freed [unix.Malloc] 3372 | request_unref(req); | ^~~~~~~~~~~~~~~~~~ 12 warnings generated. src/shared/rap.c:3322:30: warning: Assigned value is uninitialized [core.uninitialized.Assign] 3322 | out->tone_pct[k].i_sample = tone_pct_i[k]; | ^ ~~~~~~~~~~~~~ src/shared/rap.c:4269:13: warning: Use of memory after it is freed [unix.Malloc] 4269 | attached = queue_find(sessions, NULL, rap); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 warnings generated. tools/hciattach_ath3k.c:530:9: warning: File position of the stream might be 'indeterminate' after a failed operation. Can cause undefined behavior [unix.Stream] 530 | if (!fgets(byte, 3, stream)) | ^~~~~~~~~~~~~~~~~~~~~~ tools/hciattach_ath3k.c:530:9: warning: Read function called when stream is in EOF state. Function has no effect [unix.Stream] 530 | if (!fgets(byte, 3, stream)) | ^~~~~~~~~~~~~~~~~~~~~~ 2 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' [deadcode.DeadStores] 817 | 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' [deadcode.DeadStores] 865 | 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' [deadcode.DeadStores] 887 | 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' [deadcode.DeadStores] 909 | 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' [deadcode.DeadStores] 930 | 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' [deadcode.DeadStores] 974 | if ((n = read_hci_event(fd, resp, 6)) < 0) { | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ 6 warnings generated. src/shared/bap.c:1543:8: warning: Use of memory after it is freed [unix.Malloc] 1543 | bap = bt_bap_ref_safe(bap); | ^~~~~~~~~~~~~~~~~~~~ src/shared/bap.c:2359:20: warning: Use of memory after it is freed [unix.Malloc] 2359 | return queue_find(stream->bap->streams, NULL, stream); | ^~~~~~~~~~~~~~~~~~~~ 2 warnings generated. src/oui.c:50:2: warning: Value stored to 'hwdb' is never read [deadcode.DeadStores] 50 | hwdb = udev_hwdb_unref(hwdb); | ^ ~~~~~~~~~~~~~~~~~~~~~ src/oui.c:53:2: warning: Value stored to 'udev' is never read [deadcode.DeadStores] 53 | udev = udev_unref(udev); | ^ ~~~~~~~~~~~~~~~~ 2 warnings generated. tools/rfcomm.c:234:3: warning: Value stored to 'i' is never read [deadcode.DeadStores] 234 | i = execvp(cmdargv[0], cmdargv); | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:234:7: warning: Null pointer passed to 1st parameter expecting 'nonnull' [core.NonNullParamChecker] 234 | 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' [deadcode.DeadStores] 354 | if ((fd = open(devname, O_RDONLY | O_NOCTTY)) < 0) { | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:473:6: warning: The 1st argument to 'getsockname' is -1 but should be >= 0 [unix.StdCLibraryFunctions] 473 | if (getsockname(nsk, (struct sockaddr *)&laddr, &alen) < 0) { | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm.c:497:14: warning: Assigned value is uninitialized [core.uninitialized.Assign] 497 | 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' [deadcode.DeadStores] 515 | if ((fd = open(devname, O_RDONLY | O_NOCTTY)) < 0) { | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 6 warnings generated. tools/ciptool.c:351:7: warning: 5th function call argument is an uninitialized value [core.CallAndMessage] 351 | sk = do_connect(ctl, dev_id, &src, &dst, psm, (1 << CMTP_LOOPBACK)); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning 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' [unix.MallocSizeof] 941 | uint32_t *value_int = malloc(sizeof(int)); | ~~~~~~~~~~ ^~~~~~ ~~~~~~~~~~~ tools/sdptool.c:980:4: warning: 1st function call argument is an uninitialized value [core.CallAndMessage] 980 | free(allocArray[i]); | ^~~~~~~~~~~~~~~~~~~ tools/sdptool.c:3777:2: warning: Potential leak of memory pointed to by 'si.name' [unix.Malloc] 3777 | return add_service(0, &si); | ^~~~~~~~~~~~~~~~~~~~~~~~~~ tools/sdptool.c:4112:4: warning: Potential leak of memory pointed to by 'context.svc' [unix.Malloc] 4112 | return -1; | ^~~~~~~~~ 4 warnings generated. src/sdp-xml.c:129:10: warning: Assigned value is uninitialized [core.uninitialized.Assign] 129 | buf[1] = data[i + 1]; | ^ ~~~~~~~~~~~ src/sdp-xml.c:309:11: warning: Assigned value is uninitialized [core.uninitialized.Assign] 309 | buf[1] = data[i + 1]; | ^ ~~~~~~~~~~~ src/sdp-xml.c:347:11: warning: Assigned value is uninitialized [core.uninitialized.Assign] 347 | buf[1] = data[i + 1]; | ^ ~~~~~~~~~~~ 3 warnings generated. tools/rfcomm-tester.c:410:6: warning: The 1st argument to 'bind' is -1 but should be >= 0 [unix.StdCLibraryFunctions] 410 | if (bind(sk, (struct sockaddr *) &addr, sizeof(addr)) < 0) { | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/rfcomm-tester.c:428:8: warning: The 1st argument to 'connect' is -1 but should be >= 0 [unix.StdCLibraryFunctions] 428 | err = connect(sk, (struct sockaddr *) &addr, sizeof(addr)); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 warnings generated. tools/avtest.c:243:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 243 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:253:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 253 | len = write(sk, buf, 4); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:262:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 262 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:276:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 276 | len = write(sk, buf, | ^ ~~~~~~~~~~~~~~ 277 | 3 + sizeof(media_transport)); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:283:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 283 | len = write(sk, buf, | ^ ~~~~~~~~~~~~~~ 284 | 1 + media_transport_size); | ~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:290:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 290 | len = write(sk, buf, | ^ ~~~~~~~~~~~~~~ 291 | 1 + media_transport_size); | ~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:297:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 297 | len = write(sk, buf, | ^ ~~~~~~~~~~~~~~ 298 | 2 + media_transport_size); | ~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:309:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 309 | len = write(sk, buf, 4); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:313:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 313 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:322:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 322 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:326:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 326 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:335:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 335 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:342:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 342 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:364:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 364 | len = write(sk, buf, 4); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:368:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 368 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:377:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 377 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:381:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 381 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:394:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 394 | len = write(sk, buf, 4); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:398:5: warning: Value stored to 'len' is never read [deadcode.DeadStores] 398 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:405:4: warning: Value stored to 'len' is never read [deadcode.DeadStores] 405 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:415:4: warning: Value stored to 'len' is never read [deadcode.DeadStores] 415 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:580:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 580 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:588:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 588 | len = write(sk, buf, invalid ? 2 : 3); | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:602:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 602 | len = write(sk, buf, 4 + media_transport_size); | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/avtest.c:615:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 615 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:625:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 625 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:637:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 637 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:652:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 652 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:664:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 664 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:673:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 673 | len = write(sk, buf, 3); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:680:3: warning: Value stored to 'len' is never read [deadcode.DeadStores] 680 | len = write(sk, buf, 2); | ^ ~~~~~~~~~~~~~~~~~ tools/avtest.c:716:2: warning: Value stored to 'len' is never read [deadcode.DeadStores] 716 | len = write(sk, buf, AVCTP_HEADER_LENGTH + sizeof(play_pressed)); | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 32 warnings generated. tools/create-image.c:76:3: warning: Value stored to 'fd' is never read [deadcode.DeadStores] 76 | fd = -1; | ^ ~~ tools/create-image.c:84:3: warning: Value stored to 'fd' is never read [deadcode.DeadStores] 84 | fd = -1; | ^ ~~ tools/create-image.c:92:3: warning: Value stored to 'fd' is never read [deadcode.DeadStores] 92 | fd = -1; | ^ ~~ tools/create-image.c:105:2: warning: Value stored to 'fd' is never read [deadcode.DeadStores] 105 | fd = -1; | ^ ~~ tools/create-image.c:108:2: warning: Null pointer passed to 1st parameter expecting 'nonnull' [core.NonNullParamChecker] 108 | fprintf(fp, HDR_FMT, HDR_MAGIC, ino, mode, 0, 0, 1, 0, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 109 | (uintmax_t) st.st_size, 0, 0, 0, 0, namelen + 1, 0, name); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ tools/create-image.c:188:2: warning: Null pointer passed to 1st parameter expecting 'nonnull' [core.NonNullParamChecker] 188 | fclose(fp); | ^~~~~~~~~~ 6 warnings generated. tools/btgatt-client.c:1822:2: warning: Value stored to 'argv' is never read [deadcode.DeadStores] 1822 | argv += optind; | ^ ~~~~~~ 1 warning generated. tools/btgatt-server.c:1204:2: warning: Value stored to 'argv' is never read [deadcode.DeadStores] 1204 | argv -= optind; | ^ ~~~~~~ 1 warning generated. tools/check-selftest.c:42:3: warning: Value stored to 'ptr' is never read [deadcode.DeadStores] 42 | ptr = fgets(result, sizeof(result), fp); | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. tools/gatt-service.c:294:2: warning: 2nd function call argument is an uninitialized value [core.CallAndMessage] 294 | chr_write(chr, value, len); | ^~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. In file included from tools/obex-server-tool.c:17: In file included from /usr/include/fcntl.h:342: /usr/include/x86_64-linux-gnu/bits/fcntl2.h:76:10: warning: Null pointer passed to 1st parameter expecting 'nonnull' [core.NonNullParamChecker] 76 | return __open_alias (__path, __oflag, __mode); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. client/btpclient/btpclientctl.c:402:3: warning: Value stored to 'bit' is never read [deadcode.DeadStores] 402 | bit = 0; | ^ ~ client/btpclient/btpclientctl.c:1655:2: warning: Null pointer passed to 2nd parameter expecting 'nonnull' [core.NonNullParamChecker] 1655 | 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 [core.NullDereference] 353 | (*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' [unix.MallocSizeof] 209 | 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' [unix.MallocSizeof] 237 | pElem = malloc(sizeof(uint32_t)); | ^~~~~~ ~~~~~~~~~~~~~~~~ 2 warnings generated. In file included from src/adapter.c:63: ./attrib/gatt.h:61:29: error: unknown type name ‘GAttrib’ 61 | guint gatt_discover_primary(GAttrib *attrib, bt_uuid_t *uuid, gatt_cb_t func, | ^~~~~~~ ./attrib/gatt.h:64:33: error: unknown type name ‘GAttrib’ 64 | unsigned int gatt_find_included(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:67:26: error: unknown type name ‘GAttrib’ 67 | guint gatt_discover_char(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:71:22: error: unknown type name ‘GAttrib’ 71 | guint gatt_read_char(GAttrib *attrib, uint16_t handle, GAttribResultFunc func, | ^~~~~~~ ./attrib/gatt.h:71:56: error: unknown type name ‘GAttribResultFunc’ 71 | guint gatt_read_char(GAttrib *attrib, uint16_t handle, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:74:23: error: unknown type name ‘GAttrib’ 74 | guint gatt_write_char(GAttrib *attrib, uint16_t handle, const uint8_t *value, | ^~~~~~~ ./attrib/gatt.h:75:54: error: unknown type name ‘GAttribResultFunc’ 75 | size_t vlen, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:78:26: error: unknown type name ‘GAttrib’ 78 | guint gatt_discover_desc(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:82:32: error: unknown type name ‘GAttrib’ 82 | guint gatt_reliable_write_char(GAttrib *attrib, uint16_t handle, | ^~~~~~~ ./attrib/gatt.h:84:41: error: unknown type name ‘GAttribResultFunc’ 84 | GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:87:26: error: unknown type name ‘GAttrib’ 87 | guint gatt_execute_write(GAttrib *attrib, uint8_t flags, | ^~~~~~~ ./attrib/gatt.h:88:33: error: unknown type name ‘GAttribResultFunc’ 88 | GAttribResultFunc func, gpointer user_data); | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:90:22: error: unknown type name ‘GAttrib’ 90 | guint gatt_write_cmd(GAttrib *attrib, uint16_t handle, const uint8_t *value, | ^~~~~~~ ./attrib/gatt.h:93:29: error: unknown type name ‘GAttrib’ 93 | guint gatt_signed_write_cmd(GAttrib *attrib, uint16_t handle, | ^~~~~~~ ./attrib/gatt.h:100:30: error: unknown type name ‘GAttrib’ 100 | guint gatt_read_char_by_uuid(GAttrib *attrib, uint16_t start, uint16_t end, | ^~~~~~~ ./attrib/gatt.h:101:50: error: unknown type name ‘GAttribResultFunc’ 101 | bt_uuid_t *uuid, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ ./attrib/gatt.h:104:25: error: unknown type name ‘GAttrib’ 104 | guint gatt_exchange_mtu(GAttrib *attrib, uint16_t mtu, GAttribResultFunc func, | ^~~~~~~ ./attrib/gatt.h:104:56: error: unknown type name ‘GAttribResultFunc’ 104 | guint gatt_exchange_mtu(GAttrib *attrib, uint16_t mtu, GAttribResultFunc func, | ^~~~~~~~~~~~~~~~~ make[1]: *** [Makefile:7232: src/adapter.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:4303: all] Error 2 https://github.com/bluez/bluez/pull/2600 --- Regards, Linux Bluetooth