All of lore.kernel.org
 help / color / mirror / Atom feed
From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, pav@iki.fi
Subject: RE: mcp: support multiple MCP and implement local GMCS
Date: Thu, 11 Dec 2025 11:44:12 -0800 (PST)	[thread overview]
Message-ID: <693b1f0c.050a0220.446c9.afe4@mx.google.com> (raw)
In-Reply-To: <f343a41406876428005288982a73bc20ae5d10b7.1765480081.git.pav@iki.fi>

[-- Attachment #1: Type: text/plain, Size: 8279 bytes --]

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=1032419

---Test result---

Test Summary:
CheckPatch                    PENDING   0.29 seconds
GitLint                       PENDING   0.36 seconds
BuildEll                      PASS      20.82 seconds
BluezMake                     PASS      649.15 seconds
MakeCheck                     PASS      22.11 seconds
MakeDistcheck                 PASS      242.22 seconds
CheckValgrind                 PASS      309.15 seconds
CheckSmatch                   FAIL      22.08 seconds
bluezmakeextell               FAIL      14.06 seconds
IncrementalBuild              PENDING   0.29 seconds
ScanBuild                     FAIL      35.89 seconds

Details
##############################
Test: CheckPatch - PENDING
Desc: Run checkpatch.pl script
Output:

##############################
Test: GitLint - PENDING
Desc: Run gitlint
Output:

##############################
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:768:31: warning: Variable length array is used.
src/shared/gatt-helpers.c:830:31: warning: Variable length array is used.
src/shared/gatt-helpers.c:1323:31: warning: Variable length array is used.
src/shared/gatt-helpers.c:1354:23: warning: Variable length array is used.
src/shared/gatt-server.c:278:25: warning: Variable length array is used.
src/shared/gatt-server.c:618:25: warning: Variable length array is used.
src/shared/gatt-server.c:716:25: warning: Variable length array is used.
src/shared/bap.c:312:25: warning: array of flexible structures
src/shared/bap.c: note: in included file:
./src/shared/ascs.h:88:25: warning: array of flexible structures
src/shared/shell.c: note: in included file (through /usr/include/readline/readline.h):
/usr/include/readline/rltypedefs.h:35:23: warning: non-ANSI function declaration of function 'Function'
/usr/include/readline/rltypedefs.h:36:25: warning: non-ANSI function declaration of function 'VFunction'
/usr/include/readline/rltypedefs.h:37:27: warning: non-ANSI function declaration of function 'CPFunction'
/usr/include/readline/rltypedefs.h:38:29: warning: non-ANSI function declaration of function 'CPPFunction'
In file included from /usr/include/string.h:495,
                 from src/shared/uinput-util.c:21:
In function ‘strncpy’,
    inlined from ‘bt_uinput_new’ at src/shared/uinput-util.c:132:4:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:10: error: ‘__builtin_strncpy’ specified bound depends on the length of the source argument [-Werror=stringop-overflow=]
  106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/uinput-util.c: In function ‘bt_uinput_new’:
src/shared/uinput-util.c:121:10: note: length computed here
  121 |   slen = strlen(suffix);
      |          ^~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7956: src/shared/libshared_mainloop_la-uinput-util.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4220: all] Error 2
##############################
Test: bluezmakeextell - FAIL
Desc: Build Bluez with External ELL
Output:

In file included from /usr/include/string.h:495,
                 from src/shared/uinput-util.c:21:
In function ‘strncpy’,
    inlined from ‘bt_uinput_new’ at src/shared/uinput-util.c:132:4:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:10: error: ‘__builtin_strncpy’ specified bound depends on the length of the source argument [-Werror=stringop-overflow=]
  106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/uinput-util.c: In function ‘bt_uinput_new’:
src/shared/uinput-util.c:121:10: note: length computed here
  121 |   slen = strlen(suffix);
      |          ^~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7956: src/shared/libshared_mainloop_la-uinput-util.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4220: all] Error 2
##############################
Test: IncrementalBuild - PENDING
Desc: Incremental build with the patches in the series
Output:

##############################
Test: ScanBuild - FAIL
Desc: Run Scan Build
Output:

src/shared/gatt-client.c:451:21: warning: Use of memory after it is freed
        gatt_db_unregister(op->client->db, op->db_id);
                           ^~~~~~~~~~
