Olivier Lacoste-Bouchet closed merge request !2: Various Fixes at Luigi Calligaris / OpenIPMC

Olivier Lacoste-Bouchet
2022-10-12T14:36:53Z

sequence number byte does not contain the lun and it does not need to be removed from the byte. (in src/ipmi_msg_manager.c; this is an actual bug rather than a preventative measure like the others)

Check FreeRTOS returns and at least notify the user

Prevent possible buffer overrun in debug print. Since messages shouldn't exceed 32 bytes it's unlikely this happens but better to be safe than sorry

use of a const variable in a constant expression is nonstandard in C -> warning from my compiler, should instead be a #define macro