Trivial fix to check for NULL pointer prior to strncmp

The MSG_MAGICAL() macro needs to check for a NULL pointer
prior to calling strncmp on it.

Signed-off-by: Dmitry Torokhov <dtor@vmware.com>

diff --git a/open-vm-tools/lib/include/msgid.h b/open-vm-tools/lib/include/msgid.h
index 59503d1b8764ed48fee5f29d23fa44925fb5389d..4355c3365c555a8c2b4d897c52286cdaaf697279 100644 (file)
--- a/open-vm-tools/lib/include/msgid.h
+++ b/open-vm-tools/lib/include/msgid.h
@@ -39,7 +39,8 @@
 
 
 // the X hides MSG_MAGIC so it won't appear in the object file
-#define MSG_MAGICAL(s) (strncmp(s, MSG_MAGIC"X", MSG_MAGIC_LEN) == 0)
+#define MSG_MAGICAL(s) \
+   (s != NULL && strncmp(s, MSG_MAGIC"X", MSG_MAGIC_LEN) == 0)
 
 // Start after MSG_MAGIC so it won't appear in the object file either.
 #define MSG_HAS_BUTTONID(s) \