src/shared/gatt-client.c:696:2: warning: Use of memory after it is freed
        discovery_op_complete(op, false, att_ecode);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:996:2: warning: Use of memory after it is freed
        discovery_op_complete(op, success, att_ecode);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1102:2: warning: Use of memory after it is freed
        discovery_op_complete(op, success, att_ecode);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1296:2: warning: Use of memory after it is freed
        discovery_op_complete(op, success, att_ecode);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1361:2: warning: Use of memory after it is freed
        discovery_op_complete(op, success, att_ecode);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1636:6: warning: Use of memory after it is freed
        if (read_db_hash(op)) {
            ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1641:2: warning: Use of memory after it is freed
        discover_all(op);
        ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1697:56: warning: Use of memory after it is freed
        notify_data->chrc->ccc_write_id = notify_data->att_id = att_id;
                                          ~~~~~~~~~~~~~~~~~~~ ^
src/shared/gatt-client.c:2150:6: warning: Use of memory after it is freed
        if (read_db_hash(op)) {
            ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:2158:8: warning: Use of memory after it is freed
                                                        discovery_op_ref(op),
                                                        ^~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3183:2: warning: Use of memory after it is freed
        complete_write_long_op(req, success, 0, false);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3205:2: warning: Use of memory after it is freed
        request_unref(req);
        ^~~~~~~~~~~~~~~~~~
13 warnings generated.
In file included from /usr/include/string.h:495,
                 from src/shared/uinput-util.c:21:
In function ‘strncpy’,
    inlined from ‘bt_uinput_new’ at src/shared/uinput-util.c:132:4:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:10: error: ‘__builtin_strncpy’ specified bound depends on the length of the source argument [-Werror=stringop-overflow=]
  106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/uinput-util.c: In function ‘bt_uinput_new’:
src/shared/uinput-util.c:121:10: note: length computed here
  121 |   slen = strlen(suffix);
      |          ^~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7956: src/shared/libshared_mainloop_la-uinput-util.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
src/shared/bap.c:1529:8: warning: Use of memory after it is freed
        bap = bt_bap_ref_safe(bap);
              ^~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:2323:20: warning: Use of memory after it is freed
        return queue_find(stream->bap->streams, NULL, stream);
                          ^~~~~~~~~~~~~~~~~~~~
2 warnings generated.
make: *** [Makefile:4220: all] Error 2


---
Regards,
Linux Bluetooth


  reply	other threads:[~2025-12-11 19:44 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-11 19:08 [PATCH BlueZ v4 0/7] mcp: support multiple MCP and implement local GMCS Pauli Virtanen
2025-12-11 19:08 ` [PATCH BlueZ v4 1/7] shared/mcp: support multiple MCP, and add non-stub MCS server Pauli Virtanen
2025-12-11 19:44   ` bluez.test.bot [this message]
2025-12-11 19:08 ` [PATCH BlueZ v4 2/7] test-mcp: add tests for MCP / MCS Pauli Virtanen
2025-12-11 19:08 ` [PATCH BlueZ v4 3/7] mcp: adapt to new MCP API to support multiple remote MCS services Pauli Virtanen
2025-12-11 19:08 ` [PATCH BlueZ v4 4/7] shared/uinput-util: extract uinput utility function from AVCTP Pauli Virtanen
2025-12-11 19:08 ` [PATCH BlueZ v4 5/7] avctp: use uinput utilities from src/shared Pauli Virtanen
2025-12-11 19:08 ` [PATCH BlueZ v4 6/7] mcp: add local GMCS service that emits uinput media keys Pauli Virtanen
2025-12-11 19:08 ` [PATCH BlueZ v4 7/7] shared/gatt-client: fix notify_data leak in notify_data_write_ccc Pauli Virtanen
  -- strict thread matches above, loose matches on Subject: below --
2025-12-12 20:45 [PATCH BlueZ v7 1/9] shared/mcp, mcp: support multiple MCP, and add non-stub MCS server Pauli Virtanen
2025-12-12 21:37 ` mcp: support multiple MCP and implement local GMCS bluez.test.bot
2025-12-12 20:12 [PATCH BlueZ v6 1/9] shared/mcp, mcp: support multiple MCP, and add non-stub MCS server Pauli Virtanen
2025-12-12 20:29 ` mcp: support multiple MCP and implement local GMCS bluez.test.bot
2025-12-10 16:13 [PATCH BlueZ v3 1/7] shared/mcp: support multiple MCP, and add non-stub MCS server Pauli Virtanen
2025-12-10 17:21 ` mcp: support multiple MCP and implement local GMCS bluez.test.bot
2025-12-10 15:44 [PATCH BlueZ v2.. 1/7] shared/mcp: support multiple MCP, and add non-stub MCS server Pauli Virtanen
2025-12-10 16:20 ` mcp: support multiple MCP and implement local GMCS bluez.test.bot
2025-12-09 21:15 [PATCH BlueZ 1/6] shared/mcp: support multiple MCP, and add non-stub MCS server Pauli Virtanen
2025-12-09 21:34 ` mcp: support multiple MCP and implement local GMCS 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=693b1f0c.050a0220.446c9.afe4@mx.google.com \
    --to=bluez.test.bot@gmail.com \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=pav@iki.fi \
    /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.