[Valgrind-developers] [valgrind] Bug 513522 - m_libcassert.c: 'ordered comparison of pointer with integer zero' compiler warning

[Paul F. <pa...@so...>]

https://sourceware.org/cgit/valgrind/commit/?id=6f1b1f3d7d00f45bacebc2bfae667fc0ff8a461b

commit 6f1b1f3d7d00f45bacebc2bfae667fc0ff8a461b
Author: Paul Floyd <pj...@wa...>
Date:   Thu Dec 18 07:59:38 2025 +0100

    Bug 513522 - m_libcassert.c: 'ordered comparison of pointer with integer zero' compiler warning

Diff:
---
 NEWS                     | 2 ++
 coregrind/m_libcassert.c | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/NEWS b/NEWS
index 06bf59ebbf..b8e15e0df2 100644
--- a/NEWS
+++ b/NEWS
@@ -46,6 +46,8 @@ are not entered into bugzilla tend to get forgotten about or ignored.
 512037  malloc trace does not print free size or alignment
 512571  regtest problems with darwin dsymuti
 513257  Add missing syswraps for lsm_list_modules
+513522  m_libcassert.c: 'ordered comparison of pointer with integer zero'
+        compiler warning
 
 To see details of a given bug, visit
   https://bugs.kde.org/show_bug.cgi?id=XXXXXX
diff --git a/coregrind/m_libcassert.c b/coregrind/m_libcassert.c
index 35a988c65d..e757aa59e1 100644
--- a/coregrind/m_libcassert.c
+++ b/coregrind/m_libcassert.c
@@ -500,7 +500,7 @@ static void report_and_quit ( const HChar* report,
                           False, // exited_threads
                           startRegsIN);
 
-   if (VG_(debugLog_getLevel) > 0) {
+   if (VG_(debugLog_getLevel)() > 0) {
       VG_(am_show_nsegments) (1, "report_and_quit");
       (void) VG_(am_do_sync_check) ("report_and_quit", __FILE__, __LINE__);
    }