card_emu_uart_interrupt: ASSERT if we get called with wrong uart_chan

This is what we do in all other functions, not sure why this one wants to silently ignore any such programming errors. Change-Id: I022eee86a5a3b5077abe59897161578ed960f1b1
osmocom · Feb 14, 2022 · 8680677 · 8680677
1 parent 20bc014
commit 8680677
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/firmware/libcommon/source/mode_cardemu.c b/firmware/libcommon/source/mode_cardemu.c
@@ -410,10 +410,9 @@ void card_emu_uart_reset_wt(uint8_t uart_chan)
 /* call-back from card_emu.c to force a USART interrupt */
 void card_emu_uart_interrupt(uint8_t uart_chan)
 {
+ OSMO_ASSERT(uart_chan < ARRAY_SIZE(cardem_inst));
  Usart *usart = get_usart_by_chan(uart_chan);
- if (!usart) {
- return;
- }
+
  if (USART0 == usart) {
  NVIC_SetPendingIRQ(USART0_IRQn);
  } else if (USART1 == usart) {