poison: added valgrind support

If the header valgrind/memcheck.h is available, add VALGRIND_MAKE_MEM_NOACCESS() and VALGRIND_MAKE_MEM_UNDEFINED() support, which enables nice warnings in the valgrind memory checker.

poison: added valgrind support
940af669 · Max Kellermann · fd76e29f · 940af669 · 940af669
Commit 940af669 authored Mar 09, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 15 additions and 0 deletions

configure.ac configure.ac +1 -0

poison.h src/poison.h +14 -0

No files found.
--- a/configure.ac
+++ b/configure.ac
@@ -101,6 +101,7 @@ AC_CHECK_LIB(nsl,gethostbyname,MPD_LIBS="$MPD_LIBS -lnsl",)
 AC_CHECK_LIB(m,exp,MPD_LIBS="$MPD_LIBS -lm",)

 AC_CHECK_HEADERS(locale.h)
+AC_CHECK_HEADERS(valgrind/memcheck.h)


 dnl

--- a/src/poison.h
+++ b/src/poison.h
@@ -20,6 +20,12 @@
 #define MPD_POISON_H

 #ifndef NDEBUG
+#include "config.h"
+
+#ifdef HAVE_VALGRIND_MEMCHECK_H
+#include <valgrind/memcheck.h>
+#endif
+
 #include <string.h>
 #endif

@@ -37,6 +43,10 @@ poison_noaccess(void *p, size_t length)
 	(void)length;
 #else
 	memset(p, 0x01, length);
+
+#ifdef HAVE_VALGRIND_MEMCHECK_H
+	VALGRIND_MAKE_MEM_NOACCESS(p, length);
+#endif
 #endif
 }

@@ -54,6 +64,10 @@ poison_undefined(void *p, size_t length)
 	(void)length;
 #else
 	memset(p, 0x02, length);
+
+#ifdef HAVE_VALGRIND_MEMCHECK_H
+	VALGRIND_MAKE_MEM_UNDEFINED(p, length);
+#endif
 #endif
 }