diff options
author | Dan Fandrich <dan@coneharvesters.com> | 2008-09-02 23:12:00 +0000 |
---|---|---|
committer | Dan Fandrich <dan@coneharvesters.com> | 2008-09-02 23:12:00 +0000 |
commit | 29ec219b82ba65fd4be0fd7c9d678eb9ae1890b7 (patch) | |
tree | 6f90fe82daf5d1b35a838c7156d3fac93313a5ce | |
parent | 0994d7811f691f0b5c6743ae8d3ce90310cd7cea (diff) |
Fixed an out of memory problem that caused torture test failures in tests
706 and 707.
-rw-r--r-- | CHANGES | 4 | ||||
-rw-r--r-- | lib/ftp.c | 10 |
2 files changed, 12 insertions, 2 deletions
@@ -6,6 +6,10 @@ Changelog +Daniel Fandrich (2 Sep 2008) +- Fixed an out of memory problem that caused torture test failures in tests + 706 and 707. + Daniel Stenberg (2 Sep 2008) - Keith Mok added supported_protocols and supported_features to the pkg-config file for libcurl, and while doing that fix he unified with curl-config.in @@ -1825,6 +1825,11 @@ static CURLcode ftp_state_pasv_resp(struct connectdata *conn, connectport = (unsigned short)conn->port; /* we connect to the proxy's port */ + if(!addr) { + failf(data, "Can't resolve proxy host %s:%d", + conn->proxy.name, connectport); + return CURLE_FTP_CANT_GET_HOST; + } } else { /* normal, direct, ftp connection */ @@ -1833,11 +1838,12 @@ static CURLcode ftp_state_pasv_resp(struct connectdata *conn, /* BLOCKING */ rc = Curl_wait_for_resolv(conn, &addr); + connectport = newport; /* we connect to the remote port */ + if(!addr) { - failf(data, "Can't resolve new host %s:%d", newhost, newport); + failf(data, "Can't resolve new host %s:%d", newhost, connectport); return CURLE_FTP_CANT_GET_HOST; } - connectport = newport; /* we connect to the remote port */ } result = Curl_connecthost(conn, |