ChibiOS Free Embedded RTOS

Posted: **Wed Dec 16, 2015 8:47 am**

Hello all,

a code checker found the following minimal bug in ..\hal\ports\common\ARMCMx\nvic.c:

/**
 * @brief   Changes the priority of a system handler.
 *
 * @param[in] handler   the system handler number
 * @param[in] prio      the system handler priority
 */
void nvicSetSystemHandlerPriority(uint32_t handler, uint32_t prio) {

  osalDbgCheck(handler <= 12U);

#if defined(__CORE_CM0_H_GENERIC)
  SCB->SHP[_SHP_IDX(handler)] = (SCB->SHP[_SHP_IDX(handler)] & ~(0xFFU << _BIT_SHIFT(handler))) |
                                (NVIC_PRIORITY_MASK(prio) << _BIT_SHIFT(handler));
#elif defined(__CORE_CM7_H_GENERIC)
  SCB->SHPR[handler] = NVIC_PRIORITY_MASK(prio);
#else
  SCB->SHP[handler] = NVIC_PRIORITY_MASK(prio);
#endif
}

The line

Code: Select all

osalDbgCheck(handler <= 12U);

should be

Code: Select all

osalDbgCheck(handler < 12U);

at least in conjunction with core_cm*.h where

Code: Select all

SHP[12]

is defined.

Kind Regards
Michael

Posted: **Wed Dec 16, 2015 9:45 am**

Fixed, thanks.

Giovanni

ChibiOS Free Embedded RTOS

Range check off by one in nvic.c

Range check off by one in nvic.c

Re: Range check off by one in nvic.c