Commit 91e27ce7 authored by Daniel Stenberg's avatar Daniel Stenberg
Browse files

Fixed a TFTP memory leak. Enabled test 2003 to verify this.

parent 65ed6966

CHANGES

+5 −0
Original line number Diff line number Diff line
@@ -6,6 +6,11 @@ 


                                  Changelog



Daniel S (25 October 2007)

- Fixed a few memory leaks when the same easy handle is re-used to request

  URLs with different protocols. FTP and TFTP related leaks. Caught thanks to

  Dan F's new test cases.



Dan F (24 October 2007)

- Fixed the test FTP and TFTP servers to support the >10000 test number

  notation

RELEASE-NOTES

+1 −1
Original line number Diff line number Diff line
@@ -40,7 +40,7 @@ This release includes the following bugfixes: 
 o CURLOPT_POSTFIELDS could fail to send binary data

 o specifying a proxy with a trailing slash didn't work (unless it also

   contained a port number)

 o redirect from HTTP to FTP memory problem

 o redirect from HTTP to FTP or TFTP memory problems and leaks

 o re-used connections a bit too much when using non-SSL protocols tunneled

   over a HTTP proxy

 o embed the manifest in VC8 builds

lib/tftp.c

+5 −1
Original line number Diff line number Diff line
@@ -608,6 +608,10 @@ static CURLcode Curl_tftp_connect(struct connectdata *conn, bool *done) 
  tftp_state_data_t *state;

  int rc;



  /* If there already is a protocol-specific struct allocated for this

     sessionhandle, deal with it */

  Curl_reset_reqproto(conn);



  state = conn->data->reqdata.proto.tftp = calloc(sizeof(tftp_state_data_t),

                                                  1);

  if(!state)

tests/data/DISABLED

+0 −1
Original line number Diff line number Diff line
@@ -4,4 +4,3 @@ 
# per line.

# Lines starting with '#' letters are treated as comments.

#1009
 
2003