All of lore.kernel.org
 help / color / mirror / Atom feed
From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, cochcoder@proton.me
Subject: RE: [RESEND,RFC] l2cap-tester: Check compatible interval preferences
Date: Sat, 31 Oct 2026 11:58:56 -0700 (PDT)	[thread overview]
Message-ID: <6ae63a70.5986774d.3126b0.4684@mx.google.com> (raw)
In-Reply-To: <20261031181244.123461-1-cochcoder@proton.me>

[-- Attachment #1: Type: text/plain, Size: 25999 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/1197860/

---Test result---

Test Summary:
CheckPatch                    PASS      0.43 seconds
GitLint                       FAIL      0.28 seconds
BuildEll                      PASS      13.97 seconds
BluezMake                     FAIL      40.94 seconds
CheckSmatch                   FAIL      55.78 seconds
bluezmakeextell               FAIL      16.34 seconds
TestFunctional                ERROR     41.55 seconds
IncrementalBuild              FAIL      44.96 seconds
ScanBuild                     FAIL      73.11 seconds

Details
##############################
Test: GitLint - FAIL
Desc: Run gitlint
Output:
[RESEND,RFC] l2cap-tester: Check compatible interval preferences

32: B1 Line exceeds max length (104>80): "made for this standalone test patch. The companion remains an RFC; kernel acceptance is not established."
##############################
Test: BluezMake - FAIL
Desc: Build BlueZ
Output:

tools/l2cap-tester.c: In function ‘interval_loaded’:
tools/l2cap-tester.c:3619:9: error: unknown type name ‘GIOChannel’
 3619 |         GIOChannel *io;
      |         ^~~~~~~~~~
tools/l2cap-tester.c:3645:14: error: implicit declaration of function ‘g_io_channel_unix_new’; did you mean ‘io_channel_unref’? [-Wimplicit-function-declaration]
 3645 |         io = g_io_channel_unix_new(sk);
      |              ^~~~~~~~~~~~~~~~~~~~~
      |              io_channel_unref
tools/l2cap-tester.c:3645:12: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
 3645 |         io = g_io_channel_unix_new(sk);
      |            ^
tools/l2cap-tester.c:3646:9: error: implicit declaration of function ‘g_io_channel_set_close_on_unref’ [-Wimplicit-function-declaration]
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/l2cap-tester.c:3646:45: error: ‘TRUE’ undeclared (first use in this function)
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |                                             ^~~~
tools/l2cap-tester.c:3646:45: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [Makefile:7191: tools/l2cap-tester.o] Error 1
make[1]: *** Waiting for unfinished jobs....
tools/mgmt-tester.c: In function ‘main’:
tools/mgmt-tester.c:14256:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
14256 | int main(int argc, char *argv[])
      |     ^~~~
make: *** [Makefile:4282: all] 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/bap.c:309: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/x86_64-linux-gnu//sys/ioctl.h, /usr/include/x86_64-linux-gnu//sys/epoll.h, src/shared/mainloop.h):
/usr/include/x86_64-linux-gnu//sys/ttydefaults.h:55:9: warning: preprocessor token CTRL redefined
src/shared/shell.c: note: in included file (through /usr/include/readline/keymaps.h, /usr/include/readline/readline.h):
/usr/include/readline/chardefs.h:51:9: this was the original definition
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/bap.c:309: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/x86_64-linux-gnu//sys/ioctl.h, /usr/include/x86_64-linux-gnu//sys/epoll.h, src/shared/mainloop.h):
/usr/include/x86_64-linux-gnu//sys/ttydefaults.h:55:9: warning: preprocessor token CTRL redefined
src/shared/shell.c: note: in included file (through /usr/include/readline/keymaps.h, /usr/include/readline/readline.h):
/usr/include/readline/chardefs.h:51:9: this was the original definition
tools/mesh-cfgtest.c:1453:17: warning: unknown escape sequence: '\%'
tools/l2cap-tester.c:3620:9: warning: mixing declarations and code
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/l2cap-tester.c: In function ‘interval_loaded’:
tools/l2cap-tester.c:3619:9: error: unknown type name ‘GIOChannel’
 3619 |         GIOChannel *io;
      |         ^~~~~~~~~~
tools/l2cap-tester.c:3645:14: error: implicit declaration of function ‘g_io_channel_unix_new’; did you mean ‘io_channel_unref’? [-Wimplicit-function-declaration]
 3645 |         io = g_io_channel_unix_new(sk);
      |              ^~~~~~~~~~~~~~~~~~~~~
      |              io_channel_unref
tools/l2cap-tester.c:3645:12: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
 3645 |         io = g_io_channel_unix_new(sk);
      |            ^
tools/l2cap-tester.c:3646:9: error: implicit declaration of function ‘g_io_channel_set_close_on_unref’ [-Wimplicit-function-declaration]
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/l2cap-tester.c:3646:45: error: ‘TRUE’ undeclared (first use in this function)
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |                                             ^~~~
tools/l2cap-tester.c:3646:45: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [Makefile:7191: tools/l2cap-tester.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4282: all] Error 2
##############################
Test: bluezmakeextell - FAIL
Desc: Build Bluez with External ELL
Output:

tools/l2cap-tester.c: In function ‘interval_loaded’:
tools/l2cap-tester.c:3619:9: error: unknown type name ‘GIOChannel’
 3619 |         GIOChannel *io;
      |         ^~~~~~~~~~
tools/l2cap-tester.c:3645:14: error: implicit declaration of function ‘g_io_channel_unix_new’; did you mean ‘io_channel_unref’? [-Wimplicit-function-declaration]
 3645 |         io = g_io_channel_unix_new(sk);
      |              ^~~~~~~~~~~~~~~~~~~~~
      |              io_channel_unref
tools/l2cap-tester.c:3645:12: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
 3645 |         io = g_io_channel_unix_new(sk);
      |            ^
tools/l2cap-tester.c:3646:9: error: implicit declaration of function ‘g_io_channel_set_close_on_unref’ [-Wimplicit-function-declaration]
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/l2cap-tester.c:3646:45: error: ‘TRUE’ undeclared (first use in this function)
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |                                             ^~~~
tools/l2cap-tester.c:3646:45: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [Makefile:7191: tools/l2cap-tester.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4282: all] Error 2
##############################
Test: TestFunctional - ERROR
Desc: Run test-functional
Output:
Failed to build BlueZ:
Failed to build BlueZ
##############################
Test: IncrementalBuild - FAIL
Desc: Incremental build with the patches in the series
Output:

tools/l2cap-tester.c: In function ‘interval_loaded’:
tools/l2cap-tester.c:3619:9: error: unknown type name ‘GIOChannel’
 3619 |         GIOChannel *io;
      |         ^~~~~~~~~~
tools/l2cap-tester.c:3645:14: error: implicit declaration of function ‘g_io_channel_unix_new’; did you mean ‘io_channel_unref’? [-Wimplicit-function-declaration]
 3645 |         io = g_io_channel_unix_new(sk);
      |              ^~~~~~~~~~~~~~~~~~~~~
      |              io_channel_unref
tools/l2cap-tester.c:3645:12: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
 3645 |         io = g_io_channel_unix_new(sk);
      |            ^
tools/l2cap-tester.c:3646:9: error: implicit declaration of function ‘g_io_channel_set_close_on_unref’ [-Wimplicit-function-declaration]
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/l2cap-tester.c:3646:45: error: ‘TRUE’ undeclared (first use in this function)
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |                                             ^~~~
tools/l2cap-tester.c:3646:45: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [Makefile:7191: tools/l2cap-tester.o] Error 1
make[1]: *** Waiting for unfinished jobs....
tools/mgmt-tester.c: In function ‘main’:
tools/mgmt-tester.c:14256:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
14256 | int main(int argc, char *argv[])
      |     ^~~~
make: *** [Makefile:4282: all] Error 2
[RESEND,RFC] l2cap-tester: Check compatible interval preferences

tools/l2cap-tester.c: In function ‘interval_loaded’:
tools/l2cap-tester.c:3619:9: error: unknown type name ‘GIOChannel’
 3619 |         GIOChannel *io;
      |         ^~~~~~~~~~
tools/l2cap-tester.c:3645:14: error: implicit declaration of function ‘g_io_channel_unix_new’; did you mean ‘io_channel_unref’? [-Wimplicit-function-declaration]
 3645 |         io = g_io_channel_unix_new(sk);
      |              ^~~~~~~~~~~~~~~~~~~~~
      |              io_channel_unref
tools/l2cap-tester.c:3645:12: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
 3645 |         io = g_io_channel_unix_new(sk);
      |            ^
tools/l2cap-tester.c:3646:9: error: implicit declaration of function ‘g_io_channel_set_close_on_unref’ [-Wimplicit-function-declaration]
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/l2cap-tester.c:3646:45: error: ‘TRUE’ undeclared (first use in this function)
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |                                             ^~~~
tools/l2cap-tester.c:3646:45: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [Makefile:7191: tools/l2cap-tester.o] Error 1
make[1]: *** Waiting for unfinished jobs....
tools/mgmt-tester.c: In function ‘main’:
tools/mgmt-tester.c:14256:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
14256 | int main(int argc, char *argv[])
      |     ^~~~
make: *** [Makefile:4282: all] Error 2
##############################
Test: ScanBuild - FAIL
Desc: Run Scan Build
Output:

src/shared/gatt-client.c:458:21: warning: Use of memory after it is freed [unix.Malloc]
  458 |         gatt_db_unregister(op->client->db, op->db_id);
      |                            ^~~~~~~~~~
src/shared/gatt-client.c:703:2: warning: Use of memory after it is freed [unix.Malloc]
  703 |         discovery_op_complete(op, false, att_ecode);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1109:2: warning: Use of memory after it is freed [unix.Malloc]
 1109 |         discovery_op_complete(op, success, att_ecode);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1309:2: warning: Use of memory after it is freed [unix.Malloc]
 1309 |         discovery_op_complete(op, success, att_ecode);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1374:2: warning: Use of memory after it is freed [unix.Malloc]
 1374 |         discovery_op_complete(op, success, att_ecode);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1649:6: warning: Use of memory after it is freed [unix.Malloc]
 1649 |         if (read_db_hash(op)) {
      |             ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1654:2: warning: Use of memory after it is freed [unix.Malloc]
 1654 |         discover_all(op);
      |         ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1709:56: warning: Use of memory after it is freed [unix.Malloc]
 1709 |         notify_data->chrc->ccc_write_id = notify_data->att_id = att_id;
      |                                           ~~~~~~~~~~~~~~~~~~~ ^
src/shared/gatt-client.c:2176:6: warning: Use of memory after it is freed [unix.Malloc]
 2176 |         if (read_db_hash(op)) {
      |             ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:2184:8: warning: Use of memory after it is freed [unix.Malloc]
 2184 |                                                         discovery_op_ref(op),
      |                                                         ^~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3370:2: warning: Use of memory after it is freed [unix.Malloc]
 3370 |         complete_write_long_op(req, success, 0, false);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3392:2: warning: Use of memory after it is freed [unix.Malloc]
 3392 |         request_unref(req);
      |         ^~~~~~~~~~~~~~~~~~
12 warnings generated.
src/shared/rap.c:3936:30: warning: Assigned value is uninitialized [core.uninitialized.Assign]
 3936 |                         out->tone_pct[k].i_sample = tone_pct_i[k];
      |                                                   ^ ~~~~~~~~~~~~~
src/shared/rap.c:4981:13: warning: Use of memory after it is freed [unix.Malloc]
 4981 |         attached = queue_find(sessions, NULL, rap);
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 warnings generated.
src/shared/bap.c:1534:8: warning: Use of memory after it is freed [unix.Malloc]
 1534 |         bap = bt_bap_ref_safe(bap);
      |               ^~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:2350:20: warning: Use of memory after it is freed [unix.Malloc]
 2350 |         return queue_find(stream->bap->streams, NULL, stream);
      |                           ^~~~~~~~~~~~~~~~~~~~
2 warnings generated.
src/shared/gatt-client.c:458:21: warning: Use of memory after it is freed [unix.Malloc]
  458 |         gatt_db_unregister(op->client->db, op->db_id);
      |                            ^~~~~~~~~~
src/shared/gatt-client.c:703:2: warning: Use of memory after it is freed [unix.Malloc]
  703 |         discovery_op_complete(op, false, att_ecode);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1109:2: warning: Use of memory after it is freed [unix.Malloc]
 1109 |         discovery_op_complete(op, success, att_ecode);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1309:2: warning: Use of memory after it is freed [unix.Malloc]
 1309 |         discovery_op_complete(op, success, att_ecode);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1374:2: warning: Use of memory after it is freed [unix.Malloc]
 1374 |         discovery_op_complete(op, success, att_ecode);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1649:6: warning: Use of memory after it is freed [unix.Malloc]
 1649 |         if (read_db_hash(op)) {
      |             ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1654:2: warning: Use of memory after it is freed [unix.Malloc]
 1654 |         discover_all(op);
      |         ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1709:56: warning: Use of memory after it is freed [unix.Malloc]
 1709 |         notify_data->chrc->ccc_write_id = notify_data->att_id = att_id;
      |                                           ~~~~~~~~~~~~~~~~~~~ ^
src/shared/gatt-client.c:2176:6: warning: Use of memory after it is freed [unix.Malloc]
 2176 |         if (read_db_hash(op)) {
      |             ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:2184:8: warning: Use of memory after it is freed [unix.Malloc]
 2184 |                                                         discovery_op_ref(op),
      |                                                         ^~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3370:2: warning: Use of memory after it is freed [unix.Malloc]
 3370 |         complete_write_long_op(req, success, 0, false);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3392:2: warning: Use of memory after it is freed [unix.Malloc]
 3392 |         request_unref(req);
      |         ^~~~~~~~~~~~~~~~~~
12 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.
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.
src/shared/rap.c:3936:30: warning: Assigned value is uninitialized [core.uninitialized.Assign]
 3936 |                         out->tone_pct[k].i_sample = tone_pct_i[k];
      |                                                   ^ ~~~~~~~~~~~~~
src/shared/rap.c:4981:13: warning: Use of memory after it is freed [unix.Malloc]
 4981 |         attached = queue_find(sessions, NULL, rap);
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 warnings generated.
src/shared/bap.c:1534:8: warning: Use of memory after it is freed [unix.Malloc]
 1534 |         bap = bt_bap_ref_safe(bap);
      |               ^~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:2350:20: warning: Use of memory after it is freed [unix.Malloc]
 2350 |         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/l2cap-tester.c: In function ‘interval_loaded’:
tools/l2cap-tester.c:3619:9: error: unknown type name ‘GIOChannel’
 3619 |         GIOChannel *io;
      |         ^~~~~~~~~~
tools/l2cap-tester.c:3645:14: error: implicit declaration of function ‘g_io_channel_unix_new’; did you mean ‘io_channel_unref’? [-Wimplicit-function-declaration]
 3645 |         io = g_io_channel_unix_new(sk);
      |              ^~~~~~~~~~~~~~~~~~~~~
      |              io_channel_unref
tools/l2cap-tester.c:3645:12: error: assignment to ‘int *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
 3645 |         io = g_io_channel_unix_new(sk);
      |            ^
tools/l2cap-tester.c:3646:9: error: implicit declaration of function ‘g_io_channel_set_close_on_unref’ [-Wimplicit-function-declaration]
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/l2cap-tester.c:3646:45: error: ‘TRUE’ undeclared (first use in this function)
 3646 |         g_io_channel_set_close_on_unref(io, TRUE);
      |                                             ^~~~
tools/l2cap-tester.c:3646:45: note: each undeclared identifier is reported only once for each function it appears in
make[1]: *** [Makefile:7191: tools/l2cap-tester.o] Error 1
make[1]: *** Waiting for unfinished jobs....
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.
make: *** [Makefile:4282: all] Error 2


https://github.com/bluez/bluez/pull/2743

---
Regards,
Linux Bluetooth


  reply	other threads:[~2026-10-31 18:58 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-31 18:13 [RESEND RFC PATCH] l2cap-tester: Check compatible interval preferences Kyle Pepper
2026-10-31 18:58 ` bluez.test.bot [this message]
2026-10-31 19:37 ` [RFC PATCH v2] " Kyle Pepper
2026-10-31 22:56   ` [RFC,v2] " 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=6ae63a70.5986774d.3126b0.4684@mx.google.com \
    --to=bluez.test.bot@gmail.com \
    --cc=cochcoder@proton.me \
    --cc=linux-bluetooth@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.