summaryrefslogtreecommitdiff
path: root/lib/memdebug.c
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2016-04-29 15:46:40 +0200
committerDaniel Stenberg <daniel@haxx.se>2016-04-29 22:32:49 +0200
commit4f45240bc84a9aa648c8f7243be7b79e9f9323a5 (patch)
tree975f7e1e617df32c42db93cf0cbacfd10ff640c4 /lib/memdebug.c
parent63b5378a617437859da1e7ab5553d36eb83bdd44 (diff)
downloadcurl-4f45240bc84a9aa648c8f7243be7b79e9f9323a5.tar.gz
lib: include curl_printf.h as one of the last headers
curl_printf.h defines printf to curl_mprintf, etc. This can cause problems with external headers which may use __attribute__((format(printf, ...))) markers etc. To avoid that they cause problems with system includes, we include curl_printf.h after any system headers. That makes the three last headers to always be, and we keep them in this order: curl_printf.h curl_memory.h memdebug.h None of them include system headers, they all do funny #defines. Reported-by: David Benjamin Fixes #743
Diffstat (limited to 'lib/memdebug.c')
-rw-r--r--lib/memdebug.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/memdebug.c b/lib/memdebug.c
index 804b5fa34..1618bbaf3 100644
--- a/lib/memdebug.c
+++ b/lib/memdebug.c
@@ -26,10 +26,12 @@
#include <curl/curl.h>
-#include "curl_printf.h"
#include "urldata.h"
#define MEMDEBUG_NODEFINES /* don't redefine the standard functions */
+
+/* The last 3 #include files should be in this order */
+#include "curl_printf.h"
#include "curl_memory.h"
#include "memdebug.h"