Bluetooth: GATT: Fix assert when storying CCCs

Vudentz · nashif · commit 3f19a2e020dc · 2019-08-20T19:20:05.000-04:00
This fixes the following crash:

ASSERTION FAIL [0] @ ZEPHYR_BASE/subsys/logging/log_core.c:180
	argument 2 in log message "%s: Stored CCCs for %s (%s)"
	missing log_strdup().

Signed-off-by: Luiz Augusto von Dentz &lt;luiz.von.dentz@intel.com&gt;
diff --git a/subsys/bluetooth/host/gatt.c b/subsys/bluetooth/host/gatt.c
@@ -3231,7 +3231,8 @@ int bt_gatt_store_ccc(u8_t id, const bt_addr_le_t *addr)
 		return err;
 	}
 
-	BT_DBG("Stored CCCs for %s (%s)", bt_addr_le_str(addr), key);
+	BT_DBG("Stored CCCs for %s (%s)", bt_addr_le_str(addr),
+	       log_strdup(key));
 	if (len) {
 		for (int i = 0; i < save.count; i++) {
 			BT_DBG("  CCC: handle 0x%04x value 0x%04x",

Original file line number	Diff line number	Diff line change
`@@ -3231,7 +3231,8 @@ int bt_gatt_store_ccc(u8_t id, const bt_addr_le_t *addr)`
`3231`	`3231`	`return err;`
`3232`	`3232`	`}`
`3233`	`3233`
`3234`		`- BT_DBG("Stored CCCs for %s (%s)", bt_addr_le_str(addr), key);`
	`3234`	`+ BT_DBG("Stored CCCs for %s (%s)", bt_addr_le_str(addr),`
	`3235`	`+ log_strdup(key));`
`3235`	`3236`	`if (len) {`
`3236`	`3237`	`for (int i = 0; i < save.count; i++) {`
`3237`	`3238`	`BT_DBG(" CCC: handle 0x%04x value 0x%04x",`