From: Daniel Stenberg <daniel@haxx.se>
Date: Thu, 25 Dec 2025 22:17:24 +0000 (+0100)
Subject: ftp: return better on OOM in two places
X-Git-Tag: rc-8_18_0-3~1
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=abcb10f3ac6e829b125cc0f0e5071ce56d511755;p=thirdparty%2Fcurl.git

ftp: return better on OOM in two places

Found with strict torture testing.

Closes #20099
---

diff --git a/lib/ftp.c b/lib/ftp.c
index 46a309873c..650c2ee992 100644
--- a/lib/ftp.c
+++ b/lib/ftp.c
@@ -1941,7 +1941,8 @@ static CURLcode ftp_state_pasv_resp(struct Curl_easy *data,
                            CURL_CF_SSL_ENABLE : CURL_CF_SSL_DISABLE);
 
   if(result) {
-    if(ftpc->count1 == 0 && ftpcode == 229) {
+    if((result != CURLE_OUT_OF_MEMORY) &&
+       (ftpc->count1 == 0) && (ftpcode == 229)) {
       curlx_free(newhost);
       return ftp_epsv_disable(data, ftpc, conn);
     }
@@ -3552,6 +3553,8 @@ static CURLcode ftp_do_more(struct Curl_easy *data, int *completep)
   if(conn->cfilter[SECONDARYSOCKET]) {
     bool is_eptr = Curl_conn_is_tcp_listen(data, SECONDARYSOCKET);
     result = Curl_conn_connect(data, SECONDARYSOCKET, FALSE, &connected);
+    if(result == CURLE_OUT_OF_MEMORY)
+      return result;
     if(result || (!connected && !is_eptr &&
                   !Curl_conn_is_ip_connected(data, SECONDARYSOCKET))) {
       if(result && !is_eptr && (ftpc->count1 == 0)) {