Changeset 1510 in vbox for trunk/src/recompiler

Timestamp:

Mar 15, 2007 12:33:47 PM (18 years ago)

Author:

vboxsync

Message:

Using a null selector is illegal.

File:

• trunk/src/recompiler/target-i386/op.c (modified) (2 diffs)

trunk/src/recompiler/target-i386/op.c

@@ -496 +496 @@
         sync_seg(env, idx, env->segs[idx].newselector);
     }
+    /* Loading a null selector into a segment register is valid, but using it is most definitely not! */
+    if (    (env->cr[0] & (CR0_PE_MASK|CR0_PG_MASK)) == (CR0_PE_MASK|CR0_PG_MASK)
+        &&  !(env->eflags & VM_MASK)
+        &&  env->segs[idx].selector == 0
+       )
+    {
+        raise_exception(EXCP0D_GPF);
+    }
+
     A0 = (uint32_t)env->segs[idx].base;
 #else
@@ -511 +520 @@
         sync_seg(env, idx, env->segs[idx].newselector);
     }
+    /* Loading a null selector into a segment register is valid, but using it is most definitely not! */
+    if (    (env->cr[0] & (CR0_PE_MASK|CR0_PG_MASK)) == (CR0_PE_MASK|CR0_PG_MASK)
+        &&  !(env->eflags & VM_MASK)
+        &&  env->segs[idx].selector == 0
+       )
+    {
+        raise_exception(EXCP0D_GPF);
+    }
+
     A0 = (uint32_t)(A0 + env->segs[idx].base);
 #else