From: Daniel Stenberg <daniel@haxx.se>
Date: Thu, 24 Apr 2025 21:39:08 +0000 (+0200)
Subject: aws-sigv4: allow a blank string
X-Git-Tag: curl-8_14_0~218
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=2ab6b9d40526cbe8f7e0dc0e26acb42e6dd95e24;p=thirdparty%2Fcurl.git

aws-sigv4: allow a blank string

make sure a zero length sigv4 gets the default value

Reported-by: Arian van Putten
Fixes #17176
Closes #17178
---

diff --git a/lib/http_aws_sigv4.c b/lib/http_aws_sigv4.c
index c255ddcb0c..7047138279 100644
--- a/lib/http_aws_sigv4.c
+++ b/lib/http_aws_sigv4.c
@@ -676,8 +676,9 @@ CURLcode Curl_output_aws_sigv4(struct Curl_easy *data)
    * AWS is the default because most of non-amazon providers
    * are still using aws:amz as a prefix.
    */
-  line = data->set.str[STRING_AWS_SIGV4] ?
-    data->set.str[STRING_AWS_SIGV4] : "aws:amz";
+  line = data->set.str[STRING_AWS_SIGV4];
+  if(!line || !*line)
+    line = "aws:amz";
 
   /* provider0[:provider1[:region[:service]]]
 
diff --git a/src/tool_getparam.c b/src/tool_getparam.c
index e64239e5fa..e1d2864975 100644
--- a/src/tool_getparam.c
+++ b/src/tool_getparam.c
@@ -2005,7 +2005,7 @@ ParameterError getparameter(const char *flag, /* f or -long-flag */
       break;
     case C_AWS_SIGV4: /* --aws-sigv4 */
       config->authtype |= CURLAUTH_AWS_SIGV4;
-      err = getstr(&config->aws_sigv4, nextarg, DENY_BLANK);
+      err = getstr(&config->aws_sigv4, nextarg, ALLOW_BLANK);
       break;
     case C_STDERR: /* --stderr */
       tool_set_stderr_file(global, nextarg);