diff options
author | Daniel Stenberg <daniel@haxx.se> | 2016-04-29 15:46:40 +0200 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2016-04-29 22:32:49 +0200 |
commit | 4f45240bc84a9aa648c8f7243be7b79e9f9323a5 (patch) | |
tree | 975f7e1e617df32c42db93cf0cbacfd10ff640c4 /lib/socks_gssapi.c | |
parent | 63b5378a617437859da1e7ab5553d36eb83bdd44 (diff) | |
download | curl-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/socks_gssapi.c')
-rw-r--r-- | lib/socks_gssapi.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/socks_gssapi.c b/lib/socks_gssapi.c index e3fc260cb..1214048bf 100644 --- a/lib/socks_gssapi.c +++ b/lib/socks_gssapi.c @@ -6,7 +6,7 @@ * \___|\___/|_| \_\_____| * * Copyright (C) 2009, 2011, Markus Moeller, <markus_moeller@compuserve.com> - * Copyright (C) 2012 - 2015, Daniel Stenberg, <daniel@haxx.se>, et al. + * Copyright (C) 2012 - 2016, Daniel Stenberg, <daniel@haxx.se>, et al. * * This software is licensed as described in the file COPYING, which * you should have received as part of this distribution. The terms @@ -32,9 +32,10 @@ #include "timeval.h" #include "socks.h" #include "warnless.h" + +/* The last 3 #include files should be in this order */ #include "curl_printf.h" #include "curl_memory.h" -/* The last #include file should be: */ #include "memdebug.h" static gss_ctx_id_t gss_context = GSS_C_NO_CONTEXT; |