From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, iulia.tanasescu@nxp.com
Subject: RE: shared/bass: Introduce Add Source opcode handler
Date: Wed, 28 Jun 2023 08:50:14 -0700 (PDT) [thread overview]
Message-ID: <649c56b6.170a0220.e61ef.f395@mx.google.com> (raw)
In-Reply-To: <20230628150705.248285-2-iulia.tanasescu@nxp.com>
[-- Attachment #1: Type: text/plain, Size: 11117 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=760971
---Test result---
Test Summary:
CheckPatch PASS 1.48 seconds
GitLint PASS 0.51 seconds
BuildEll PASS 34.60 seconds
BluezMake FAIL 25.80 seconds
MakeCheck FAIL 40.85 seconds
MakeDistcheck FAIL 44.29 seconds
CheckValgrind FAIL 21.60 seconds
CheckSmatch FAIL 29.55 seconds
bluezmakeextell FAIL 19.27 seconds
IncrementalBuild FAIL 1069.65 seconds
ScanBuild FAIL 746.90 seconds
Details
##############################
Test: BluezMake - FAIL
Desc: Build BlueZ
Output:
src/shared/bass.c: In function ‘bass_handle_add_src_op’:
src/shared/bass.c:835:21: error: ‘bis_sync_bit’ undeclared (first use in this function)
835 | bis[num_bis] = bis_sync_bit + 1;
| ^~~~~~~~~~~~
src/shared/bass.c:835:21: note: each undeclared identifier is reported only once for each function it appears in
src/shared/bass.c:855:10: error: ‘bc_addr’ undeclared (first use in this function); did you mean ‘in_addr’?
855 | bacpy(&bc_addr.bc_bdaddr, &bcast_src->addr);
| ^~~~~~~
| in_addr
make[1]: *** [Makefile:8399: src/shared/libshared_mainloop_la-bass.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4528: all] Error 2
##############################
Test: MakeCheck - FAIL
Desc: Run Bluez Make Check
Output:
src/shared/bass.c: In function ‘bass_handle_add_src_op’:
src/shared/bass.c:835:21: error: ‘bis_sync_bit’ undeclared (first use in this function)
835 | bis[num_bis] = bis_sync_bit + 1;
| ^~~~~~~~~~~~
src/shared/bass.c:835:21: note: each undeclared identifier is reported only once for each function it appears in
src/shared/bass.c:855:10: error: ‘bc_addr’ undeclared (first use in this function); did you mean ‘in_addr’?
855 | bacpy(&bc_addr.bc_bdaddr, &bcast_src->addr);
| ^~~~~~~
| in_addr
make[1]: *** [Makefile:8182: src/shared/libshared_glib_la-bass.lo] Error 1
make: *** [Makefile:11863: check] Error 2
##############################
Test: MakeDistcheck - FAIL
Desc: Run Bluez Make Distcheck
Output:
../../src/shared/bass.c: In function ‘bass_handle_add_src_op’:
../../src/shared/bass.c:835:21: error: ‘bis_sync_bit’ undeclared (first use in this function)
835 | bis[num_bis] = bis_sync_bit + 1;
| ^~~~~~~~~~~~
../../src/shared/bass.c:835:21: note: each undeclared identifier is reported only once for each function it appears in
../../src/shared/bass.c:855:10: error: ‘bc_addr’ undeclared (first use in this function); did you mean ‘in_addr’?
855 | bacpy(&bc_addr.bc_bdaddr, &bcast_src->addr);
| ^~~~~~~
| in_addr
make[2]: *** [Makefile:8399: src/shared/libshared_mainloop_la-bass.lo] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [Makefile:4528: all] Error 2
make: *** [Makefile:11784: distcheck] Error 1
##############################
Test: CheckValgrind - FAIL
Desc: Run Bluez Make Check with Valgrind
Output:
src/shared/bass.c: In function ‘bass_handle_add_src_op’:
src/shared/bass.c:835:21: error: ‘bis_sync_bit’ undeclared (first use in this function)
835 | bis[num_bis] = bis_sync_bit + 1;
| ^~~~~~~~~~~~
src/shared/bass.c:835:21: note: each undeclared identifier is reported only once for each function it appears in
src/shared/bass.c:855:10: error: ‘bc_addr’ undeclared (first use in this function); did you mean ‘in_addr’?
855 | bacpy(&bc_addr.bc_bdaddr, &bcast_src->addr);
| ^~~~~~~
| in_addr
make[1]: *** [Makefile:8399: src/shared/libshared_mainloop_la-bass.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:11863: check] Error 2
##############################
Test: CheckSmatch - FAIL
Desc: Run smatch tool with source
Output:
src/shared/crypto.c:271:21: warning: Variable length array is used.
src/shared/crypto.c:272:23: warning: Variable length array is used.
src/shared/gatt-helpers.c: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:276:25: warning: Variable length array is used.
src/shared/gatt-server.c:619:25: warning: Variable length array is used.
src/shared/gatt-server.c:718:25: warning: Variable length array is used.
src/shared/shell.c: note: in included file (through /usr/include/readline/readline.h):
/usr/include/readline/rltypedefs.h:35:23: warning: non-ANSI function declaration of function 'Function'
/usr/include/readline/rltypedefs.h:36:25: warning: non-ANSI function declaration of function 'VFunction'
/usr/include/readline/rltypedefs.h:37:27: warning: non-ANSI function declaration of function 'CPFunction'
/usr/include/readline/rltypedefs.h:38:29: warning: non-ANSI function declaration of function 'CPPFunction'
src/shared/bass.c: In function ‘bass_handle_add_src_op’:
src/shared/bass.c:835:21: error: ‘bis_sync_bit’ undeclared (first use in this function)
835 | bis[num_bis] = bis_sync_bit + 1;
| ^~~~~~~~~~~~
src/shared/bass.c:835:21: note: each undeclared identifier is reported only once for each function it appears in
src/shared/bass.c:855:10: error: ‘bc_addr’ undeclared (first use in this function); did you mean ‘in_addr’?
855 | bacpy(&bc_addr.bc_bdaddr, &bcast_src->addr);
| ^~~~~~~
| in_addr
make[1]: *** [Makefile:8399: src/shared/libshared_mainloop_la-bass.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4528: all] Error 2
##############################
Test: bluezmakeextell - FAIL
Desc: Build Bluez with External ELL
Output:
src/shared/bass.c: In function ‘bass_handle_add_src_op’:
src/shared/bass.c:835:21: error: ‘bis_sync_bit’ undeclared (first use in this function)
835 | bis[num_bis] = bis_sync_bit + 1;
| ^~~~~~~~~~~~
src/shared/bass.c:835:21: note: each undeclared identifier is reported only once for each function it appears in
src/shared/bass.c:855:10: error: ‘bc_addr’ undeclared (first use in this function); did you mean ‘in_addr’?
855 | bacpy(&bc_addr.bc_bdaddr, &bcast_src->addr);
| ^~~~~~~
| in_addr
make[1]: *** [Makefile:8399: src/shared/libshared_mainloop_la-bass.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4528: all] Error 2
##############################
Test: IncrementalBuild - FAIL
Desc: Incremental build with the patches in the series
Output:
[BlueZ,v2,2/2] shared/bass: Introduce Add Source opcode handler
src/shared/bass.c: In function ‘bass_handle_add_src_op’:
src/shared/bass.c:835:21: error: ‘bis_sync_bit’ undeclared (first use in this function)
835 | bis[num_bis] = bis_sync_bit + 1;
| ^~~~~~~~~~~~
src/shared/bass.c:835:21: note: each undeclared identifier is reported only once for each function it appears in
src/shared/bass.c:855:10: error: ‘bc_addr’ undeclared (first use in this function); did you mean ‘in_addr’?
855 | bacpy(&bc_addr.bc_bdaddr, &bcast_src->addr);
| ^~~~~~~
| in_addr
make[1]: *** [Makefile:8399: src/shared/libshared_mainloop_la-bass.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4528: all] Error 2
##############################
Test: ScanBuild - FAIL
Desc: Run Scan Build
Output:
src/shared/ad.c:369:19: warning: Use of zero-allocated memory
buf[(*pos)++] = ad_type;
^
1 warning generated.
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:993:2: warning: Use of memory after it is freed
discovery_op_complete(op, success, att_ecode);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1099:2: warning: Use of memory after it is freed
discovery_op_complete(op, success, att_ecode);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1291:2: warning: Use of memory after it is freed
discovery_op_complete(op, success, att_ecode);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1356:2: warning: Use of memory after it is freed
discovery_op_complete(op, success, att_ecode);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1631:6: warning: Use of memory after it is freed
if (read_db_hash(op)) {
^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1636:2: warning: Use of memory after it is freed
discover_all(op);
^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:2140:6: warning: Use of memory after it is freed
if (read_db_hash(op)) {
^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:2148:8: warning: Use of memory after it is freed
discovery_op_ref(op),
^~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3236:2: warning: Use of memory after it is freed
complete_write_long_op(req, success, 0, false);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3258:2: warning: Use of memory after it is freed
request_unref(req);
^~~~~~~~~~~~~~~~~~
12 warnings generated.
src/shared/bass.c: In function ‘bass_handle_add_src_op’:
src/shared/bass.c:835:21: error: ‘bis_sync_bit’ undeclared (first use in this function)
835 | bis[num_bis] = bis_sync_bit + 1;
| ^~~~~~~~~~~~
src/shared/bass.c:835:21: note: each undeclared identifier is reported only once for each function it appears in
src/shared/bass.c:855:10: error: ‘bc_addr’ undeclared (first use in this function); did you mean ‘in_addr’?
855 | bacpy(&bc_addr.bc_bdaddr, &bcast_src->addr);
| ^~~~~~~
| in_addr
make[1]: *** [Makefile:8399: src/shared/libshared_mainloop_la-bass.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4528: all] Error 2
---
Regards,
Linux Bluetooth
next prev parent reply other threads:[~2023-06-28 15:50 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-28 15:07 [PATCH BlueZ v2 0/2] shared/bass: Introduce Add Source opcode handler Iulia Tanasescu
2023-06-28 15:07 ` [PATCH BlueZ v2 1/2] btio: Add option for binding iso broadcast address Iulia Tanasescu
2023-06-28 15:50 ` bluez.test.bot [this message]
2023-06-28 15:07 ` [PATCH BlueZ v2 2/2] shared/bass: Introduce Add Source opcode handler Iulia Tanasescu
-- strict thread matches above, loose matches on Subject: below --
2023-06-29 8:07 [PATCH BlueZ v3 1/2] btio: Add options for binding iso broadcast address Iulia Tanasescu
2023-06-29 9:25 ` shared/bass: Introduce Add Source opcode handler bluez.test.bot
2023-06-23 7:46 [PATCH BlueZ 1/2] btio: Add option for binding iso broadcast address Iulia Tanasescu
2023-06-23 9:48 ` shared/bass: Introduce Add Source opcode handler 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=649c56b6.170a0220.e61ef.f395@mx.google.com \
--to=bluez.test.bot@gmail.com \
--cc=iulia.tanasescu@nxp.com \
--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.