From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, cheng.jiang@oss.qualcomm.com
Subject: RE: Ranging Profile and Ranging Service PTS fixes
Date: Thu, 01 Oct 2026 09:03:17 -0700 (PDT) [thread overview]
Message-ID: <6abe8445.e5f0f813.32aaf1.3b9b@mx.google.com> (raw)
In-Reply-To: <20261001131900.707485-2-cheng.jiang@oss.qualcomm.com>
[-- Attachment #1: Type: text/plain, Size: 26188 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/1177680/
---Test result---
Test Summary:
CheckPatch FAIL 5.81 seconds
GitLint PASS 3.49 seconds
BuildEll PASS 19.30 seconds
BluezMake FAIL 22.19 seconds
MakeCheck FAIL 49.12 seconds
MakeDistcheck PASS 140.88 seconds
CheckValgrind FAIL 18.15 seconds
CheckSmatch FAIL 27.94 seconds
bluezmakeextell FAIL 14.71 seconds
TestFunctional ERROR 23.97 seconds
IncrementalBuild FAIL 578.13 seconds
ScanBuild FAIL 43.48 seconds
Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script
Output:
[01/11] shared/gatt-client: Allow registering notify with explicit CCC value
WARNING:LONG_LINE: line length of 82 exceeds 80 columns
#170: FILE: src/shared/gatt-client.c:1782:
+ if (ccc_value == 0x0001 && !(chrc->properties & BT_GATT_CHRC_PROP_NOTIFY))
WARNING:LONG_LINE: line length of 84 exceeds 80 columns
#173: FILE: src/shared/gatt-client.c:1785:
+ if (ccc_value == 0x0002 && !(chrc->properties & BT_GATT_CHRC_PROP_INDICATE))
/home/runner/work/bluez/bluez/src/patch/14861158.patch total: 0 errors, 2 warnings, 97 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/14861158.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.
[03/11] shared/rap: Honor CCCD transport for RAS on-demand ranging data
WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#136:
and the next segment is queued from the confirmation callback; notifications
WARNING:LONG_LINE_COMMENT: line length of 81 exceeds 80 columns
#192: FILE: src/shared/rap.c:395:
+ * transport (notification vs indication) the client actually enabled. */
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#192: FILE: src/shared/rap.c:395:
+ * transport (notification vs indication) the client actually enabled. */
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#248: FILE: src/shared/rap.c:1248:
+ ras_cp_send_response_code(rap, ras, RAS_CP_RSP_CODE_SERVER_BUSY);
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#302: FILE: src/shared/rap.c:1403:
+ * Notification). */
WARNING:LONG_LINE: line length of 84 exceeds 80 columns
#655: FILE: src/shared/rap.c:2086:
+ rap->cache.counter);
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#664: FILE: src/shared/rap.c:2095:
+ ras_cp_send_response_code(rap, ras, RAS_CP_RSP_CODE_SERVER_BUSY);
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#694: FILE: src/shared/rap.c:2203:
+ if (ras_data_ccc_use_indication(ras->realtime_ccc_value))
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#709: FILE: src/shared/rap.c:2215:
+ if (ras_data_ccc_use_indication(ras->ondemand_ccc_value))
WARNING:LONG_LINE: line length of 83 exceeds 80 columns
#799: FILE: src/shared/rap.c:4184:
+ DBG(rap, "Ignoring RFU RAS CP response code 0x%02x", code);
/home/runner/work/bluez/bluez/src/patch/14861164.patch total: 0 errors, 10 warnings, 623 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/14861164.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.
[06/11] shared/rap: Add real-time ranging data delivery on the responder
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#228: FILE: src/shared/rap.c:2204:
+ gatt_db_attribute_get_value(ras->realtime_chrc));
WARNING:LONG_LINE: line length of 83 exceeds 80 columns
#235: FILE: src/shared/rap.c:2211:
+ id = bt_att_send(bt_rap_get_att(rap), BT_ATT_OP_HANDLE_IND,
WARNING:LONG_LINE: line length of 84 exceeds 80 columns
#236: FILE: src/shared/rap.c:2212:
+ pdu, 2 + segment->len, realtime_ind_conf_cb,
WARNING:LONG_LINE_COMMENT: line length of 87 exceeds 80 columns
#286: FILE: src/shared/rap.c:2289:
+ /* Only the final completed procedure segment terminates the stream. */
WARNING:LONG_LINE_STRING: line length of 82 exceeds 80 columns
#342: FILE: src/shared/rap.c:2323:
+ DBG(rap, "Failed to queue real-time RAS segment");
WARNING:LONG_LINE_COMMENT: line length of 91 exceeds 80 columns
#364: FILE: src/shared/rap.c:2709:
+ /* A new real-time procedure supersedes queued old ranging data. */
/home/runner/work/bluez/bluez/src/patch/14861163.patch total: 0 errors, 6 warnings, 238 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/14861163.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.
[07/11] shared/rap: Add requester ranging data subscription API
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#201: FILE: src/shared/rap.c:3433:
+ notify->id = bt_gatt_client_register_notify_with_ccc(rap->client,
WARNING:LONG_LINE: line length of 87 exceeds 80 columns
#206: FILE: src/shared/rap.c:3438:
+ value_handle, ras_register, rap_notify, notify,
WARNING:LONG_LINE: line length of 96 exceeds 80 columns
#324: FILE: src/shared/rap.c:4425:
+ uint16_t ccc_value, rap_notify_t func, const char *name,
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#357: FILE: src/shared/rap.c:4465:
+ * A Ready notification, not its current value, starts a retrieval. */
WARNING:LONG_LINE: line length of 82 exceeds 80 columns
#373: FILE: src/shared/rap.c:4481:
+ !gatt_db_attribute_get_char_data(ras->realtime_chrc, NULL,
WARNING:LONG_LINE: line length of 89 exceeds 80 columns
#374: FILE: src/shared/rap.c:4482:
+ &value_handle, NULL, NULL, NULL))
WARNING:LONG_LINE: line length of 83 exceeds 80 columns
#377: FILE: src/shared/rap.c:4485:
+ rap->realtime_notify_id = bt_rap_register_notify(rap, value_handle,
WARNING:LONG_LINE_COMMENT: line length of 82 exceeds 80 columns
#389: FILE: src/shared/rap.c:4497:
+ /* Data is segmented and can be large; keep it notification-based.
WARNING:LONG_LINE_COMMENT: line length of 85 exceeds 80 columns
#390: FILE: src/shared/rap.c:4498:
+ * The control/status characteristics use the requested transport. */
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#390: FILE: src/shared/rap.c:4498:
+ * The control/status characteristics use the requested transport. */
WARNING:LONG_LINE: line length of 86 exceeds 80 columns
#393: FILE: src/shared/rap.c:4501:
+ ras_segment_notify_cb, "on-demand", &rap->ondemand_notify_id);
WARNING:LONG_LINE: line length of 92 exceeds 80 columns
#396: FILE: src/shared/rap.c:4504:
+ ras_cp_response_notify_cb, "RAS Control Point", &rap->cp_notify_id);
WARNING:LONG_LINE: line length of 95 exceeds 80 columns
#399: FILE: src/shared/rap.c:4507:
+ ras_data_ready_notify_cb, "Ranging Data Ready", &rap->ready_notify_id);
WARNING:LONG_LINE_STRING: line length of 83 exceeds 80 columns
#402: FILE: src/shared/rap.c:4510:
+ ras_data_overwritten_notify_cb, "Ranging Data Overwritten",
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#406: FILE: src/shared/rap.c:4514:
+ &value_handle, NULL, NULL, NULL))
WARNING:LONG_LINE: line length of 87 exceeds 80 columns
#407: FILE: src/shared/rap.c:4515:
+ rap_read_value(rap, value_handle, ras_data_ready_read_complete,
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#525: FILE: src/shared/rap.h:319:
+ * called after remote RAS discovery and Features read complete. */
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#531: FILE: src/shared/rap.h:325:
+ * disables the characteristic CCCD when this is its final subscription. */
/home/runner/work/bluez/bluez/src/patch/14861166.patch total: 0 errors, 18 warnings, 372 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/14861166.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.
[08/11] profiles/ranging: Expose requester ranging data mode over D-Bus
WARNING:LONG_LINE_COMMENT: line length of 81 exceeds 80 columns
#134: FILE: profiles/ranging/rap.c:881:
+ /* The Requester owns the Real-time CCCD subscription. Unregister it when
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#135: FILE: profiles/ranging/rap.c:882:
+ * the procedure ends so the peer receives the required CCCD disable. */
/home/runner/work/bluez/bluez/src/patch/14861162.patch total: 0 errors, 2 warnings, 52 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/14861162.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.
[09/11] client: Add Channel Sounding ranging data mode control
WARNING:LONG_LINE: line length of 82 exceeds 80 columns
#151: FILE: client/cs.c:1450:
+ bt_shell_printf("SetRangingDataMode failed: %s\n", error.message);
WARNING:LONG_LINE: line length of 89 exceeds 80 columns
#189: FILE: client/cs.c:1488:
+ bt_shell_printf("Invalid Ranging Data transport: %s\n", argv[3]);
WARNING:LONG_LINE_STRING: line length of 83 exceeds 80 columns
#202: FILE: client/cs.c:1501:
+ bt_shell_printf("No ChannelSounding1 interface for that device\n");
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#207: FILE: client/cs.c:1506:
+ ranging_data_mode_setup, ranging_data_mode_reply,
WARNING:LONG_LINE_STRING: line length of 91 exceeds 80 columns
#219: FILE: client/cs.c:1627:
+ { "ranging-data-mode", "<dev_addr> <disabled/realtime/ondemand> [notify/indicate]",
WARNING:LONG_LINE: line length of 85 exceeds 80 columns
#221: FILE: client/cs.c:1629:
+ "Set RAP Requester Ranging Data subscription mode" },
/home/runner/work/bluez/bluez/src/patch/14861165.patch total: 0 errors, 6 warnings, 94 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/14861165.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: BluezMake - FAIL
Desc: Build BlueZ
Output:
src/shared/rap.c:1256:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1256 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:414:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
414 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c: In function ‘send_ras_segment_data’:
src/shared/rap.c:2264:21: error: variable ‘ras’ set but not used [-Werror=unused-but-set-variable]
2264 | struct ras *ras;
| ^~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7947: src/shared/libshared_mainloop_la-rap.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4241: all] Error 2
##############################
Test: MakeCheck - FAIL
Desc: Run Bluez Make Check
Output:
src/shared/rap.c:1256:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1256 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:414:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
414 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c: In function ‘send_ras_segment_data’:
src/shared/rap.c:2264:21: error: variable ‘ras’ set but not used [-Werror=unused-but-set-variable]
2264 | struct ras *ras;
| ^~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7660: src/shared/libshared_glib_la-rap.lo] Error 1
make: *** [Makefile:10979: check] Error 2
##############################
Test: CheckValgrind - FAIL
Desc: Run Bluez Make Check with Valgrind
Output:
src/shared/rap.c:1256:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1256 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:414:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
414 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c: In function ‘send_ras_segment_data’:
src/shared/rap.c:2264:21: error: variable ‘ras’ set but not used [-Werror=unused-but-set-variable]
2264 | struct ras *ras;
| ^~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7947: src/shared/libshared_mainloop_la-rap.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:10979: 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: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:318: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/rap.c:1256:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1256 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:414:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
414 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c: In function ‘send_ras_segment_data’:
src/shared/rap.c:2264:21: error: variable ‘ras’ set but not used [-Werror=unused-but-set-variable]
2264 | struct ras *ras;
| ^~~
src/shared/crypto.c:271:21: warning: Variable length array is used.
src/shared/crypto.c:272:23: warning: Variable length array is used.
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7947: src/shared/libshared_mainloop_la-rap.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4241: all] Error 2
##############################
Test: bluezmakeextell - FAIL
Desc: Build Bluez with External ELL
Output:
src/shared/rap.c:1256:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1256 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:414:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
414 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c: In function ‘send_ras_segment_data’:
src/shared/rap.c:2264:21: error: variable ‘ras’ set but not used [-Werror=unused-but-set-variable]
2264 | struct ras *ras;
| ^~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7947: src/shared/libshared_mainloop_la-rap.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4241: 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:
src/shared/rap.c:1212:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1212 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:412:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
412 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:1212:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1212 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:412:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
412 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:1212:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1212 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:412:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
412 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7660: src/shared/libshared_glib_la-rap.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7947: src/shared/libshared_mainloop_la-rap.lo] Error 1
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7387: src/shared/libshared_ell_la-rap.lo] Error 1
make: *** [Makefile:4241: all] Error 2
[03/11] shared/rap: Honor CCCD transport for RAS on-demand ranging data
src/shared/rap.c:1212:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1212 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:412:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
412 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:1212:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1212 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:412:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
412 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:1212:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1212 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:412:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
412 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7660: src/shared/libshared_glib_la-rap.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7947: src/shared/libshared_mainloop_la-rap.lo] Error 1
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7387: src/shared/libshared_ell_la-rap.lo] Error 1
make: *** [Makefile:4241: 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:1303:2: warning: Use of memory after it is freed [unix.Malloc]
1303 | discovery_op_complete(op, success, att_ecode);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1368:2: warning: Use of memory after it is freed [unix.Malloc]
1368 | discovery_op_complete(op, success, att_ecode);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1643:6: warning: Use of memory after it is freed [unix.Malloc]
1643 | if (read_db_hash(op)) {
| ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:1648:2: warning: Use of memory after it is freed [unix.Malloc]
1648 | 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:2168:6: warning: Use of memory after it is freed [unix.Malloc]
2168 | if (read_db_hash(op)) {
| ^~~~~~~~~~~~~~~~
src/shared/gatt-client.c:2176:8: warning: Use of memory after it is freed [unix.Malloc]
2176 | discovery_op_ref(op),
| ^~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3362:2: warning: Use of memory after it is freed [unix.Malloc]
3362 | complete_write_long_op(req, success, 0, false);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3384:2: warning: Use of memory after it is freed [unix.Malloc]
3384 | request_unref(req);
| ^~~~~~~~~~~~~~~~~~
12 warnings generated.
src/shared/rap.c:1212:13: error: redundant redeclaration of ‘ras_ondemand_cache_clear’ [-Werror=redundant-decls]
1212 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
src/shared/rap.c:412:13: note: previous declaration of ‘ras_ondemand_cache_clear’ with type ‘void(struct bt_rap *)’
412 | static void ras_ondemand_cache_clear(struct bt_rap *rap);
| ^~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7947: src/shared/libshared_mainloop_la-rap.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
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.
make: *** [Makefile:4241: all] Error 2
https://github.com/bluez/bluez/pull/2617
---
Regards,
Linux Bluetooth
next prev parent reply other threads:[~2026-10-01 16:03 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-01 13:18 [PATCH 00/11] Ranging Profile and Ranging Service PTS fixes Cheng Jiang
2026-10-01 13:18 ` [PATCH 01/11] shared/gatt-client: Allow registering notify with explicit CCC value Cheng Jiang
2026-10-01 16:03 ` bluez.test.bot [this message]
2026-10-01 13:18 ` [PATCH 02/11] shared/gatt-db: Add gatt_db_attribute_indicate() Cheng Jiang
2026-10-01 15:48 ` Luiz Augusto von Dentz
2026-10-09 8:45 ` Cheng Jiang
2026-10-01 13:18 ` [PATCH 03/11] shared/rap: Honor CCCD transport for RAS on-demand ranging data Cheng Jiang
2026-10-01 13:18 ` [PATCH 04/11] shared/rap: Add RAS Control Point Abort Operation Cheng Jiang
2026-10-01 13:18 ` [PATCH 05/11] shared/rap: Shorten the requester Control Point response timeout Cheng Jiang
2026-10-01 13:18 ` [PATCH 06/11] shared/rap: Add real-time ranging data delivery on the responder Cheng Jiang
2026-10-01 13:18 ` [PATCH 07/11] shared/rap: Add requester ranging data subscription API Cheng Jiang
2026-10-01 13:18 ` [PATCH 08/11] profiles/ranging: Expose requester ranging data mode over D-Bus Cheng Jiang
2026-10-01 13:18 ` [PATCH 09/11] client: Add Channel Sounding ranging data mode control Cheng Jiang
2026-10-01 13:18 ` [PATCH 10/11] doc: Document " Cheng Jiang
2026-10-01 13:18 ` [PATCH 11/11] doc/qualification: Add RAP/RAS PTS qualification results Cheng Jiang
-- strict thread matches above, loose matches on Subject: below --
2026-10-09 9:25 [PATCH v2 01/14] shared/gatt-client: Allow registering notify with explicit CCC value Cheng Jiang
2026-10-09 15:42 ` Ranging Profile and Ranging Service PTS fixes 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=6abe8445.e5f0f813.32aaf1.3b9b@mx.google.com \
--to=bluez.test.bot@gmail.com \
--cc=cheng.jiang@oss.qualcomm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox