OvmfPkg/AcpiPlatformDxe: drop double right shift in ADD/WRITE POINTER cmds

[mirror_edk2.git] / OvmfPkg / AcpiPlatformDxe / QemuFwCfgAcpi.c

diff --git a/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c b/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
index eadd690bef4e591697feda1cf3995e171e5b319f..6a0ecd1ad96260d9bf8e5d0142d1e908698d42ab 100644 (file)
--- a/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
+++ b/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
@@ -277,8 +277,8 @@ ProcessCmdAddPointer (
   ASSERT ((UINTN)Blob2->Base <= MAX_ADDRESS - Blob2->Size);\r
 \r
   PointerValue += (UINT64)(UINTN)Blob2->Base;\r
-  if (RShiftU64 (\r
-        RShiftU64 (PointerValue, AddPointer->PointerSize * 8 - 1), 1) != 0) {\r
+  if (AddPointer->PointerSize < 8 &&\r
+      RShiftU64 (PointerValue, AddPointer->PointerSize * 8) != 0) {\r
     DEBUG ((EFI_D_ERROR, "%a: relocated pointer value unrepresentable in "\r
       "\"%a\"\n", __FUNCTION__, AddPointer->PointerFile));\r
     return EFI_PROTOCOL_ERROR;\r
@@ -438,8 +438,8 @@ ProcessCmdWritePointer (
   ASSERT ((UINTN)PointeeBlob->Base <= MAX_ADDRESS - PointeeBlob->Size);\r
 \r
   PointerValue += (UINT64)(UINTN)PointeeBlob->Base;\r
-  if (RShiftU64 (\r
-        RShiftU64 (PointerValue, WritePointer->PointerSize * 8 - 1), 1) != 0) {\r
+  if (WritePointer->PointerSize < 8 &&\r
+      RShiftU64 (PointerValue, WritePointer->PointerSize * 8) != 0) {\r
     DEBUG ((DEBUG_ERROR, "%a: pointer value unrepresentable in \"%a\"\n",\r
       __FUNCTION__, WritePointer->PointerFile));\r
     return EFI_PROTOCOL_ERROR;\r