Bugs item #1856628, was opened at 2007-12-23 02:04
Message generated for change (Comment added) made by bagder
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1856628&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: bad behaviour
>Status: Closed
>Resolution: Fixed
>Priority: 7
Private: No
Submitted By: Gary Maxwell (gmaxwell_447)
Assigned to: Daniel Stenberg (bagder)
Summary: Memory leak in Curl_ssl_addsessionid()

Initial Comment:
Found in 7.17.0, confirmed to still exist in the 22-Dec-2007 snapshot (7.17.2).

On line 387 of sslgen.c, in the Curl_ssl_addsessionid() function, "store->name" is assigned a new value without freeing the value already assigned, causing a minor memory leak.

We found this while performing continuous HTTPS testing, and noticed a small but ever-increasing memory leak, tracking it down to this part of the code where the fragment becomes lost.

Patch submitted.

Cheers!
Gary Maxwell
Casabi, Inc.

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

>Comment By: Daniel Stenberg (bagder)
Date: 2007-12-25 00:46

Message:
Logged In: YES
user_id=1110
Originator: NO

Thanks a lot for the report and fix, committed in CVS just now!

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

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