Fixes the size comparison in the avtp_<format>_valid function, i.e., converts the length to octets

Author: warlock20

src/avtp/acf/Can.c

@@ -297,7 +297,8 @@ uint8_t Avtp_Can_IsValid(Avtp_Can_t* pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if (Avtp_Can_GetAcfMsgLength(pdu) > bufferSize) {
+    // Avtp_Can_GetAcfMsgLength returns quadlets. Convert the length field to octets
+    if (Avtp_Can_GetAcfMsgLength(pdu) * 4 > bufferSize) {
         return FALSE;
     }
     return TRUE;

src/avtp/acf/CanBrief.c

@@ -267,7 +267,8 @@ uint8_t Avtp_CanBrief_IsValid(Avtp_CanBrief_t* pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if (Avtp_CanBrief_GetAcfMsgLength(pdu) > bufferSize) {
+    // Avtp_CanBrief_GetAcfMsgLength returns quadlets. Convert the length field to octets
+    if (Avtp_CanBrief_GetAcfMsgLength(pdu) * 4 > bufferSize) {
         return FALSE;
     }
 

src/avtp/acf/FlexRay.c

@@ -254,7 +254,8 @@ uint8_t Avtp_FlexRay_IsValid(Avtp_FlexRay_t *pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if (Avtp_FlexRay_GetAcfMsgLength(pdu) > bufferSize)
+    // Avtp_FlexRay_GetAcfMsgLength returns quadlets. Convert the length field to octets
+    if (Avtp_FlexRay_GetAcfMsgLength(pdu) * 4 > bufferSize)
     {
         return FALSE;
     }

src/avtp/acf/Gpc.c

@@ -113,7 +113,8 @@ uint8_t Avtp_Gpc_IsValid(Avtp_Gpc_t* pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if (Avtp_Gpc_GetAcfMsgLength(pdu) > bufferSize) {
+    // Avtp_Gpc_GetAcfMsgLength returns quadlets. Convert the length field to octets
+    if (Avtp_Gpc_GetAcfMsgLength(pdu) * 4 > bufferSize) {
         return FALSE;
     }
 

src/avtp/acf/Lin.c

@@ -160,7 +160,8 @@ uint8_t Avtp_Lin_IsValid(Avtp_Lin_t* pdu, size_t bufferSize){
         return FALSE;
     }
 
-    if (Avtp_Lin_GetAcfMsgLength(pdu) > bufferSize) {
+    // Avtp_Lin_GetAcfMsgLength returns quadlets. Convert the length field to octets
+    if (Avtp_Lin_GetAcfMsgLength(pdu) *4 > bufferSize) {
         return FALSE;
     }
 

src/avtp/acf/Most.c

@@ -211,7 +211,8 @@ uint8_t Avtp_Most_IsValid(Avtp_Most_t *pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if (Avtp_Most_GetAcfMsgLength(pdu) > bufferSize)
+    // Avtp_Most_GetAcfMsgLength returns quadlets. Convert the length field to octets
+    if (Avtp_Most_GetAcfMsgLength(pdu) * 4 > bufferSize)
     {
         return FALSE;
     }

src/avtp/acf/Ntscf.c

@@ -158,7 +158,8 @@ uint8_t Avtp_Ntscf_IsValid(Avtp_Ntscf_t* pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if (Avtp_Ntscf_GetNtscfDataLength(pdu) > bufferSize) {
+    // Avtp_Ntscf_GetNtscfDataLength returns quadlets. Convert the length field to octets
+    if (Avtp_Ntscf_GetNtscfDataLength(pdu) * 4> bufferSize) {
         return FALSE;
     }
 

src/avtp/acf/Sensor.c

@@ -163,7 +163,8 @@ uint8_t Avtp_Sensor_IsValid(Avtp_Sensor_t* pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if (Avtp_Sensor_GetAcfMsgLength(pdu) > bufferSize) {
+    // Avtp_Sensor_GetAcfMsgLength returns quadlets. Convert the length field to octets
+    if (Avtp_Sensor_GetAcfMsgLength(pdu) *4 > bufferSize) {
         return FALSE;
     }
 

src/avtp/acf/SensorBrief.c

@@ -153,7 +153,8 @@ uint8_t Avtp_SensorBrief_IsValid(Avtp_SensorBrief_t* pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if(Avtp_SensorBrief_GetAcfMsgLength(pdu) > bufferSize) {
+    // Avtp_SensorBrief_GetAcfMsgLength returns quadlets. Convert the length field to octets
+    if(Avtp_SensorBrief_GetAcfMsgLength(pdu) *4 > bufferSize) {
         return FALSE;
     }
 

src/avtp/acf/Tscf.c

@@ -217,7 +217,8 @@ uint8_t Avtp_Tscf_IsValid(Avtp_Tscf_t* pdu, size_t bufferSize)
         return FALSE;
     }
 
-    if (Avtp_Tscf_GetStreamDataLength(pdu) > bufferSize) {
+    // Avtp_Tscf_GetStreamDataLength returns quadlets. Convert the length field to octets
+    if (Avtp_Tscf_GetStreamDataLength(pdu) * 4 > bufferSize) {
         return FALSE;
     }