Changeset 99131 in vbox for trunk/include

Timestamp:

Mar 23, 2023 8:50:34 AM (21 months ago)

Author:

vboxsync

Message:

include/iprt/asm.h: Revert r156471 and replace the buggy AssertStmt() with an AssertRelease() to fend off unaligned accesses

File:

• trunk/include/iprt/asm.h (modified) (1 diff)

trunk/include/iprt/asm.h

@@ -6819 +6819 @@
 
 # else
-    int32_t offBitmap = iBit >> 3;
-    rc.u32 = ASMAtomicUoReadU8(&((uint8_t volatile *)pvBitmap)[offBitmap]) >> (iBit & 7);
+    int32_t offBitmap = iBit / 32;
+    AssertRelease(!((uintptr_t)pvBitmap & (sizeof(uintptr_t) - 1))); /* This assumes a power of two uintptr_t size. */
+    rc.u32 = RT_LE2H_U32(ASMAtomicUoReadU32(&((uint32_t volatile *)pvBitmap)[offBitmap])) >> (iBit & 31);
     rc.u32 &= 1;
 # endif