From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, luiz.dentz@gmail.com
Subject: RE: Add HoG functional tests and shared/hog
Date: Mon, 28 Sep 2026 15:26:03 -0700 (PDT) [thread overview]
Message-ID: <6abae97b.eeba9552.340f2b.8b59@mx.google.com> (raw)
In-Reply-To: <20260928200031.1209311-2-luiz.dentz@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 39575 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/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
next prev parent reply other threads:[~2026-09-28 22:26 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-28 20:00 [PATCH BlueZ v6 00/23] Add HoG functional tests and shared/hog Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 01/23] shared/gatt-client: Fix calling destroy after unregistering notify Luiz Augusto von Dentz
2026-09-28 22:26 ` bluez.test.bot [this message]
2026-09-28 20:00 ` [PATCH BlueZ v6 02/23] client/gatt: Fix setting descriptor value from scripts Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 03/23] client/mgmt: Print Connection Subrate event Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 04/23] emulator: Default to the latest BR/EDR+LE version Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 05/23] client/scripts: Add HoG device scripts Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 06/23] doc: Add functional-hog documentation Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 07/23] test: functional: add HoG tests Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 08/23] test: functional: limit the workers by the memory available Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 09/23] client/agent: Fix crash on Cancel with no pending request Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 10/23] shared/uhid: Fix size of Get Report reply with a Report ID Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 11/23] shared/uhid: Keep reading when an event is not available Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 12/23] shared/tester: Allow expecting a PDU with no response Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 13/23] shared/gatt-client: Fix calling idle callbacks again while notifying Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 14/23] shared/gatt-client: Add bt_gatt_client_is_idle Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 15/23] shared/hog: Add initial implementation Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 16/23] unit/test-hog: Use shared/hog Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 17/23] test: functional: change the HoG SCI mode with the HID Control Point Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 18/23] input/hog: Use shared/hog Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 19/23] doc: Add CONFIG_HIDRAW to the tester kernel config Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 20/23] unit/test-uhid: Add Get Report tests Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 21/23] device: Use bt_att instead of GAttrib Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 22/23] attrib: Remove GAttrib and gatttool Luiz Augusto von Dentz
2026-09-28 20:00 ` [PATCH BlueZ v6 23/23] attrib: Remove directory Luiz Augusto von Dentz
2026-09-29 20:50 ` [PATCH BlueZ v6 00/23] Add HoG functional tests and shared/hog patchwork-bot+bluetooth
-- strict thread matches above, loose matches on Subject: below --
2026-09-24 22:30 [PATCH BlueZ v4 01/20] shared/gatt-client: Fix calling destroy after unregistering notify Luiz Augusto von Dentz
2026-09-25 0:45 ` Add HoG functional tests and shared/hog bluez.test.bot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6abae97b.eeba9552.340f2b.8b59@mx.google.com \
--to=bluez.test.bot@gmail.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox