Skip to content

Commit e40e91f

Browse files
olivier-le-sagenordicjm
authored andcommitted
bluetooth: ras: rreq: split up error logs for different API misuse cases
Customers are coming to us with with logs from their apps, which end up being unhelpful because there are too many possible failure conditions for this log message. Signed-off-by: Olivier Lesage <[email protected]>
1 parent ac7ef96 commit e40e91f

File tree

1 file changed

+19
-3
lines changed

1 file changed

+19
-3
lines changed

subsys/bluetooth/services/ras/rreq/ras_rreq.c

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1207,9 +1207,25 @@ void bt_ras_rreq_rd_subevent_data_parse(struct net_buf_simple *peer_ranging_data
12071207
bt_ras_rreq_subevent_header_cb_t subevent_header_cb,
12081208
bt_ras_rreq_step_data_cb_t step_data_cb, void *user_data)
12091209
{
1210-
if (!peer_ranging_data_buf || !local_step_data_buf || peer_ranging_data_buf->len == 0 ||
1211-
local_step_data_buf->len == 0) {
1212-
LOG_ERR("Tried to parse empty step data.");
1210+
bool error = false;
1211+
1212+
if (!peer_ranging_data_buf) {
1213+
LOG_ERR("No peer step data provided.");
1214+
error = true;
1215+
} else if (peer_ranging_data_buf->len == 0) {
1216+
LOG_ERR("Tried to parse empty peer step data.");
1217+
error = true;
1218+
}
1219+
1220+
if (!local_step_data_buf) {
1221+
LOG_ERR("No local step data provided.");
1222+
error = true;
1223+
} else if (local_step_data_buf->len == 0) {
1224+
LOG_ERR("Tried to parse empty local step data.");
1225+
error = true;
1226+
}
1227+
1228+
if (error) {
12131229
return;
12141230
}
12151231

0 commit comments

Comments
 (0)