RedfishPkg/HostInterfaceBmcUsbNic: Fix incorrect HI protocol record size

The size of structure must be minus with byte that is occupied by the initial array. Signed-off-by: Abner Chang <[email protected]> Cc: Nickle Wang <[email protected]> Cc: Igor Kulchytskyy <[email protected]> Cc: Mike Maslenkin <[email protected]> Reviewed-by: Nickle Wang <[email protected]> Acked-by: Mike Maslenkin <[email protected]>
YuweiChen1110 · Dec 5, 2023 · 5cdeff1 · 5cdeff1
1 parent cf31257
commit 5cdeff1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInterfaceBmcUsbNicLib.c b/RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInterfaceBmcUsbNicLib.c
@@ -180,7 +180,7 @@ RedfishPlatformHostInterfaceProtocolData (
       HostNameLength     = (UINT8)AsciiStrSize (HostNameString);
       ThisProtocolRecord = (MC_HOST_INTERFACE_PROTOCOL_RECORD *)AllocateZeroPool (
                                                                   sizeof (MC_HOST_INTERFACE_PROTOCOL_RECORD) - 1 +
-                                                                  sizeof (REDFISH_OVER_IP_PROTOCOL_DATA) +
+                                                                  sizeof (REDFISH_OVER_IP_PROTOCOL_DATA) - 1 +
                                                                   HostNameLength
                                                                   );
       if (ThisProtocolRecord == NULL) {
@@ -189,7 +189,7 @@ RedfishPlatformHostInterfaceProtocolData (
       }
 
       ThisProtocolRecord->ProtocolType        = MCHostInterfaceProtocolTypeRedfishOverIP;
-      ThisProtocolRecord->ProtocolTypeDataLen = sizeof (REDFISH_OVER_IP_PROTOCOL_DATA) + HostNameLength;
+      ThisProtocolRecord->ProtocolTypeDataLen = sizeof (REDFISH_OVER_IP_PROTOCOL_DATA) -1 + HostNameLength;
       RedfishOverIpData                       = (REDFISH_OVER_IP_PROTOCOL_DATA *)&ThisProtocolRecord->ProtocolTypeData[0];
       //
       // Fill up REDFISH_OVER_IP_PROTOCOL_DATA