2024-02-12 12:08:02

by Shahid Vichhi

[permalink] [raw]
Subject: [PATCH v4] monitor/att: Enable the notification logging support for the CCP

---
monitor/att.c | 661 ++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 661 insertions(+)

diff --git a/monitor/att.c b/monitor/att.c
index e0164f3ddf3b..03e101d6a6b8 100644
--- a/monitor/att.c
+++ b/monitor/att.c
@@ -2433,6 +2433,645 @@ static void seeking_speed_notify(const struct l2cap_frame *frame)
print_seeking_speed(frame);
}

+static void print_bearer_name(const struct l2cap_frame *frame)
+{
+ char *name;
+
+ name = name2utf8((uint8_t *)frame->data, frame->size);
+
+ print_field(" Bearer Name: %s", name);
+}
+
+static void bearer_name_read(const struct l2cap_frame *frame)
+{
+ print_bearer_name(frame);
+}
+
+static void bearer_name_notify(const struct l2cap_frame *frame)
+{
+ print_bearer_name(frame);
+}
+
+static void bearer_uci_read(const struct l2cap_frame *frame)
+{
+ char *name;
+
+ name = name2utf8((uint8_t *)frame->data, frame->size);
+
+ print_field(" Bearer Uci Name: %s", name);
+}
+
+static void print_technology_name(const struct l2cap_frame *frame)
+{
+ int8_t tech_id;
+ const char *str;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&tech_id)) {
+ print_text(COLOR_ERROR, " Technology id:: invalid size");
+ goto done;
+ }
+
+ switch (tech_id) {
+ case 0x01:
+ str = "3G";
+ break;
+ case 0x02:
+ str = "4G";
+ break;
+ case 0x03:
+ str = "LTE";
+ break;
+ case 0x04:
+ str = "WiFi";
+ break;
+ case 0x05:
+ str = "5G";
+ break;
+ case 0x06:
+ str = "GSM";
+ break;
+ case 0x07:
+ str = "CDMA";
+ break;
+ case 0x08:
+ str = "2G";
+ break;
+ case 0x09:
+ str = "WCDMA";
+ break;
+ default:
+ str = "Reserved";
+ break;
+ }
+
+ print_field("Technology: %s (0x%2.2x)", str, tech_id);
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void bearer_technology_read(const struct l2cap_frame *frame)
+{
+ print_technology_name(frame);
+}
+
+static void bearer_technology_notify(const struct l2cap_frame *frame)
+{
+ print_technology_name(frame);
+}
+
+static void print_uri_scheme_list(const struct l2cap_frame *frame)
+{
+ char *name;
+
+ name = name2utf8((uint8_t *)frame->data, frame->size);
+
+ print_field(" Uri scheme Name: %s", name);
+}
+
+static void bearer_uri_schemes_list_read(const struct l2cap_frame *frame)
+{
+ print_uri_scheme_list(frame);
+}
+
+static void print_signal_strength(const struct l2cap_frame *frame)
+{
+ uint8_t signal_strength;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&signal_strength)) {
+ print_text(COLOR_ERROR, " signal_strength:: invalid size");
+ goto done;
+ }
+
+ print_field(" signal_strength: %x", signal_strength);
+
+ if (signal_strength == 0)
+ print_field(" No Service");
+ else if (signal_strength == 0x64)
+ print_field(" Maximum signal strength");
+ else if ((signal_strength > 0) && (signal_strength < 0x64))
+ print_field(" Implementation specific");
+ else if (signal_strength == 0xFF)
+ print_field(" Signal strength is unavailable");
+ else
+ print_field(" RFU");
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void bearer_signal_strength_read(const struct l2cap_frame *frame)
+{
+ print_signal_strength(frame);
+}
+
+static void bearer_signal_strength_notify(const struct l2cap_frame *frame)
+{
+ print_signal_strength(frame);
+}
+
+static void
+print_signal_strength_rep_intrvl(const struct l2cap_frame *frame)
+{
+ int8_t reporting_intrvl;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&reporting_intrvl)) {
+ print_text(COLOR_ERROR, "Reporting_interval:: invalid size");
+ goto done;
+ }
+
+ print_field(" Reporting_interval: 0x%x", reporting_intrvl);
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void
+bearer_signal_strength_rep_intrvl_read(const struct l2cap_frame *frame)
+{
+ print_signal_strength_rep_intrvl(frame);
+}
+
+static void
+bearer_signal_strength_rep_intrvl_write(const struct l2cap_frame *frame)
+{
+ print_signal_strength_rep_intrvl(frame);
+}
+
+static void print_call_list(const struct l2cap_frame *frame)
+{
+ uint8_t list_item_length;
+ uint8_t call_index;
+ uint8_t call_state;
+ uint8_t call_flag;
+ char *call_uri;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&list_item_length)) {
+ print_text(COLOR_ERROR, " list_item_length:: invalid size");
+ goto done;
+ }
+
+ print_field(" list_item_length: 0x%x", list_item_length);
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&call_index)) {
+ print_text(COLOR_ERROR, " call_index:: invalid size");
+ goto done;
+ }
+
+ print_field(" call_index: 0x%x", call_index);
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&call_state)) {
+ print_text(COLOR_ERROR, " call_state:: invalid size");
+ goto done;
+ }
+
+ print_field(" call_state: 0x%x", call_state);
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&call_flag)) {
+ print_text(COLOR_ERROR, " call_flag:: invalid size");
+ goto done;
+ }
+
+ print_field(" call_flag: 0x%x", call_flag);
+
+ call_uri = name2utf8((uint8_t *)frame->data, frame->size);
+
+ print_field(" call_uri: %s", call_uri);
+
+done:
+ if (frame->size)
+ print_hex_field(" call_list Data", frame->data, frame->size);
+}
+
+static void bearer_current_call_list_read(const struct l2cap_frame *frame)
+{
+ print_call_list(frame);
+}
+
+static void bearer_current_call_list_notify(const struct l2cap_frame *frame)
+{
+ print_call_list(frame);
+}
+
+static void print_ccid(const struct l2cap_frame *frame)
+{
+ int8_t ccid;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&ccid)) {
+ print_text(COLOR_ERROR, " ccid:: invalid size");
+ goto done;
+ }
+
+ print_field(" ccid: %x", ccid);
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void call_content_control_id_read(const struct l2cap_frame *frame)
+{
+ print_ccid(frame);
+}
+
+static void print_status_flag(const struct l2cap_frame *frame)
+{
+ int16_t flag;
+
+ if (!l2cap_frame_get_le16((void *)frame, (uint16_t *)&flag)) {
+ print_text(COLOR_ERROR, " status flag:: invalid size");
+ goto done;
+ }
+
+ print_field(" status flag:");
+
+ if (flag & 0x1)
+ print_field(" Inband Ringtone Enabled:");
+ else
+ print_field(" Inband Ringtone Disabled:");
+
+ if (flag & 0x2)
+ print_field(" Server in silent Mode");
+ else
+ print_field(" Server Not in silent Mode");
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void status_flag_read(const struct l2cap_frame *frame)
+{
+ print_status_flag(frame);
+}
+
+static void status_flag_notify(const struct l2cap_frame *frame)
+{
+ print_status_flag(frame);
+}
+
+static void print_target_uri(const struct l2cap_frame *frame)
+{
+ char *name;
+ uint8_t call_idx;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&call_idx)) {
+ print_text(COLOR_ERROR, " call_idx:: invalid size");
+ goto done;
+ }
+
+ print_field(" call_idx: %x", call_idx);
+
+ name = name2utf8((uint8_t *)frame->data, frame->size);
+
+ print_field(" Uri: %s", name);
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void incom_target_bearer_uri_read(const struct l2cap_frame *frame)
+{
+ print_target_uri(frame);
+}
+
+static void incom_target_bearer_uri_notify(const struct l2cap_frame *frame)
+{
+ print_target_uri(frame);
+}
+
+static void print_call_state(const struct l2cap_frame *frame)
+{
+ uint8_t call_Index;
+ uint8_t call_state;
+ uint8_t call_flag;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&call_Index)) {
+ print_text(COLOR_ERROR, " call_Index:: invalid index");
+ goto done;
+ }
+
+ print_field(" call_Index: 0x%2.2x", call_Index);
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&call_state)) {
+ print_text(COLOR_ERROR, " call_state:: invalid state");
+ goto done;
+ }
+
+ print_field(" call_state: 0x%2.2x", call_state);
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&call_flag)) {
+ print_text(COLOR_ERROR, " call_flag:: invalid flag");
+ goto done;
+ }
+
+ print_field(" call_flag: 0x%2.2x", call_flag);
+
+done:
+ if (frame->size)
+ print_hex_field(" call_state Data", frame->data, frame->size);
+}
+
+static void call_state_read(const struct l2cap_frame *frame)
+{
+ print_call_state(frame);
+}
+
+static void call_state_notify(const struct l2cap_frame *frame)
+{
+ print_call_state(frame);
+}
+
+static void print_call_cp(const struct l2cap_frame *frame)
+{
+ uint8_t opcode;
+ uint8_t parameter;
+ const char *str;
+ char *name;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&opcode)) {
+ print_text(COLOR_ERROR, " opcode:: invalid size");
+ goto done;
+ }
+
+ print_field(" opcode: 0x%2.2x", opcode);
+
+ switch (opcode) {
+ case 0x00:
+ str = "Accept";
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&parameter)) {
+ print_text(COLOR_ERROR, " parameter:: invalid size");
+ goto done;
+ }
+ print_field(" Operation: %s (0x%2.2x)", str, parameter);
+ break;
+ case 0x01:
+ str = "Terminate";
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&parameter)) {
+ print_text(COLOR_ERROR, " parameter:: invalid size");
+ goto done;
+ }
+ print_field(" Operation: %s (0x%2.2x)", str, parameter);
+ break;
+ case 0x02:
+ str = "Local Hold";
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&parameter)) {
+ print_text(COLOR_ERROR, " parameter:: invalid size");
+ goto done;
+ }
+ print_field(" Operation: %s (0x%2.2x)", str, parameter);
+ break;
+ case 0x03:
+ str = "Local Retrieve";
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&parameter)) {
+ print_text(COLOR_ERROR, " parameter:: invalid size");
+ goto done;
+ }
+ print_field(" Operation: %s (0x%2.2x)", str, parameter);
+ break;
+ case 0x04:
+ str = "Originate";
+ name = name2utf8((uint8_t *)frame->data, frame->size);
+ print_field(" Operation: %s Uri: %s", str, name);
+ break;
+ case 0x05:
+ str = "Join";
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&parameter)) {
+ print_text(COLOR_ERROR, " parameter:: invalid size");
+ goto done;
+ }
+ print_field(" Operation: %s (0x%2.2x)", str, parameter);
+ break;
+ default:
+ str = "RFU";
+ print_field(" Operation: %s", str);
+ break;
+ }
+
+done:
+ if (frame->size)
+ print_hex_field("call_cp Data", frame->data, frame->size);
+}
+
+static void print_call_cp_notification(const struct l2cap_frame *frame)
+{
+ uint8_t opcode;
+ uint8_t result_code;
+ const char *str;
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&opcode)) {
+ print_text(COLOR_ERROR, " result_code:: invalid opcode");
+ goto done;
+ }
+
+ print_field(" opcode: 0x%2.2x", opcode);
+
+ if (!l2cap_frame_get_u8((void *)frame, (uint8_t *)&result_code)) {
+ print_text(COLOR_ERROR, " result_code:: invalid result_code");
+ goto done;
+ }
+
+ print_field(" result_code: 0x%2.2x", result_code);
+
+ switch (result_code) {
+ case 0x00:
+ str = "SUCCESS";
+ break;
+ case 0x01:
+ str = "OPCODE NOT SUPPORTED";
+ break;
+ case 0x02:
+ str = "OPERATION NOT POSSIBLE";
+ break;
+ case 0x03:
+ str = "INVALID CALL INDEX";
+ break;
+ case 0x04:
+ str = "STATE MISMATCH";
+ break;
+ case 0x05:
+ str = "LACK OF RESOURCES";
+ break;
+ case 0x06:
+ str = "INVALID OUTGOING URI";
+ break;
+ default:
+ str = "RFU";
+ break;
+ }
+
+ print_field(" Status: %s", str);
+
+done:
+ if (frame->size)
+ print_hex_field(" call_cp Data", frame->data, frame->size);
+}
+
+static void call_cp_write(const struct l2cap_frame *frame)
+{
+ print_call_cp(frame);
+}
+
+static void call_cp_notify(const struct l2cap_frame *frame)
+{
+ print_call_cp_notification(frame);
+}
+
+static void print_call_cp_opt(const struct l2cap_frame *frame)
+{
+ uint16_t operation;
+
+ if (!l2cap_frame_get_le16((void *)frame, (uint16_t *)&operation)) {
+ print_text(COLOR_ERROR, " status operation:: invalid size");
+ goto done;
+ }
+
+ print_field(" operation: 0x%2x", operation);
+
+ if (operation & 0x1) {
+ print_field(" Local Hold and Local Retrieve "
+ "Call Control Point Opcodes supported");
+ } else {
+ print_field(" Local Hold and Local Retrieve "
+ "Call Control Point Opcodes not supported");
+ }
+
+ if (operation & 0x2)
+ print_field(" Join Call Control Point Opcode supported");
+ else
+ print_field(" Join Call Control Point Opcode not supported");
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void call_cp_oipt_opcodes_read(const struct l2cap_frame *frame)
+{
+ print_call_cp_opt(frame);
+}
+
+static void print_term_reason(const struct l2cap_frame *frame)
+{
+ uint8_t call_id, reason;
+
+ if (!l2cap_frame_get_u8((void *)frame, &call_id)) {
+ print_text(COLOR_ERROR, "Call Index: invalid size");
+ goto done;
+ }
+ print_field(" call Index: %u", call_id);
+
+ if (!l2cap_frame_get_u8((void *)frame, &reason)) {
+ print_text(COLOR_ERROR, "Reason: invalid size");
+ goto done;
+ }
+
+ print_field(" Reason:");
+
+ switch (reason) {
+ case 0x00:
+ print_field(" Improper URI");
+ break;
+ case 0x01:
+ print_field(" Call Failed");
+ break;
+ case 0x02:
+ print_field(" Remote party ended the call");
+ break;
+ case 0x03:
+ print_field(" Server ended the call");
+ break;
+ case 0x04:
+ print_field(" Line was Busy");
+ break;
+ case 0x05:
+ print_field(" Network Congestion");
+ break;
+ case 0x06:
+ print_field(" Client terminated the call");
+ break;
+ case 0x07:
+ print_field(" No service");
+ break;
+ case 0x08:
+ print_field(" No answer");
+ break;
+ case 0x09:
+ print_field(" Unspecified");
+ break;
+ default:
+ print_field(" RFU");
+ break;
+ }
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void call_termination_reason_notify(const struct l2cap_frame *frame)
+{
+ print_term_reason(frame);
+}
+
+static void print_incom_call(const struct l2cap_frame *frame)
+{
+ char *name;
+ uint8_t call_id;
+
+ if (!l2cap_frame_get_u8((void *)frame, &call_id)) {
+ print_text(COLOR_ERROR, "Call Index: invalid size");
+ goto done;
+ }
+
+ print_field(" Call Index: %u", call_id);
+
+ name = name2utf8((uint8_t *)frame->data, frame->size);
+
+ print_field(" call_string: %s", name);
+
+done:
+ if (frame->size)
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void incoming_call_read(const struct l2cap_frame *frame)
+{
+ print_incom_call(frame);
+}
+
+static void incoming_call_notify(const struct l2cap_frame *frame)
+{
+ print_incom_call(frame);
+}
+
+static void print_call_friendly_name(const struct l2cap_frame *frame)
+{
+ char *name;
+ uint8_t call_id;
+
+ if (!l2cap_frame_get_u8((void *)frame, &call_id)) {
+ print_text(COLOR_ERROR, "Call Index: invalid size");
+ goto done;
+ }
+
+ print_hex_field(" Data", frame->data, frame->size);
+}
+
+static void call_friendly_name_read(const struct l2cap_frame *frame)
+{
+ print_call_friendly_name(frame);
+}
+
+static void call_friendly_name_notify(const struct l2cap_frame *frame)
+{
+ print_call_friendly_name(frame);
+}
+
static const char *play_order_str(uint8_t order)
{
switch (order) {
@@ -3392,6 +4031,28 @@ static const struct gatt_handler {
GATT_HANDLER(0x2bc7, NULL, bcast_audio_scan_cp_write, NULL),
GATT_HANDLER(0x2bc8, bcast_recv_state_read, NULL,
bcast_recv_state_notify),
+ GATT_HANDLER(0x2bb3, bearer_name_read, NULL, bearer_name_notify),
+ GATT_HANDLER(0x2bb4, bearer_uci_read, NULL, NULL),
+ GATT_HANDLER(0x2bb5, bearer_technology_read, NULL,
+ bearer_technology_notify),
+ GATT_HANDLER(0x2bb6, bearer_uri_schemes_list_read, NULL, NULL),
+ GATT_HANDLER(0x2bb7, bearer_signal_strength_read, NULL,
+ bearer_signal_strength_notify),
+ GATT_HANDLER(0x2bb8, bearer_signal_strength_rep_intrvl_read,
+ bearer_signal_strength_rep_intrvl_write, NULL),
+ GATT_HANDLER(0x2bb9, bearer_current_call_list_read, NULL,
+ bearer_current_call_list_notify),
+ GATT_HANDLER(0x2bba, call_content_control_id_read, NULL, NULL),
+ GATT_HANDLER(0x2bbb, status_flag_read, NULL, status_flag_notify),
+ GATT_HANDLER(0x2bbc, incom_target_bearer_uri_read, NULL,
+ incom_target_bearer_uri_notify),
+ GATT_HANDLER(0x2bbd, call_state_read, NULL, call_state_notify),
+ GATT_HANDLER(0x2bbe, NULL, call_cp_write, call_cp_notify),
+ GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
+ GATT_HANDLER(0x2bc0, NULL, NULL, call_termination_reason_notify),
+ GATT_HANDLER(0x2bc1, incoming_call_read, NULL, incoming_call_notify),
+ GATT_HANDLER(0x2bc2, call_friendly_name_read, NULL,
+ call_friendly_name_notify),
GMAS
};

--
2.34.1



2024-02-12 12:58:43

by bluez.test.bot

[permalink] [raw]
Subject: RE: [v4] monitor/att: Enable the notification logging support for the CCP

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=825189

---Test result---

Test Summary:
CheckPatch PASS 0.80 seconds
GitLint PASS 0.33 seconds
BuildEll PASS 24.18 seconds
BluezMake FAIL 146.62 seconds
MakeCheck FAIL 1.29 seconds
MakeDistcheck FAIL 89.09 seconds
CheckValgrind FAIL 108.75 seconds
CheckSmatch FAIL 254.45 seconds
bluezmakeextell FAIL 82.19 seconds
IncrementalBuild FAIL 587.25 seconds
ScanBuild FAIL 796.96 seconds

Details
##############################
Test: BluezMake - FAIL
Desc: Build BlueZ
Output:

tools/mgmt-tester.c: In function ‘main’:
tools/mgmt-tester.c:12723:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
12723 | int main(int argc, char *argv[])
| ^~~~
monitor/att.c: In function ‘print_call_friendly_name’:
monitor/att.c:3059:3: error: label ‘done’ used but not defined
3059 | goto done;
| ^~~~
monitor/att.c:3054:8: error: unused variable ‘name’ [-Werror=unused-variable]
3054 | char *name;
| ^~~~
monitor/att.c: At top level:
monitor/att.c:4051:23: error: ‘call_cp_opt_opcodes_read’ undeclared here (not in a function); did you mean ‘call_cp_oipt_opcodes_read’?
4051 | GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
| ^~~~~~~~~~~~~~~~~~~~~~~~
monitor/att.c:3980:10: note: in definition of macro ‘GATT_HANDLER’
3980 | .read = _read, \
| ^~~~~
monitor/att.c:2953:13: error: ‘call_cp_oipt_opcodes_read’ defined but not used [-Werror=unused-function]
2953 | static void call_cp_oipt_opcodes_read(const struct l2cap_frame *frame)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7773: monitor/att.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4630: all] Error 2
##############################
Test: MakeCheck - FAIL
Desc: Run Bluez Make Check
Output:

monitor/att.c: In function ‘print_call_friendly_name’:
monitor/att.c:3059:3: error: label ‘done’ used but not defined
3059 | goto done;
| ^~~~
monitor/att.c:3054:8: error: unused variable ‘name’ [-Werror=unused-variable]
3054 | char *name;
| ^~~~
monitor/att.c: At top level:
monitor/att.c:4051:23: error: ‘call_cp_opt_opcodes_read’ undeclared here (not in a function); did you mean ‘call_cp_oipt_opcodes_read’?
4051 | GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
| ^~~~~~~~~~~~~~~~~~~~~~~~
monitor/att.c:3980:10: note: in definition of macro ‘GATT_HANDLER’
3980 | .read = _read, \
| ^~~~~
monitor/att.c:2953:13: error: ‘call_cp_oipt_opcodes_read’ defined but not used [-Werror=unused-function]
2953 | static void call_cp_oipt_opcodes_read(const struct l2cap_frame *frame)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7773: monitor/att.o] Error 1
make: *** [Makefile:12115: check] Error 2
##############################
Test: MakeDistcheck - FAIL
Desc: Run Bluez Make Distcheck
Output:

../../monitor/att.c: In function ‘print_call_friendly_name’:
../../monitor/att.c:3059:3: error: label ‘done’ used but not defined
3059 | goto done;
| ^~~~
../../monitor/att.c: At top level:
../../monitor/att.c:4051:23: error: ‘call_cp_opt_opcodes_read’ undeclared here (not in a function); did you mean ‘call_cp_oipt_opcodes_read’?
4051 | GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
| ^~~~~~~~~~~~~~~~~~~~~~~~
../../monitor/att.c:3980:10: note: in definition of macro ‘GATT_HANDLER’
3980 | .read = _read, \
| ^~~~~
make[2]: *** [Makefile:7773: monitor/att.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [Makefile:4630: all] Error 2
make: *** [Makefile:12036: distcheck] Error 1
##############################
Test: CheckValgrind - FAIL
Desc: Run Bluez Make Check with Valgrind
Output:

tools/mgmt-tester.c: In function ‘main’:
tools/mgmt-tester.c:12723:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
12723 | int main(int argc, char *argv[])
| ^~~~
monitor/att.c: In function ‘print_call_friendly_name’:
monitor/att.c:3059:3: error: label ‘done’ used but not defined
3059 | goto done;
| ^~~~
monitor/att.c:3054:8: error: unused variable ‘name’ [-Werror=unused-variable]
3054 | char *name;
| ^~~~
monitor/att.c: At top level:
monitor/att.c:4051:23: error: ‘call_cp_opt_opcodes_read’ undeclared here (not in a function); did you mean ‘call_cp_oipt_opcodes_read’?
4051 | GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
| ^~~~~~~~~~~~~~~~~~~~~~~~
monitor/att.c:3980:10: note: in definition of macro ‘GATT_HANDLER’
3980 | .read = _read, \
| ^~~~~
monitor/att.c:2953:13: error: ‘call_cp_oipt_opcodes_read’ defined but not used [-Werror=unused-function]
2953 | static void call_cp_oipt_opcodes_read(const struct l2cap_frame *frame)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7773: monitor/att.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:12115: 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/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'
tools/mesh-cfgtest.c:1453:17: warning: unknown escape sequence: '\%'
tools/sco-tester.c: note: in included file:
./lib/bluetooth.h:216:15: warning: array of flexible structures
./lib/bluetooth.h:221:31: warning: array of flexible structures
tools/bneptest.c:634:39: warning: unknown escape sequence: '\%'
tools/seq2bseq.c:57:26: warning: Variable length array is used.
tools/obex-client-tool.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'
android/avctp.c:505:34: warning: Variable length array is used.
android/avctp.c:556:34: warning: Variable length array is used.
unit/test-avrcp.c:373:26: warning: Variable length array is used.
unit/test-avrcp.c:398:26: warning: Variable length array is used.
unit/test-avrcp.c:414:24: warning: Variable length array is used.
android/avrcp-lib.c:1085:34: warning: Variable length array is used.
android/avrcp-lib.c:1583:34: warning: Variable length array is used.
android/avrcp-lib.c:1612:34: warning: Variable length array is used.
android/avrcp-lib.c:1638:34: warning: Variable length array is used.
mesh/mesh-io-mgmt.c:523:67: warning: Variable length array is used.
client/display.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/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'
monitor/packet.c: note: in included file:
monitor/display.h:82:26: warning: Variable length array is used.
monitor/packet.c:1867:26: warning: Variable length array is used.
monitor/packet.c: note: in included file:
monitor/bt.h:3606:52: warning: array of flexible structures
monitor/bt.h:3594:40: warning: array of flexible structures
monitor/l2cap.c: note: in included file:
monitor/display.h:82:26: warning: Variable length array is used.
monitor/msft.c: note: in included file:
monitor/msft.h:88:44: warning: array of flexible structures
monitor/att.c: note: in included file:
monitor/display.h:82:26: warning: Variable length array is used.
tools/rctest.c:624:33: warning: non-ANSI function declaration of function 'automated_send_recv'
tools/hex2hcd.c:135:26: warning: Variable length array is used.
tools/meshctl.c:324:33: warning: non-ANSI function declaration of function 'forget_mesh_devices'
monitor/att.c: In function ‘print_call_friendly_name’:
monitor/att.c:3059:3: error: label ‘done’ used but not defined
3059 | goto done;
| ^~~~
monitor/att.c:3054:8: error: unused variable ‘name’ [-Werror=unused-variable]
3054 | char *name;
| ^~~~
monitor/att.c: At top level:
monitor/att.c:4051:23: error: ‘call_cp_opt_opcodes_read’ undeclared here (not in a function); did you mean ‘call_cp_oipt_opcodes_read’?
4051 | GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
| ^~~~~~~~~~~~~~~~~~~~~~~~
monitor/att.c:3980:10: note: in definition of macro ‘GATT_HANDLER’
3980 | .read = _read, \
| ^~~~~
monitor/att.c:2953:13: error: ‘call_cp_oipt_opcodes_read’ defined but not used [-Werror=unused-function]
2953 | static void call_cp_oipt_opcodes_read(const struct l2cap_frame *frame)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7773: monitor/att.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4630: all] Error 2
##############################
Test: bluezmakeextell - FAIL
Desc: Build Bluez with External ELL
Output:

monitor/att.c: In function ‘print_call_friendly_name’:
monitor/att.c:3059:3: error: label ‘done’ used but not defined
3059 | goto done;
| ^~~~
monitor/att.c:3054:8: error: unused variable ‘name’ [-Werror=unused-variable]
3054 | char *name;
| ^~~~
monitor/att.c: At top level:
monitor/att.c:4051:23: error: ‘call_cp_opt_opcodes_read’ undeclared here (not in a function); did you mean ‘call_cp_oipt_opcodes_read’?
4051 | GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
| ^~~~~~~~~~~~~~~~~~~~~~~~
monitor/att.c:3980:10: note: in definition of macro ‘GATT_HANDLER’
3980 | .read = _read, \
| ^~~~~
monitor/att.c:2953:13: error: ‘call_cp_oipt_opcodes_read’ defined but not used [-Werror=unused-function]
2953 | static void call_cp_oipt_opcodes_read(const struct l2cap_frame *frame)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7773: monitor/att.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4630: all] Error 2
##############################
Test: IncrementalBuild - FAIL
Desc: Incremental build with the patches in the series
Output:
[v4] monitor/att: Enable the notification logging support for the CCP

tools/mgmt-tester.c: In function ‘main’:
tools/mgmt-tester.c:12723:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
12723 | int main(int argc, char *argv[])
| ^~~~
unit/test-avdtp.c: In function ‘main’:
unit/test-avdtp.c:766:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
766 | int main(int argc, char *argv[])
| ^~~~
unit/test-avrcp.c: In function ‘main’:
unit/test-avrcp.c:989:5: note: variable tracking size limit exceeded with ‘-fvar-tracking-assignments’, retrying without
989 | int main(int argc, char *argv[])
| ^~~~
monitor/att.c: In function ‘print_call_friendly_name’:
monitor/att.c:3059:3: error: label ‘done’ used but not defined
3059 | goto done;
| ^~~~
monitor/att.c:3054:8: error: unused variable ‘name’ [-Werror=unused-variable]
3054 | char *name;
| ^~~~
monitor/att.c: At top level:
monitor/att.c:4051:23: error: ‘call_cp_opt_opcodes_read’ undeclared here (not in a function); did you mean ‘call_cp_oipt_opcodes_read’?
4051 | GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
| ^~~~~~~~~~~~~~~~~~~~~~~~
monitor/att.c:3980:10: note: in definition of macro ‘GATT_HANDLER’
3980 | .read = _read, \
| ^~~~~
monitor/att.c:2953:13: error: ‘call_cp_oipt_opcodes_read’ defined but not used [-Werror=unused-function]
2953 | static void call_cp_oipt_opcodes_read(const struct l2cap_frame *frame)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7773: monitor/att.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4630: all] Error 2
##############################
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: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:3237:2: warning: Use of memory after it is freed
complete_write_long_op(req, success, 0, false);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3259:2: warning: Use of memory after it is freed
request_unref(req);
^~~~~~~~~~~~~~~~~~
12 warnings generated.
src/shared/shell.c:1228:13: warning: Access to field 'options' results in a dereference of a null pointer (loaded from variable 'opt')
if (c != opt->options[index - offset].val) {
^~~~~~~~~~~~
1 warning generated.
src/shared/bap.c:1132:2: warning: Use of memory after it is freed
DBG(bap, "stream %p", stream);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:40:2: note: expanded from macro 'DBG'
bap_debug(_bap, "%s:%s() " fmt, __FILE__, __func__, ## arg)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:1145:2: warning: Use of memory after it is freed
DBG(stream->bap, "stream %p", stream);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:40:2: note: expanded from macro 'DBG'
bap_debug(_bap, "%s:%s() " fmt, __FILE__, __func__, ## arg)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:1240:33: warning: Use of memory after it is freed
for (entry = queue_get_entries(bap->state_cbs); entry;
^~~~~~~~~~~~~~
src/shared/bap.c:1698:3: warning: Use of memory after it is freed
stream_set_state(stream, BT_BAP_STREAM_STATE_CONFIG);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
4 warnings 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:3237:2: warning: Use of memory after it is freed
complete_write_long_op(req, success, 0, false);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3259:2: warning: Use of memory after it is freed
request_unref(req);
^~~~~~~~~~~~~~~~~~
12 warnings generated.
src/shared/shell.c:1228:13: warning: Access to field 'options' results in a dereference of a null pointer (loaded from variable 'opt')
if (c != opt->options[index - offset].val) {
^~~~~~~~~~~~
1 warning generated.
src/shared/bap.c:1132:2: warning: Use of memory after it is freed
DBG(bap, "stream %p", stream);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:40:2: note: expanded from macro 'DBG'
bap_debug(_bap, "%s:%s() " fmt, __FILE__, __func__, ## arg)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:1145:2: warning: Use of memory after it is freed
DBG(stream->bap, "stream %p", stream);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:40:2: note: expanded from macro 'DBG'
bap_debug(_bap, "%s:%s() " fmt, __FILE__, __func__, ## arg)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:1240:33: warning: Use of memory after it is freed
for (entry = queue_get_entries(bap->state_cbs); entry;
^~~~~~~~~~~~~~
src/shared/bap.c:1698:3: warning: Use of memory after it is freed
stream_set_state(stream, BT_BAP_STREAM_STATE_CONFIG);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
4 warnings generated.
tools/hciattach.c:816:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n'
if ((n = read_hci_event(fd, resp, 10)) < 0) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/hciattach.c:864:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n'
if ((n = read_hci_event(fd, resp, 4)) < 0) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/hciattach.c:886:8: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n'
if ((n = read_hci_event(fd, resp, 10)) < 0) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/hciattach.c:908:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n'
if ((n = read_hci_event(fd, resp, 4)) < 0) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/hciattach.c:929:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n'
if ((n = read_hci_event(fd, resp, 4)) < 0) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/hciattach.c:973:7: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n'
if ((n = read_hci_event(fd, resp, 6)) < 0) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
6 warnings generated.
src/oui.c:50:2: warning: Value stored to 'hwdb' is never read
hwdb = udev_hwdb_unref(hwdb);
^ ~~~~~~~~~~~~~~~~~~~~~
src/oui.c:53:2: warning: Value stored to 'udev' is never read
udev = udev_unref(udev);
^ ~~~~~~~~~~~~~~~~
2 warnings generated.
tools/hcidump.c:180:9: warning: Potential leak of memory pointed to by 'dp'
if (fds[i].fd == sock)
^~~
tools/hcidump.c:248:17: warning: Assigned value is garbage or undefined
dh->ts_sec = htobl(frm.ts.tv_sec);
^ ~~~~~~~~~~~~~~~~~~~~
tools/hcidump.c:326:9: warning: 1st function call argument is an uninitialized value
if (be32toh(dp.flags) & 0x02) {
^~~~~~~~~~~~~~~~~
/usr/include/endian.h:46:22: note: expanded from macro 'be32toh'
# define be32toh(x) __bswap_32 (x)
^~~~~~~~~~~~~~
tools/hcidump.c:341:20: warning: 1st function call argument is an uninitialized value
frm.data_len = be32toh(dp.len);
^~~~~~~~~~~~~~~
/usr/include/endian.h:46:22: note: expanded from macro 'be32toh'
# define be32toh(x) __bswap_32 (x)
^~~~~~~~~~~~~~
tools/hcidump.c:346:14: warning: 1st function call argument is an uninitialized value
opcode = be32toh(dp.flags) & 0xffff;
^~~~~~~~~~~~~~~~~
/usr/include/endian.h:46:22: note: expanded from macro 'be32toh'
# define be32toh(x) __bswap_32 (x)
^~~~~~~~~~~~~~
tools/hcidump.c:384:17: warning: Assigned value is garbage or undefined
frm.data_len = btohs(dh.len);
^ ~~~~~~~~~~~~~
tools/hcidump.c:394:11: warning: Assigned value is garbage or undefined
frm.len = frm.data_len;
^ ~~~~~~~~~~~~
tools/hcidump.c:398:9: warning: 1st function call argument is an uninitialized value
ts = be64toh(ph.ts);
^~~~~~~~~~~~~~
/usr/include/endian.h:51:22: note: expanded from macro 'be64toh'
# define be64toh(x) __bswap_64 (x)
^~~~~~~~~~~~~~
tools/hcidump.c:403:13: warning: 1st function call argument is an uninitialized value
frm.in = be32toh(dp.flags) & 0x01;
^~~~~~~~~~~~~~~~~
/usr/include/endian.h:46:22: note: expanded from macro 'be32toh'
# define be32toh(x) __bswap_32 (x)
^~~~~~~~~~~~~~
tools/hcidump.c:408:11: warning: Assigned value is garbage or undefined
frm.in = dh.in;
^ ~~~~~
tools/hcidump.c:437:7: warning: Null pointer passed to 1st parameter expecting 'nonnull'
fd = open(file, open_flags, 0644);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
11 warnings generated.
tools/rfcomm.c:228:3: warning: Value stored to 'i' is never read
i = execvp(cmdargv[0], cmdargv);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/rfcomm.c:228:7: warning: Null pointer passed to 1st parameter expecting 'nonnull'
i = execvp(cmdargv[0], cmdargv);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/rfcomm.c:348:8: warning: Although the value stored to 'fd' is used in the enclosing expression, the value is never actually read from 'fd'
if ((fd = open(devname, O_RDONLY | O_NOCTTY)) < 0) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/rfcomm.c:491:14: warning: Assigned value is garbage or undefined
req.channel = raddr.rc_channel;
^ ~~~~~~~~~~~~~~~~
tools/rfcomm.c:509:8: warning: Although the value stored to 'fd' is used in the enclosing expression, the value is never actually read from 'fd'
if ((fd = open(devname, O_RDONLY | O_NOCTTY)) < 0) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
5 warnings generated.
src/sdp-xml.c:126:10: warning: Assigned value is garbage or undefined
buf[1] = data[i + 1];
^ ~~~~~~~~~~~
src/sdp-xml.c:300:11: warning: Assigned value is garbage or undefined
buf[1] = data[i + 1];
^ ~~~~~~~~~~~
src/sdp-xml.c:338:11: warning: Assigned value is garbage or undefined
buf[1] = data[i + 1];
^ ~~~~~~~~~~~
3 warnings generated.
tools/ciptool.c:350:7: warning: 5th function call argument is an uninitialized value
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'
uint32_t *value_int = malloc(sizeof(int));
~~~~~~~~~~ ^~~~~~ ~~~~~~~~~~~
tools/sdptool.c:980:4: warning: 1st function call argument is an uninitialized value
free(allocArray[i]);
^~~~~~~~~~~~~~~~~~~
tools/sdptool.c:3777:2: warning: Potential leak of memory pointed to by 'si.name'
return add_service(0, &si);
^~~~~~~~~~~~~~~~~~~~~~~~~~
tools/sdptool.c:4112:4: warning: Potential leak of memory pointed to by 'context.svc'
return -1;
^~~~~~~~~
4 warnings generated.
tools/avtest.c:225:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:235:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 4);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:244:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:258:5: warning: Value stored to 'len' is never read
len = write(sk, buf,
^ ~~~~~~~~~~~~~~
tools/avtest.c:265:5: warning: Value stored to 'len' is never read
len = write(sk, buf,
^ ~~~~~~~~~~~~~~
tools/avtest.c:272:5: warning: Value stored to 'len' is never read
len = write(sk, buf,
^ ~~~~~~~~~~~~~~
tools/avtest.c:279:5: warning: Value stored to 'len' is never read
len = write(sk, buf,
^ ~~~~~~~~~~~~~~
tools/avtest.c:291:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 4);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:295:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:304:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:308:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:317:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:324:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:346:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 4);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:350:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:359:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:363:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:376:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 4);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:380:5: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:387:4: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:397:4: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:562:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:570:3: warning: Value stored to 'len' is never read
len = write(sk, buf, invalid ? 2 : 3);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/avtest.c:584:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 4 + sizeof(media_transport));
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/avtest.c:597:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:607:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:619:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:634:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:646:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:655:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 3);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:662:3: warning: Value stored to 'len' is never read
len = write(sk, buf, 2);
^ ~~~~~~~~~~~~~~~~~
tools/avtest.c:698:2: warning: Value stored to 'len' is never read
len = write(sk, buf, AVCTP_HEADER_LENGTH + sizeof(play_pressed));
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
32 warnings generated.
tools/btproxy.c:836:15: warning: Null pointer passed to 1st parameter expecting 'nonnull'
tcp_port = atoi(optarg);
^~~~~~~~~~~~
tools/btproxy.c:839:8: warning: Null pointer passed to 1st parameter expecting 'nonnull'
if (strlen(optarg) > 3 && !strncmp(optarg, "hci", 3))
^~~~~~~~~~~~~~
2 warnings generated.
tools/create-image.c:76:3: warning: Value stored to 'fd' is never read
fd = -1;
^ ~~
tools/create-image.c:84:3: warning: Value stored to 'fd' is never read
fd = -1;
^ ~~
tools/create-image.c:92:3: warning: Value stored to 'fd' is never read
fd = -1;
^ ~~
tools/create-image.c:105:2: warning: Value stored to 'fd' is never read
fd = -1;
^ ~~
4 warnings generated.
tools/btgatt-client.c:1824:2: warning: Value stored to 'argv' is never read
argv += optind;
^ ~~~~~~
1 warning generated.
tools/btgatt-server.c:1212:2: warning: Value stored to 'argv' is never read
argv -= optind;
^ ~~~~~~
1 warning generated.
tools/check-selftest.c:42:3: warning: Value stored to 'ptr' is never read
ptr = fgets(result, sizeof(result), fp);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
tools/gatt-service.c:294:2: warning: 2nd function call argument is an uninitialized value
chr_write(chr, value, len);
^~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
tools/obex-server-tool.c:133:13: warning: Null pointer passed to 1st parameter expecting 'nonnull'
data->fd = open(name, O_WRONLY | O_CREAT | O_NOCTTY, 0600);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tools/obex-server-tool.c:192:13: warning: Null pointer passed to 1st parameter expecting 'nonnull'
data->fd = open(name, O_RDONLY | O_NOCTTY, 0);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 warnings generated.
tools/test-runner.c:945:2: warning: 2nd function call argument is an uninitialized value
printf("Running command %s\n", cmdname ? cmdname : argv[0]);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
tools/btpclientctl.c:402:3: warning: Value stored to 'bit' is never read
bit = 0;
^ ~
tools/btpclientctl.c:1655:2: warning: Null pointer passed to 2nd parameter expecting 'nonnull'
memcpy(cp->data, ad_data, ad_len);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 warnings generated.
src/sdpd-request.c:211:13: warning: Result of 'malloc' is converted to a pointer of type 'char', which is incompatible with sizeof operand type 'uint16_t'
pElem = malloc(sizeof(uint16_t));
^~~~~~ ~~~~~~~~~~~~~~~~
src/sdpd-request.c:239:13: warning: Result of 'malloc' is converted to a pointer of type 'char', which is incompatible with sizeof operand type 'uint32_t'
pElem = malloc(sizeof(uint32_t));
^~~~~~ ~~~~~~~~~~~~~~~~
2 warnings generated.
android/avrcp-lib.c:1968:3: warning: 1st function call argument is an uninitialized value
g_free(text[i]);
^~~~~~~~~~~~~~~
1 warning generated.
profiles/health/hdp.c:644:3: warning: Use of memory after it is freed
hdp_tmp_dc_data_unref(dc_data);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
profiles/health/hdp.c:800:19: warning: Use of memory after it is freed
path = g_strdup(chan->path);
^~~~~~~~~~
profiles/health/hdp.c:1779:6: warning: Use of memory after it is freed
hdp_tmp_dc_data_ref(hdp_conn),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
profiles/health/hdp.c:1836:30: warning: Use of memory after it is freed
reply = g_dbus_create_error(data->msg, ERROR_INTERFACE ".HealthError",
^~~~~~~~~
4 warnings generated.
profiles/health/hdp_util.c:1052:2: warning: Use of memory after it is freed
conn_data->func(conn_data->data, gerr);
^~~~~~~~~~~~~~~
1 warning generated.
attrib/gatt.c:970:2: warning: Potential leak of memory pointed to by 'long_write'
return prepare_write(long_write);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
src/sdpd-request.c:211:13: warning: Result of 'malloc' is converted to a pointer of type 'char', which is incompatible with sizeof operand type 'uint16_t'
pElem = malloc(sizeof(uint16_t));
^~~~~~ ~~~~~~~~~~~~~~~~
src/sdpd-request.c:239:13: warning: Result of 'malloc' is converted to a pointer of type 'char', which is incompatible with sizeof operand type 'uint32_t'
pElem = malloc(sizeof(uint32_t));
^~~~~~ ~~~~~~~~~~~~~~~~
2 warnings generated.
src/sdp-xml.c:126:10: warning: Assigned value is garbage or undefined
buf[1] = data[i + 1];
^ ~~~~~~~~~~~
src/sdp-xml.c:300:11: warning: Assigned value is garbage or undefined
buf[1] = data[i + 1];
^ ~~~~~~~~~~~
src/sdp-xml.c:338:11: warning: Assigned value is garbage or undefined
buf[1] = data[i + 1];
^ ~~~~~~~~~~~
3 warnings generated.
src/sdp-client.c:353:14: warning: Access to field 'cb' results in a dereference of a null pointer
(*ctxt)->cb = cb;
~~~~~~~~~~~~^~~~
1 warning generated.
src/gatt-database.c:1154:10: warning: Value stored to 'bits' during its initialization is never read
uint8_t bits[] = { BT_GATT_CHRC_CLI_FEAT_ROBUST_CACHING,
^~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
gobex/gobex-header.c:67:2: warning: Null pointer passed to 2nd parameter expecting 'nonnull'
memcpy(to, from, count);
^~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
gobex/gobex-transfer.c:423:7: warning: Use of memory after it is freed
if (!g_slist_find(transfers, transfer))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
mesh/main.c:161:3: warning: Value stored to 'optarg' is never read
optarg += strlen("auto");
^ ~~~~~~~~~~~~~~
1 warning generated.
lib/hci.c:97:4: warning: Value stored to 'ptr' is never read
ptr += sprintf(ptr, "%s", m->str);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
gdbus/watch.c:204:3: warning: Attempt to free released memory
g_free(l->data);
^~~~~~~~~~~~~~~
1 warning generated.
lib/sdp.c:509:16: warning: Dereference of undefined pointer value
int8_t dtd = *(uint8_t *) dtds[i];
^~~~~~~~~~~~~~~~~~~~
lib/sdp.c:537:17: warning: Dereference of undefined pointer value
uint8_t dtd = *(uint8_t *) dtds[i];
^~~~~~~~~~~~~~~~~~~~
lib/sdp.c:582:12: warning: Access to field 'attrId' results in a dereference of a null pointer (loaded from variable 'd')
d->attrId = attr;
~ ^
lib/sdp.c:1872:26: warning: Potential leak of memory pointed to by 'ap'
for (; pdlist; pdlist = pdlist->next) {
^~~~~~
lib/sdp.c:1886:6: warning: Potential leak of memory pointed to by 'pds'
ap = sdp_list_append(ap, pds);
~~~^~~~~~~~~~~~~~~~~~~~~~~~~~
lib/sdp.c:1931:10: warning: Potential leak of memory pointed to by 'u'
*seqp = sdp_list_append(*seqp, u);
~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
lib/sdp.c:2036:4: warning: Potential leak of memory pointed to by 'lang'
sdp_list_free(*langSeq, free);
^~~~~~~~~~~~~
lib/sdp.c:2125:9: warning: Potential leak of memory pointed to by 'profDesc'
return 0;
^
lib/sdp.c:3257:8: warning: Potential leak of memory pointed to by 'pSvcRec'
pSeq = sdp_list_append(pSeq, pSvcRec);
~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
lib/sdp.c:3258:9: warning: Potential leak of memory pointed to by 'pSeq'
pdata += sizeof(uint32_t);
~~~~~~^~~~~~~~~~~~~~~~~~~
lib/sdp.c:4594:13: warning: Potential leak of memory pointed to by 'rec_list'
} while (scanned < attr_list_len && pdata_len > 0);
^~~~~~~
lib/sdp.c:4890:40: warning: Potential leak of memory pointed to by 'tseq'
for (d = sdpdata->val.dataseq; d; d = d->next) {
^
lib/sdp.c:4926:8: warning: Potential leak of memory pointed to by 'subseq'
tseq = sdp_list_append(tseq, subseq);
~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
13 warnings 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:3237:2: warning: Use of memory after it is freed
complete_write_long_op(req, success, 0, false);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/gatt-client.c:3259:2: warning: Use of memory after it is freed
request_unref(req);
^~~~~~~~~~~~~~~~~~
12 warnings generated.
src/shared/shell.c:1228:13: warning: Access to field 'options' results in a dereference of a null pointer (loaded from variable 'opt')
if (c != opt->options[index - offset].val) {
^~~~~~~~~~~~
1 warning generated.
src/shared/bap.c:1132:2: warning: Use of memory after it is freed
DBG(bap, "stream %p", stream);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:40:2: note: expanded from macro 'DBG'
bap_debug(_bap, "%s:%s() " fmt, __FILE__, __func__, ## arg)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:1145:2: warning: Use of memory after it is freed
DBG(stream->bap, "stream %p", stream);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:40:2: note: expanded from macro 'DBG'
bap_debug(_bap, "%s:%s() " fmt, __FILE__, __func__, ## arg)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/bap.c:1240:33: warning: Use of memory after it is freed
for (entry = queue_get_entries(bap->state_cbs); entry;
^~~~~~~~~~~~~~
src/shared/bap.c:1698:3: warning: Use of memory after it is freed
stream_set_state(stream, BT_BAP_STREAM_STATE_CONFIG);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
4 warnings generated.
monitor/l2cap.c:1638:4: warning: Value stored to 'data' is never read
data += len;
^ ~~~
monitor/l2cap.c:1639:4: warning: Value stored to 'size' is never read
size -= len;
^ ~~~
2 warnings generated.
monitor/hwdb.c:59:2: warning: Value stored to 'hwdb' is never read
hwdb = udev_hwdb_unref(hwdb);
^ ~~~~~~~~~~~~~~~~~~~~~
monitor/hwdb.c:64:2: warning: Value stored to 'udev' is never read
udev = udev_unref(udev);
^ ~~~~~~~~~~~~~~~~
monitor/hwdb.c:106:2: warning: Value stored to 'hwdb' is never read
hwdb = udev_hwdb_unref(hwdb);
^ ~~~~~~~~~~~~~~~~~~~~~
monitor/hwdb.c:111:2: warning: Value stored to 'udev' is never read
udev = udev_unref(udev);
^ ~~~~~~~~~~~~~~~~
4 warnings generated.
monitor/att.c: In function ‘print_call_friendly_name’:
monitor/att.c:3059:3: error: label ‘done’ used but not defined
3059 | goto done;
| ^~~~
monitor/att.c:3054:8: error: unused variable ‘name’ [-Werror=unused-variable]
3054 | char *name;
| ^~~~
monitor/att.c: At top level:
monitor/att.c:4051:23: error: ‘call_cp_opt_opcodes_read’ undeclared here (not in a function); did you mean ‘call_cp_oipt_opcodes_read’?
4051 | GATT_HANDLER(0x2bbf, call_cp_opt_opcodes_read, NULL, NULL),
| ^~~~~~~~~~~~~~~~~~~~~~~~
monitor/att.c:3980:10: note: in definition of macro ‘GATT_HANDLER’
3980 | .read = _read, \
| ^~~~~
monitor/att.c:2953:13: error: ‘call_cp_oipt_opcodes_read’ defined but not used [-Werror=unused-function]
2953 | static void call_cp_oipt_opcodes_read(const struct l2cap_frame *frame)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:7773: monitor/att.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:4630: all] Error 2


---
Regards,
Linux Bluetooth