Fix really stupid use of uninitialised value introduced in rev 1.8.

Basically this bug invalidates all snapshots from 0501 until now.

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@212

diff --git a/coregrind/vg_memory.c b/coregrind/vg_memory.c
index 14d557d5f403581735d53151cab07b64adc259f7..59d183b48a4fa327b9660e640a367a8072d8baf3 100644 (file)
--- a/coregrind/vg_memory.c
+++ b/coregrind/vg_memory.c
@@ -2128,7 +2128,6 @@ void VG_(do_sanity_checks) ( Bool force_expensive )
 {
    Int          i;
    Bool         do_expensive_checks;
-   ThreadState* tst;
 
    if (VG_(sanity_level) < 1) return;
 
@@ -2148,10 +2147,6 @@ void VG_(do_sanity_checks) ( Bool force_expensive )
 
    if (VG_(clo_instrument)) {
 
-      /* Check that the eflags tag is as expected. */
-      UInt vv = tst->sh_eflags;
-      vg_assert(vv == VGM_EFLAGS_VALID || VGM_EFLAGS_INVALID);
-
       /* Check that nobody has spuriously claimed that the first or
          last 16 pages of memory have become accessible [...] */
       vg_assert(VG_(first_and_last_secondaries_look_plausible));

diff --git a/vg_memory.c b/vg_memory.c
index 14d557d5f403581735d53151cab07b64adc259f7..59d183b48a4fa327b9660e640a367a8072d8baf3 100644 (file)
--- a/vg_memory.c
+++ b/vg_memory.c
@@ -2128,7 +2128,6 @@ void VG_(do_sanity_checks) ( Bool force_expensive )
 {
    Int          i;
    Bool         do_expensive_checks;
-   ThreadState* tst;
 
    if (VG_(sanity_level) < 1) return;
 
@@ -2148,10 +2147,6 @@ void VG_(do_sanity_checks) ( Bool force_expensive )
 
    if (VG_(clo_instrument)) {
 
-      /* Check that the eflags tag is as expected. */
-      UInt vv = tst->sh_eflags;
-      vg_assert(vv == VGM_EFLAGS_VALID || VGM_EFLAGS_INVALID);
-
       /* Check that nobody has spuriously claimed that the first or
          last 16 pages of memory have become accessible [...] */
       vg_assert(VG_(first_and_last_secondaries_look_plausible));