Skip to content
Permalink
Browse files

url: always clone the CUROPT_CURLU handle

Since a few code paths actually update that data.

Fixes #3753
Closes #3761

Reported-by: Poul T Lomholt
  • Loading branch information...
bagder committed Apr 11, 2019
1 parent f73de32 commit 9a4ad1b0567ec66876f853f2127ceaa799394bcf
Showing with 2 additions and 3 deletions.
  1. +2 −3 lib/url.c
@@ -303,8 +303,7 @@ static void up_free(struct Curl_easy *data)
Curl_safefree(up->options);
Curl_safefree(up->path);
Curl_safefree(up->query);
if(data->set.uh != data->state.uh)
curl_url_cleanup(data->state.uh);
curl_url_cleanup(data->state.uh);
data->state.uh = NULL;
}

@@ -1879,7 +1878,7 @@ static CURLcode parseurlandfillconn(struct Curl_easy *data,

/* parse the URL */
if(data->set.uh) {
uh = data->state.uh = data->set.uh;
uh = data->state.uh = curl_url_dup(data->set.uh);
}
else {
uh = data->state.uh = curl_url();

0 comments on commit 9a4ad1b

Please sign in to comment.
You can’t perform that action at this time.