diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2021-01-05 00:03:57 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2021-01-06 00:01:10 +0100 |
| commit | d7c0e82149d40994d8234239cf4ec8c65f5196f0 (patch) | |
| tree | c62c2db23a812fc96542b166929db2704d6b48ae | |
| parent | 76cb8a4201279d090c32e6ed717b48f64f2f1aa0 (diff) | |
| download | curl-bagder/global-cleanup.tar.gz | |
global_init: debug builds allocates a byte in initbagder/global-cleanup
... to make build tools/valgrind warn if no curl_global_cleanup is
called.
This is conditionally only done for debug builds with the env variable
CURL_GLOBAL_INIT set.
Closes #6410
| -rw-r--r-- | lib/easy.c | 15 | ||||
| -rwxr-xr-x | tests/runtests.pl | 1 |
2 files changed, 15 insertions, 1 deletions
diff --git a/lib/easy.c b/lib/easy.c index 311946bdf..04e59144b 100644 --- a/lib/easy.c +++ b/lib/easy.c @@ -5,7 +5,7 @@ * | (__| |_| | _ <| |___ * \___|\___/|_| \_\_____| * - * Copyright (C) 1998 - 2020, Daniel Stenberg, <daniel@haxx.se>, et al. + * Copyright (C) 1998 - 2021, 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 @@ -124,6 +124,10 @@ curl_wcsdup_callback Curl_cwcsdup = (curl_wcsdup_callback)_wcsdup; # pragma warning(default:4232) /* MSVC extension, dllimport identity */ #endif +#ifdef DEBUGBUILD +static char *leakpointer; +#endif + /** * curl_global_init() globally initializes curl given a bitwise set of the * different features of what to initialize. @@ -190,6 +194,12 @@ static CURLcode global_init(long flags, bool memoryfuncs) init_flags = flags; +#ifdef DEBUGBUILD + if(getenv("CURL_GLOBAL_INIT")) + /* alloc data that will leak if *cleanup() is not called! */ + leakpointer = malloc(1); +#endif + return CURLE_OK; fail: @@ -265,6 +275,9 @@ void curl_global_cleanup(void) #ifdef USE_WOLFSSH (void)wolfSSH_Cleanup(); #endif +#ifdef DEBUGBUILD + free(leakpointer); +#endif init_flags = 0; } diff --git a/tests/runtests.pl b/tests/runtests.pl index 6c5edc84d..0e0e26c3c 100755 --- a/tests/runtests.pl +++ b/tests/runtests.pl @@ -381,6 +381,7 @@ if (!$USER) { $ENV{'CURL_MEMDEBUG'} = $memdump; $ENV{'CURL_ENTROPY'}="12345678"; $ENV{'CURL_FORCETIME'}=1; # for debug NTLM magic +$ENV{'CURL_GLOBAL_INIT'}=1; # debug curl_global_init/cleanup use $ENV{'HOME'}=$pwd; $ENV{'COLUMNS'}=79; # screen width! |