generic-library/curl
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

- Christian Krause reported and fixed a memory leak that would occur with HTTP

Browse Source 
GSS/kerberos authentication (http://curl.haxx.se/bug/view.cgi?id=2284386)
This commit is contained in:
Daniel Stenberg 2008-11-19 14:22:01 +00:00
parent 797bc8504c
commit 4b3ae5e157
3 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
CHANGES
View File

@ -7,6 +7,9 @@
Changelog Changelog
Daniel Stenberg (19 Nov 2008) Daniel Stenberg (19 Nov 2008)
- Christian Krause reported and fixed a memory leak that would occur with HTTP
GSS/kerberos authentication (http://curl.haxx.se/bug/view.cgi?id=2284386)
- Andreas Wurf and Markus Koetter helped me analyze a problem that Andreas got - Andreas Wurf and Markus Koetter helped me analyze a problem that Andreas got
when uploading files to a single FTP server using multiple easy handle when uploading files to a single FTP server using multiple easy handle
handles with the multi interface. Occasionally a handle would stall in handles with the multi interface. Occasionally a handle would stall in

1
RELEASE-NOTES
View File

@ -19,6 +19,7 @@ This release includes the following bugfixes:
used used
o re-use of connections with the multi interface when multiple handles used o re-use of connections with the multi interface when multiple handles used
the same server the same server
o memory leak with HTTP GSS/kerberos authentication
This release includes the following known bugs: This release includes the following known bugs:

4
lib/http.c
View File

@ -458,6 +458,10 @@ CURLcode Curl_http_auth_act(struct connectdata *conn)
} }
if(pickhost || pickproxy) { if(pickhost || pickproxy) {
/* In case this is GSS auth, the newurl field is already allocated so
we must make sure to free it before allocating a new one. As figured
out in bug #2284386 */
Curl_safefree(data->req.newurl);
data->req.newurl = strdup(data->change.url); /* clone URL */ data->req.newurl = strdup(data->change.url); /* clone URL */
if(!data->req.newurl) if(!data->req.newurl)
return CURLE_OUT_OF_MEMORY; return CURLE_OUT_OF_MEMORY;