Changeset 66844 in vbox for trunk/src/VBox/HostDrivers/Support/posix

Timestamp:

May 9, 2017 9:27:32 AM (8 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

115302

Message:

Support/posix: Just use push imm64...

File:

• trunk/src/VBox/HostDrivers/Support/posix/SUPR3HardenedMain-posix.cpp (modified) (3 diffs)

trunk/src/VBox/HostDrivers/Support/posix/SUPR3HardenedMain-posix.cpp

@@ -369 +369 @@
 
     /*
-     * Each relative call requires extra bytes as it is converted to two push imm32
+     * Each relative call requires extra bytes as it is converted to a push imm64
      * + a jmp qword [$+8 wrt RIP] to avoid clobbering registers.
      */
-    cbPatchMem += cRelCalls * RT_ALIGN_32(2 * 5 + 6 + 8, 8);
+    cbPatchMem += cRelCalls * RT_ALIGN_32(9 + 6 + 8, 8);
     cbPatchMem += 14; /* jmp qword [$+8 wrt RIP] + 8 byte address to jump to. */
     cbPatchMem = RT_ALIGN_32(cbPatchMem, 8);
@@ -457 +457 @@
             uintptr_t uAddr = (uintptr_t)&pbTarget[offInsn + cbInstr] + (intptr_t)Dis.Param1.uValue;
 
-            /* Skip the first two push instructions till the return address is known. */
+            /* Skip the initial push instructions till the return address is known. */
             uint8_t *pbPatchMemPush = pbPatchMem;
-            pbPatchMem += 2 * 5;
+            pbPatchMem += 9;
 
             *pbPatchMem++ = 0xff; /* jmp qword [$+8 wrt RIP] */
@@ -468 +468 @@
             pbPatchMem += sizeof(uint64_t);
 
-            /* Create two pushes now which will put the return address onto the stack. */
+            /* Push the return address onto the stack. */
             uintptr_t uAddrReturn = (uintptr_t)pbPatchMem;
-            *pbPatchMemPush++ = 0x68; /* push imm32 */
-            *(uint32_t *)pbPatchMemPush = (uint32_t)(uAddrReturn >> 32);
-            pbPatchMemPush   += sizeof(uint32_t);
-
-            *pbPatchMemPush++ = 0x68; /* push imm32 */
-            *(uint32_t *)pbPatchMemPush = (uint32_t)uAddrReturn;
-            pbPatchMemPush   += sizeof(uint32_t);
+            *pbPatchMemPush++ = 0x68; /* push imm64 */
+            *(uint64_t *)pbPatchMemPush = uAddrReturn;
         }
         else