]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
hostip: include easy_lock.h before using GLOBAL_INIT_IS_THREADSAFE
authorDaniel Stenberg <daniel@haxx.se>
Tue, 16 May 2023 21:40:42 +0000 (23:40 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Wed, 17 May 2023 06:03:10 +0000 (08:03 +0200)
Since that header file is the only place that define can be defined.

Reported-by: Marc Deslauriers
Follow-up to 13718030ad4b3209

Closes #11121

lib/hostip.c
lib/hostip.h

index 615f36c16f771aa0847a4e32167df5d66b460ab9..4ff348436be158191959ad7665f666e53916b395 100644 (file)
@@ -70,6 +70,8 @@
 #include <SystemConfiguration/SCDynamicStoreCopySpecific.h>
 #endif
 
+#include "easy_lock.h"
+
 #if defined(CURLRES_SYNCH) &&                   \
   defined(HAVE_ALARM) &&                        \
   defined(SIGALRM) &&                           \
 #define USE_ALARM_TIMEOUT
 #endif
 
-#ifdef USE_ALARM_TIMEOUT
-#include "easy_lock.h"
-#endif
-
 #define MAX_HOSTCACHE_LEN (255 + 7) /* max FQDN + colon + port number + zero */
 
 #define MAX_DNS_CACHE_SIZE 29999
@@ -289,8 +287,8 @@ void Curl_hostcache_prune(struct Curl_easy *data)
 /* Beware this is a global and unique instance. This is used to store the
    return address that we can jump back to from inside a signal handler. This
    is not thread-safe stuff. */
-sigjmp_buf curl_jmpenv;
-curl_simple_lock curl_jmpenv_lock;
+static sigjmp_buf curl_jmpenv;
+static curl_simple_lock curl_jmpenv_lock;
 #endif
 
 /* lookup address, returns entry if found and not stale */
index 4b5481f65979ec5304482bc4269bebcfcef277e8..0dd19e87ccde27e98f2d88aec659b8ebe0a6ff0c 100644 (file)
@@ -186,15 +186,6 @@ Curl_cache_addr(struct Curl_easy *data, struct Curl_addrinfo *addr,
 #define CURL_INADDR_NONE INADDR_NONE
 #endif
 
-#ifdef HAVE_SIGSETJMP
-/* Forward-declaration of variable defined in hostip.c. Beware this
- * is a global and unique instance. This is used to store the return
- * address that we can jump back to from inside a signal handler.
- * This is not thread-safe stuff.
- */
-extern sigjmp_buf curl_jmpenv;
-#endif
-
 /*
  * Function provided by the resolver backend to set DNS servers to use.
  */