commit 28ac2856e7b423cfb5168f7161b72f746890534a
author Tasnim <tasnim.ouni@st.com> Fri Dec 22 17:14:41 2023 +0100
committer Tasnim <tasnim.ouni@st.com> Fri Dec 22 17:25:13 2023 +0100
tree 56d6b777d08ff9d4faae9a325470fefd15bbdbb6
parent 8c7077398fb8c4481f7d843e25b4f1471971b442

[HAL][CAN] Improve protection against bad inputs

Src/stm32f7xx_hal_can.c

diff --git a/Src/stm32f7xx_hal_can.c b/Src/stm32f7xx_hal_can.c
index afe3f68..3db667a 100644
--- a/Src/stm32f7xx_hal_can.c
+++ b/Src/stm32f7xx_hal_can.c
@@ -1552,7 +1552,15 @@
                         hcan->Instance->sFIFOMailBox[RxFifo].RIR) >> CAN_RI0R_EXID_Pos;
     }
     pHeader->RTR = (CAN_RI0R_RTR & hcan->Instance->sFIFOMailBox[RxFifo].RIR);
-    pHeader->DLC = (CAN_RDT0R_DLC & hcan->Instance->sFIFOMailBox[RxFifo].RDTR) >> CAN_RDT0R_DLC_Pos;
+    if (((CAN_RDT0R_DLC & hcan->Instance->sFIFOMailBox[RxFifo].RDTR) >> CAN_RDT0R_DLC_Pos) >= 8U)
+    {
+      /* Truncate DLC to 8 if received field is over range */
+      pHeader->DLC = 8U;
+    }
+    else
+    {
+      pHeader->DLC = (CAN_RDT0R_DLC & hcan->Instance->sFIFOMailBox[RxFifo].RDTR) >> CAN_RDT0R_DLC_Pos;
+    }
     pHeader->FilterMatchIndex = (CAN_RDT0R_FMI & hcan->Instance->sFIFOMailBox[RxFifo].RDTR) >> CAN_RDT0R_FMI_Pos;
     pHeader->Timestamp = (CAN_RDT0R_TIME & hcan->Instance->sFIFOMailBox[RxFifo].RDTR) >> CAN_RDT0R_TIME_Pos;