Bugs item #1780194, was opened at 2007-08-23 15:04
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1780194&group_id=976

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: libcurl
Group: wrong behaviour
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Ivosh (iraisr)
Assigned to: Daniel Stenberg (bagder)
Summary: memory leak if proxy empty string

Initial Comment:
In libcurl version 7.16.2, there is a memory leak in CreateConnection function (url.c module) if data->set.proxy is an empty string.

On line 2969, proxy gets its content via strdup().
On line 3365 the test fails because (proxy != NULL) && (*proxy == '\0').
But proxy is not freed anywhere below.
I fixed the leak by adding the following at line 3478:
  else if (proxy != NULL) {
      free(proxy);
      proxy = NULL;
  }

But I think better logic will be not to call strdup for proxy at all. In this case also logic on line 2982 needs to be altered (because data->set.proxy specified an empty proxy).

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1780194&group_id=976
Received on 2007-08-23