Fix comparison in function VG_(iseqsigset).

Patch by dcb314@hotmail.com
Fixes BZ #351474.

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

diff --git a/NEWS b/NEWS
index 2c2ea7ca47f4c8a2ec4bb19c59b39697a21fe83d..f760fd1d530ca96479d8f698c881c9ce4074c1bc 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -307,6 +307,7 @@ where XXXXXX is the bug number as listed below.
 350854  hard-to-understand code in VG_(load_ELF)()
 351140  arm64 syscalls setuid (146) and setresgid (149) not implemented
 351386  Cannot run ld.so.1 under Valgrind
+351474  Fix VG_(iseqsigset) as obvious
 n-i-bz  Provide implementations of certain compiler builtins to support
         compilers who may not provide those
 n-i-bz  Old STABS code is still being compiled, but never used. Remove it.

diff --git a/coregrind/m_libcsignal.c b/coregrind/m_libcsignal.c
index b84bc083a37d1eeab78049e4c580fbe8369d09e4..8c5b7f6b6538adcfac3ebe7da55bbac62a1b6c76 100644 (file)
--- a/coregrind/m_libcsignal.c
+++ b/coregrind/m_libcsignal.c
@@ -131,7 +131,7 @@ Bool VG_(iseqsigset)( const vki_sigset_t* set1, const vki_sigset_t* set2 )
       else {
          /* Partial word comparison. */
          ULong mask = (1UL << (_VKI_MAXSIG % _VKI_NSIG_BPW)) - 1;
-         if ((set1->sig[i] & mask) != (set1->sig[i] & mask)) return False;
+         if ((set1->sig[i] & mask) != (set2->sig[i] & mask)) return False;
          break;
       }
    }