tests: Use volatile pointer for NULL dereference

Clang warns that it can silently discard a non-volatile write to a NULL
pointer (perhaps it constitutes undefined behaviour?), and recommends
changing it to volatile.

This patch slavishly complies with the demand of the unfeeling machine.

Signed-off-by: Daniel Stone <daniels@collabora.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
diff --git a/tests/sanity-test.c b/tests/sanity-test.c
index 66ca16f..2495a11 100644
--- a/tests/sanity-test.c
+++ b/tests/sanity-test.c
@@ -70,8 +70,10 @@
 
 FAIL_TEST(fail_segv)
 {
+	char * volatile *null = 0;
+
 	test_disable_coredumps();
-	* (char **) 0 = "Goodbye, world";
+	*null = "Goodbye, world";
 }
 
 FAIL_TEST(sanity_assert)