Fix bounds check using a define instead of table size. The bounds check was incorrectly picking a define that wasn't even the last in the table. It now uses the size of the table.

paul-szczepanek-arm · paul-szczepanek-arm · commit 0affb7374722 · 2020-06-03T15:19:39.000+01:00
diff --git a/features/FEATURE_BLE/targets/TARGET_CORDIO/stack/ble-host/sources/stack/dm/dm_main.c b/features/FEATURE_BLE/targets/TARGET_CORDIO/stack/ble-host/sources/stack/dm/dm_main.c
@@ -230,7 +230,7 @@ dmCb_t dmCb;
 /*************************************************************************************************/
 static void dmHciEvtCback(hciEvt_t *pEvent)
 {
-  WSF_ASSERT(pEvent->hdr.event <= HCI_LE_READ_ANTENNA_INFO_CMD_CMPL_CBACK_EVT);
+  WSF_ASSERT(pEvent->hdr.event < sizeof(dmHciToIdTbl));
 
   /* if DM not resetting or resetting but incoming event is HCI reset sequence complete event */
   if (!dmCb.resetting || (pEvent->hdr.event == HCI_RESET_SEQ_CMPL_CBACK_EVT))

Original file line number	Diff line number	Diff line change
`@@ -230,7 +230,7 @@ dmCb_t dmCb;`
`230`	`230`	`/*************************************************************************************************/`
`231`	`231`	`static void dmHciEvtCback(hciEvt_t *pEvent)`
`232`	`232`	`{`
`233`		`- WSF_ASSERT(pEvent->hdr.event <= HCI_LE_READ_ANTENNA_INFO_CMD_CMPL_CBACK_EVT);`
	`233`	`+ WSF_ASSERT(pEvent->hdr.event < sizeof(dmHciToIdTbl));`
`234`	`234`
`235`	`235`	`/* if DM not resetting or resetting but incoming event is HCI reset sequence complete event */`
`236`	`236`	`if (!dmCb.resetting \|\| (pEvent->hdr.event == HCI_RESET_SEQ_CMPL_CBACK_EVT))`