From cee0fb3247d83d8c07eba62f539c8c0609609c43 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Fri, 2 Dec 2016 11:25:35 +0100 Subject: curl: support zero-length argument strings in config files ... like 'user-agent = ""' Adjusted test 71 to verify. --- src/tool_parsecfg.c | 14 +++++--------- tests/data/test71 | 4 ++-- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/src/tool_parsecfg.c b/src/tool_parsecfg.c index d9454c23a..7cf11d8a4 100644 --- a/src/tool_parsecfg.c +++ b/src/tool_parsecfg.c @@ -5,7 +5,7 @@ * | (__| |_| | _ <| |___ * \___|\___/|_| \_\_____| * - * Copyright (C) 1998 - 2015, Daniel Stenberg, , et al. + * Copyright (C) 1998 - 2016, Daniel Stenberg, , et al. * * This software is licensed as described in the file COPYING, which * you should have received as part of this distribution. The terms @@ -209,14 +209,10 @@ int parseconfig(const char *filename, struct GlobalConfig *global) filename, lineno, option); } } - } - - if(!*param) { - /* do this so getparameter can check for required parameters. - Otherwise it always thinks there's a parameter. */ - if(alloced_param) - Curl_safefree(param); - param = NULL; + if(!*param) + /* do this so getparameter can check for required parameters. + Otherwise it always thinks there's a parameter. */ + param = NULL; } #ifdef DEBUG_CONFIG diff --git a/tests/data/test71 b/tests/data/test71 index b71f969e7..341a0033f 100644 --- a/tests/data/test71 +++ b/tests/data/test71 @@ -31,6 +31,7 @@ HTTP and -F upload in config file -F name=daniel -F tool=curl -F file=@log/test71.txt +user-agent = "" http://%HOSTIP:%HTTPPORT/we/want/71 -K - @@ -46,11 +47,10 @@ bar # Verify data after the test has been "shot" -^(User-Agent:|Content-Type: multipart/form-data;|------------).* +^(Content-Type: multipart/form-data;|------------).* POST /we/want/71 HTTP/1.1 -User-Agent: curl/7.10.4 (i686-pc-linux-gnu) libcurl/7.10.4 OpenSSL/0.9.7a ipv6 zlib/1.1.3 Host: %HOSTIP:%HTTPPORT Accept: */* Content-Length: 408 -- cgit v1.2.3