espressif/esp-nimble
1
0
Fork 0
mirror of https://github.com/espressif/esp-nimble.git synced 2026-10-05 22:37:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

mesh: cfg_cli: Fix trying to write a NULL pointer

Browse Source 
It's not always guaranteed that param->status will be non-NULL,
especially not after a subsequent patch to fix a race condition with
the response waiting.
This commit is contained in:
Michał Narajowski
2018-04-10 12:32:26 +02:00
parent 8992eaabc3
commit f9e291cc87
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nimble/host/mesh/src/cfg_cli.c
+6 -2
View File
@@ -313,7 +313,9 @@ static void mod_pub_status(struct bt_mesh_model *model,
return; return;
} }
*param->status = status; if (param->status) {
*param->status = status;
}
if (param->pub) { if (param->pub) {
param->pub->addr = net_buf_simple_pull_le16(buf); param->pub->addr = net_buf_simple_pull_le16(buf);
@@ -378,7 +380,9 @@ static void mod_sub_status(struct bt_mesh_model *model,
*param->sub_addr = sub_addr; *param->sub_addr = sub_addr;
} }
*param->status = status; if (param->status) {
*param->status = status;
}
k_sem_give(&cli->op_sync); k_sem_give(&cli->op_sync);
} }