michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: modem: hl7800: remove VLA

Browse source 
Remove variable length array from function to comply with MISRA checks.

Signed-off-by: Ryan Erickson <ryan.erickson@lairdconnect.com>
This commit is contained in:
Ryan Erickson 2020-08-31 13:57:51 -05:00 • committed by Maureen Helm
commit ad04354110
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
drivers/modem/hl7800.c
View file

@ -3279,6 +3279,9 @@ static void hl7800_rx(void)
static char rx_msg[MDM_HANDLER_MATCH_MAX_LEN]; static char rx_msg[MDM_HANDLER_MATCH_MAX_LEN];
bool unlock = false; bool unlock = false;
bool remove_line_from_buf = true; bool remove_line_from_buf = true;
#ifdef HL7800_LOG_UNHANDLED_RX_MSGS
char msg[MDM_MAX_RESP_SIZE];
#endif
static const struct cmd_handler handlers[] = { static const struct cmd_handler handlers[] = {
/* MODEM Information */ /* MODEM Information */
@ -3440,7 +3443,6 @@ static void hl7800_rx(void)
/* Handle unhandled commands */ /* Handle unhandled commands */
if (IS_ENABLED(HL7800_LOG_UNHANDLED_RX_MSGS) && if (IS_ENABLED(HL7800_LOG_UNHANDLED_RX_MSGS) &&
!cmd_handled && frag && len > 1) { !cmd_handled && frag && len > 1) {
char msg[len + 1];
out_len = net_buf_linearize(msg, sizeof(msg), out_len = net_buf_linearize(msg, sizeof(msg),
rx_buf, 0, len); rx_buf, 0, len);
msg[out_len] = 0; msg[out_len] = 0;