test-segfault: Fix build error caused by a null pointer dereference warning

Only do the deliberate crash via undefined behaviour (which the compiler
is quite right to warn us about!) if raise() isn't available.

The pointer needs to be volatile otherwise the compiler is free to remove
the store.

Part-of: https://gitlab.freedesktop.org/dbus/dbus/-/merge_requests/275
Reviewed-by: Simon McVittie <smcv@collabora.com>

diff --git a/test/test-segfault.c b/test/test-segfault.c
index 5c559ccb9a0e4ec027fd32526141d8d0b24aa103..ef905c58d198d925b49ae3ebd578ae577408c545 100644 (file)
--- a/test/test-segfault.c
+++ b/test/test-segfault.c
@@ -10,15 +10,16 @@
 int
 main (int argc, char **argv)
 {
-  char *p;  
-
   _dbus_disable_crash_handling ();
 
 #ifdef HAVE_RAISE
   raise (SIGSEGV);
+#else
+  {
+    volatile char *p = NULL;
+    *p = 'a';
+  }
 #endif
-  p = NULL;
-  *p = 'a';
-  
+
   return 0;
 }