[Valgrind-developers] [valgrind] Enable -Wlogical-op for Valgrind; fix an obvious typo.

[Ivo R. <ir...@so...>]

https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=f19a956e0a96b4f6b37c50b00ae24ecf09a7a3f5

commit f19a956e0a96b4f6b37c50b00ae24ecf09a7a3f5
Author: Ivo Raisr <iv...@iv...>
Date:   Thu Jan 18 17:20:02 2018 +0100

    Enable -Wlogical-op for Valgrind; fix an obvious typo.
    
    Fixes BZ#389065.
    Reported by: dc...@ho...

Diff:
---
 Makefile.all.am                   | 1 +
 NEWS                              | 1 +
 configure.ac                      | 1 +
 coregrind/m_syswrap/syswrap-xen.c | 2 +-
 4 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/Makefile.all.am b/Makefile.all.am
index 2d698cf..30e79dd 100644
--- a/Makefile.all.am
+++ b/Makefile.all.am
@@ -113,6 +113,7 @@ AM_CFLAGS_BASE = \
 	@FLAG_W_FORMAT_SECURITY@ \
 	@FLAG_W_IGNORED_QUALIFIERS@ \
 	@FLAG_W_MISSING_PARAMETER_TYPE@ \
+	@FLAG_W_LOGICAL_OP@ \
 	@FLAG_W_OLD_STYLE_DECLARATION@ \
 	@FLAG_FNO_STACK_PROTECTOR@ \
 	@FLAG_FSANITIZE@ \
diff --git a/NEWS b/NEWS
index ac73dde..ee853d5 100644
--- a/NEWS
+++ b/NEWS
@@ -86,6 +86,7 @@ where XXXXXX is the bug number as listed below.
 386397  PPC64, valgrind truncates powerpc timebase to 32-bits.
 387712  s390x cgijnl reports Conditional jump depends on uninitialised value
 387773  .gnu_debugaltlink paths resolve relative to .debug file, not symlink
+389065  valgrind meets gcc flag -Wlogical-op
 
 n-i-bz  Fix missing workq_ops operations (macOS)
 n-i-bz  fix bug in strspn replacement
diff --git a/configure.ac b/configure.ac
index 901c595..98d7ac6 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2004,6 +2004,7 @@ AC_GCC_WARNING_SUBST([cast-qual], [FLAG_W_CAST_QUAL])
 AC_GCC_WARNING_SUBST([old-style-declaration], [FLAG_W_OLD_STYLE_DECLARATION])
 AC_GCC_WARNING_SUBST([ignored-qualifiers], [FLAG_W_IGNORED_QUALIFIERS])
 AC_GCC_WARNING_SUBST([missing-parameter-type], [FLAG_W_MISSING_PARAMETER_TYPE])
+AC_GCC_WARNING_SUBST([logical-op], [FLAG_W_LOGICAL_OP])
 
 # Does this compiler support -Wformat-security ?
 # Special handling is needed, because certain GCC versions require -Wformat
diff --git a/coregrind/m_syswrap/syswrap-xen.c b/coregrind/m_syswrap/syswrap-xen.c
index 1e374ab..54153ab 100644
--- a/coregrind/m_syswrap/syswrap-xen.c
+++ b/coregrind/m_syswrap/syswrap-xen.c
@@ -1279,7 +1279,7 @@ PRE(domctl)
       switch (domctl->interface_version) {
       case 0x000000b:
           if (domctl->u.monitor_op_0000000b.op == VKI_XEN_DOMCTL_MONITOR_OP_ENABLE ||
-              domctl->u.monitor_op_0000000b.op == VKI_XEN_DOMCTL_MONITOR_OP_ENABLE) {
+              domctl->u.monitor_op_0000000b.op == VKI_XEN_DOMCTL_MONITOR_OP_DISABLE) {
              switch (domctl->u.monitor_op_0000000b.event) {
              case VKI_XEN_DOMCTL_MONITOR_EVENT_WRITE_CTRLREG:
                 __PRE_XEN_DOMCTL_READ(monitor_op, monitor_op_0000000b, u.mov_to_cr);