summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2021-01-27 16:39:24 +0100
committerDaniel Stenberg <daniel@haxx.se>2021-01-28 07:55:25 +0100
commit553f286a8f17e84b6957101ab37b7d817a6a2929 (patch)
treec762ef54ad971fab97717e1bfff985a6e31f6819
parent3611f6a41e3059654085307fa5a3011912f255f6 (diff)
downloadcurl-553f286a8f17e84b6957101ab37b7d817a6a2929.tar.gz
tool_writeout: make %{errormsg} blank for no errors
Closes #6539
-rw-r--r--src/tool_writeout.c7
-rw-r--r--tests/data/test10292
2 files changed, 5 insertions, 4 deletions
diff --git a/src/tool_writeout.c b/src/tool_writeout.c
index 9ce0a997e..c548bd8f2 100644
--- a/src/tool_writeout.c
+++ b/src/tool_writeout.c
@@ -5,7 +5,7 @@
* | (__| |_| | _ <| |___
* \___|\___/|_| \_\_____|
*
- * Copyright (C) 1998 - 2020, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 1998 - 2021, Daniel Stenberg, <daniel@haxx.se>, et al.
*
* This software is licensed as described in the file COPYING, which
* you should have received as part of this distribution. The terms
@@ -137,8 +137,9 @@ void ourWriteOut(CURL *curl, struct per_transfer *per, const char *writeinfo,
fprintf(stream, "%d", (int)result);
break;
case VAR_ERRORMSG:
- fputs(per->errorbuffer[0] ? per->errorbuffer :
- curl_easy_strerror(result), stream);
+ if(result)
+ fputs(per->errorbuffer[0] ? per->errorbuffer :
+ curl_easy_strerror(result), stream);
break;
case VAR_INPUT_URL:
if(per->this_url)
diff --git a/tests/data/test1029 b/tests/data/test1029
index 36797e334..b1f5982e4 100644
--- a/tests/data/test1029
+++ b/tests/data/test1029
@@ -50,7 +50,7 @@ Content-Length: 62
Connection: close
This server reply is for testing a simple Location: following
-http://%HOSTIP:%HTTPPORT/we/want/our/data/10290002.txt?coolsite=yes http://%HOSTIP:%HTTPPORT/we/want/our/1029 0 No error
+http://%HOSTIP:%HTTPPORT/we/want/our/data/10290002.txt?coolsite=yes http://%HOSTIP:%HTTPPORT/we/want/our/1029 0
</stdout>
</verify>
</testcase>