check for invalid mailbox header length

OpenEtherCATsociety · Nov 29, 2022 · bdf57ad · bdf57ad
1 parent 6a5babf
commit bdf57ad
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/soem/ethercatcoe.c b/soem/ethercatcoe.c
@@ -238,7 +238,8 @@ int ecx_SDOread(ecx_contextt *context, uint16 slave, uint16 index, uint8 subinde
                               /* slave response should be CoE, SDO response */
                               if ((((aSDOp->MbxHeader.mbxtype & 0x0f) == ECT_MBXT_COE) &&
                                    ((etohs(aSDOp->CANOpen) >> 12) == ECT_COES_SDORES) &&
-                                   ((aSDOp->Command & 0xe0) == 0x00)))
+                                   ((aSDOp->Command & 0xe0) == 0x00) &&
+                                   (etohs(aSDOp->MbxHeader.length) >= 3)))
                                         {
                                  /* calculate mailbox transfer size */
                                  Framedatasize = etohs(aSDOp->MbxHeader.length) - 3;