Eliminate 'Condition 0==datastart always false' cppcheck warning (dyn_load)

* dyn_load.c [HAVE_DL_ITERATE_PHDR]
(GC_register_dynamic_libraries_dl_iterate_phdr): Cast datastart to
char*volatile* before dereferencing and checking against NULL.

diff --git a/dyn_load.c b/dyn_load.c
index 836e4bcebfb5c6d348bb172d23e738103351e192..42534c1ec29e6c143a59c9ac37c99ddc7648c82f 100644 (file)
--- a/dyn_load.c
+++ b/dyn_load.c
@@ -621,7 +621,8 @@ STATIC GC_bool GC_register_dynamic_libraries_dl_iterate_phdr(void)
 #     else
         dataend = DATAEND;
 #     endif
-      if (NULL == datastart || (word)datastart > (word)dataend)
+      if (NULL == *(char * volatile *)&datastart
+          || (word)datastart > (word)dataend)
         ABORT_ARG2("Wrong DATASTART/END pair",
                    ": %p .. %p", (void *)datastart, (void *)dataend);