Thread (1 message) 1 message, 1 author, 2019-08-26

Re: [PATCH] http: don't leak urlmatch_config.vars

From: Junio C Hamano <hidden>
Date: 2019-08-26 18:10:01

Mike Hommey [off-list ref] writes:
Signed-off-by: Mike Hommey <redacted>
---
 http.c | 1 +
 1 file changed, 1 insertion(+)
This matches what builtin/config.c::get_urlmatch() does, which makes
sense.

Thanks.
quoted hunk
diff --git a/http.c b/http.c
index 27aa0a3192..9e33584f2d 100644
--- a/http.c
+++ b/http.c
@@ -1073,6 +1073,7 @@ void http_init(struct remote *remote, const char *url, int proactive_auth)
 
 	git_config(urlmatch_config_entry, &config);
 	free(normalized_url);
+	string_list_clear(&config.vars, 1);
 
 #if LIBCURL_VERSION_NUM >= 0x073800
 	if (http_ssl_backend) {
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help