]> granicus.if.org Git - curl/commitdiff
ftp: don't access pointer before NULL check
authorDaniel Stenberg <daniel@haxx.se>
Sat, 22 Sep 2018 20:30:02 +0000 (22:30 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Sun, 23 Sep 2018 21:08:36 +0000 (23:08 +0200)
Detected by Coverity. CID 1439611.

Follow-up from 46e164069d1a523

lib/ftp.c

index 270b1e5f0db845417828b8d368fad7829435f185..a9668170c1d40c0dad1f7e7e63f2f033003f1155 100644 (file)
--- a/lib/ftp.c
+++ b/lib/ftp.c
@@ -3142,7 +3142,6 @@ static CURLcode ftp_done(struct connectdata *conn, CURLcode status,
   int ftpcode;
   CURLcode result = CURLE_OK;
   char *path = NULL;
-  const char *path_to_use = ftp->path;
 
   if(!ftp)
     return CURLE_OK;
@@ -3194,7 +3193,7 @@ static CURLcode ftp_done(struct connectdata *conn, CURLcode status,
 
   if(!result)
     /* get the "raw" path */
-    result = Curl_urldecode(data, path_to_use, 0, &path, NULL, TRUE);
+    result = Curl_urldecode(data, ftp->path, 0, &path, NULL, TRUE);
   if(result) {
     /* We can limp along anyway (and should try to since we may already be in
      * the error path) */