Fixed assert when calling cancel_alarm(0) (#1173)

mthiesen · web-flow · commit c7211db2b3f0 · 2023-05-26T12:22:09.000-05:00
diff --git a/src/common/pico_time/time.c b/src/common/pico_time/time.c
@@ -291,6 +291,7 @@ alarm_id_t alarm_pool_add_alarm_at_force_in_context(alarm_pool_t *pool, absolute
 }
 
 bool alarm_pool_cancel_alarm(alarm_pool_t *pool, alarm_id_t alarm_id) {
+    if (!alarm_id) return false;
     bool rc = false;
     uint32_t save = spin_lock_blocking(pool->lock);
     pheap_node_id_t id = (pheap_node_id_t) alarm_id;

Original file line number	Diff line number	Diff line change
`@@ -291,6 +291,7 @@ alarm_id_t alarm_pool_add_alarm_at_force_in_context(alarm_pool_t *pool, absolute`
`291`	`291`	`}`
`292`	`292`
`293`	`293`	`bool alarm_pool_cancel_alarm(alarm_pool_t *pool, alarm_id_t alarm_id) {`
	`294`	`+ if (!alarm_id) return false;`
`294`	`295`	`bool rc = false;`
`295`	`296`	`uint32_t save = spin_lock_blocking(pool->lock);`
`296`	`297`	`pheap_node_id_t id = (pheap_node_id_t) alarm_id;`