Skip to content

Commit e15852c

Browse files
erwangogalak
authored andcommitted
samples: counter: Add err check on set_alarm call
Error checking was missing following call to counter_set_channel_alarm. This was reported in coverity report 203523. Fixes #18374 Additionally remove unneeded err initialization in main function Signed-off-by: Erwan Gouriou <[email protected]>
1 parent ae5db66 commit e15852c

File tree

1 file changed

+8
-2
lines changed
  • samples/drivers/counter/alarm/src

1 file changed

+8
-2
lines changed

samples/drivers/counter/alarm/src/main.c

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ static void test_counter_interrupt_fn(struct device *counter_dev,
2323
u64_t now_usec = counter_ticks_to_us(counter_dev, now_ticks);
2424
int now_sec = (int)(now_usec / USEC_PER_SEC);
2525
struct counter_alarm_cfg *config = user_data;
26+
int err;
2627

2728
printk("!!! Alarm !!!\n");
2829
printk("Now: %u\n", now_sec);
@@ -34,13 +35,18 @@ static void test_counter_interrupt_fn(struct device *counter_dev,
3435
(u32_t)(counter_ticks_to_us(counter_dev,
3536
config->ticks) / USEC_PER_SEC),
3637
config->ticks);
37-
counter_set_channel_alarm(counter_dev, ALARM_CHANNEL_ID, user_data);
38+
39+
err = counter_set_channel_alarm(counter_dev, ALARM_CHANNEL_ID,
40+
user_data);
41+
if (err != 0) {
42+
printk("Alarm could not be set\n");
43+
}
3844
}
3945

4046
void main(void)
4147
{
4248
struct device *counter_dev;
43-
int err = 0;
49+
int err;
4450

4551
printk("Counter alarm sample\n\n");
4652
counter_dev = device_get_binding(DT_RTC_0_NAME);

0 commit comments

Comments
 (0)