From def2ca2628fff1973f558e45dd8fe84ca00b54d5 Mon Sep 17 00:00:00 2001 From: Patrick Monnerat Date: Mon, 13 Nov 2017 14:20:41 +0100 Subject: [PATCH] zlib/brotli: only include header files in modules needing them There is a conflict on symbol 'free_func' between openssl/crypto.h and zlib.h on AIX. This is an attempt to resolve it. Bug: https://curl.haxx.se/mail/lib-2017-11/0032.html Reported-By: Michael Felt --- lib/content_encoding.c | 13 +++++++++++++ lib/urldata.h | 12 ------------ lib/version.c | 12 ++++++++++++ 3 files changed, 25 insertions(+), 12 deletions(-) diff --git a/lib/content_encoding.c b/lib/content_encoding.c index 626bb8ec9..3d061375c 100644 --- a/lib/content_encoding.c +++ b/lib/content_encoding.c @@ -25,6 +25,19 @@ #include "urldata.h" #include #include + +#ifdef HAVE_ZLIB_H +#include +#ifdef __SYMBIAN32__ +/* zlib pollutes the namespace with this definition */ +#undef WIN32 +#endif +#endif + +#ifdef HAVE_BROTLI +#include +#endif + #include "sendf.h" #include "http.h" #include "content_encoding.h" diff --git a/lib/urldata.h b/lib/urldata.h index 7dfb26b6d..94f692223 100644 --- a/lib/urldata.h +++ b/lib/urldata.h @@ -88,18 +88,6 @@ #include "timeval.h" -#ifdef HAVE_ZLIB_H -#include /* for content-encoding */ -#ifdef __SYMBIAN32__ -/* zlib pollutes the namespace with this definition */ -#undef WIN32 -#endif -#endif - -#ifdef HAVE_BROTLI -#include -#endif - #include #include "http_chunks.h" /* for the structs and enum stuff */ diff --git a/lib/version.c b/lib/version.c index 66c761e34..79cc0bba4 100644 --- a/lib/version.c +++ b/lib/version.c @@ -64,6 +64,18 @@ #define CURL_LIBSSH2_VERSION LIBSSH2_VERSION #endif +#ifdef HAVE_ZLIB_H +#include +#ifdef __SYMBIAN32__ +/* zlib pollutes the namespace with this definition */ +#undef WIN32 +#endif +#endif + +#ifdef HAVE_BROTLI +#include +#endif + void Curl_version_init(void); /* For thread safety purposes this function is called by global_init so that