From: Ray Satiro <raysatiro@yahoo.com>
Date: Mon, 9 Jun 2014 21:57:14 +0000 (+0200)
Subject: progress callback: skip last callback update on errors
X-Git-Tag: curl-7_37_1~43
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=907520c4b93616bddea15757bbf0bfb45cde8101;p=thirdparty%2Fcurl.git

progress callback: skip last callback update on errors

When an error has been detected, skip the final forced call to the
progress callback by making sure to pass the current return code
variable in the Curl_done() call in the CURLM_STATE_DONE state.

This avoids the "extra" callback that could occur even if you returned
error from the progress callback.

Bug: http://curl.haxx.se/mail/lib-2014-06/0062.html
Reported by: Jonathan Cardoso Machado
---

diff --git a/lib/multi.c b/lib/multi.c
index 1fb341ca6f..ca975a0565 100644
--- a/lib/multi.c
+++ b/lib/multi.c
@@ -1610,7 +1610,7 @@ static CURLMcode multi_runsingle(struct Curl_multi *multi,
         Curl_multi_process_pending_handles(multi);
 
         /* post-transfer command */
-        res = Curl_done(&data->easy_conn, CURLE_OK, FALSE);
+        res = Curl_done(&data->easy_conn, data->result, FALSE);
 
         /* allow a previously set error code take precedence */
         if(!data->result)
diff --git a/lib/url.c b/lib/url.c
index 770b0cc95e..5209c8dce1 100644
--- a/lib/url.c
+++ b/lib/url.c
@@ -5796,9 +5796,9 @@ CURLcode Curl_done(struct connectdata **connp,
   if(conn->handler->done)
     result = conn->handler->done(conn, status, premature);
   else
-    result = CURLE_OK;
+    result = status;
 
-  if(Curl_pgrsDone(conn) && !result)
+  if(!result && Curl_pgrsDone(conn))
     result = CURLE_ABORTED_BY_CALLBACK;
 
   /* if the transfer was completed in a paused state there can be buffered