summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2018-08-15 09:17:43 +0200
committerDaniel Stenberg <daniel@haxx.se>2018-08-15 09:17:43 +0200
commite22a4a27c64d55b1003cdf2ebb8fd505f4bb7a8c (patch)
treec66e89ffba022734446a49315c989d657e2d721a
parent39cb7130c3d302f7447937cb4f23d73f92561d3b (diff)
downloadcurl-e22a4a27c64d55b1003cdf2ebb8fd505f4bb7a8c.tar.gz
curl: warn the user if a given file name looks like an option
... simply because this is usually a sign of the user having omitted the file name and the next option is instead "eaten" by the parser as a file name.
-rw-r--r--src/tool_getparam.c54
1 files changed, 31 insertions, 23 deletions
diff --git a/src/tool_getparam.c b/src/tool_getparam.c
index 1a81c3803..0093af0a3 100644
--- a/src/tool_getparam.c
+++ b/src/tool_getparam.c
@@ -65,7 +65,8 @@ struct LongShort {
enum {
ARG_NONE, /* stand-alone but not a boolean */
ARG_BOOL, /* accepts a --no-[name] prefix */
- ARG_STRING /* requires an argument */
+ ARG_STRING, /* requires an argument */
+ ARG_FILENAME /* requires an argument, usually a file name */
} desc;
};
@@ -75,7 +76,7 @@ static const struct LongShort aliases[]= {
{"*@", "url", ARG_STRING},
{"*4", "dns-ipv4-addr", ARG_STRING},
{"*6", "dns-ipv6-addr", ARG_STRING},
- {"*a", "random-file", ARG_STRING},
+ {"*a", "random-file", ARG_FILENAME},
{"*b", "egd-file", ARG_STRING},
{"*B", "oauth2-bearer", ARG_STRING},
{"*c", "connect-timeout", ARG_STRING},
@@ -87,9 +88,9 @@ static const struct LongShort aliases[]= {
/* 'epsv' made like this to make --no-epsv and --epsv to work
although --disable-epsv is the documented option */
{"*F", "dns-servers", ARG_STRING},
- {"*g", "trace", ARG_STRING},
+ {"*g", "trace", ARG_FILENAME},
{"*G", "npn", ARG_BOOL},
- {"*h", "trace-ascii", ARG_STRING},
+ {"*h", "trace-ascii", ARG_FILENAME},
{"*H", "alpn", ARG_BOOL},
{"*i", "limit-rate", ARG_STRING},
{"*j", "compressed", ARG_BOOL},
@@ -108,7 +109,7 @@ static const struct LongShort aliases[]= {
{"*s", "max-redirs", ARG_STRING},
{"*t", "proxy-ntlm", ARG_BOOL},
{"*u", "crlf", ARG_BOOL},
- {"*v", "stderr", ARG_STRING},
+ {"*v", "stderr", ARG_FILENAME},
{"*w", "interface", ARG_STRING},
{"*x", "krb", ARG_STRING},
{"*x", "krb4", ARG_STRING},
@@ -177,7 +178,7 @@ static const struct LongShort aliases[]= {
{"$J", "metalink", ARG_BOOL},
{"$K", "sasl-ir", ARG_BOOL},
{"$L", "test-event", ARG_BOOL},
- {"$M", "unix-socket", ARG_STRING},
+ {"$M", "unix-socket", ARG_FILENAME},
{"$N", "path-as-is", ARG_BOOL},
{"$O", "socks5-gssapi-service", ARG_STRING},
/* 'socks5-gssapi-service' merged with'proxy-service-name' and
@@ -188,7 +189,7 @@ static const struct LongShort aliases[]= {
{"$R", "expect100-timeout", ARG_STRING},
{"$S", "tftp-no-options", ARG_BOOL},
{"$U", "connect-to", ARG_STRING},
- {"$W", "abstract-unix-socket", ARG_STRING},
+ {"$W", "abstract-unix-socket", ARG_FILENAME},
{"$X", "tls-max", ARG_STRING},
{"$Y", "suppress-connect-headers", ARG_BOOL},
{"$Z", "compressed-ssh", ARG_BOOL},
@@ -219,19 +220,19 @@ static const struct LongShort aliases[]= {
{"da", "data-ascii", ARG_STRING},
{"db", "data-binary", ARG_STRING},
{"de", "data-urlencode", ARG_STRING},
- {"D", "dump-header", ARG_STRING},
+ {"D", "dump-header", ARG_FILENAME},
{"e", "referer", ARG_STRING},
- {"E", "cert", ARG_STRING},
- {"Ea", "cacert", ARG_STRING},
+ {"E", "cert", ARG_FILENAME},
+ {"Ea", "cacert", ARG_FILENAME},
{"Eb", "cert-type", ARG_STRING},
- {"Ec", "key", ARG_STRING},
+ {"Ec", "key", ARG_FILENAME},
{"Ed", "key-type", ARG_STRING},
{"Ee", "pass", ARG_STRING},
{"Ef", "engine", ARG_STRING},
- {"Eg", "capath", ARG_STRING},
+ {"Eg", "capath", ARG_FILENAME},
{"Eh", "pubkey", ARG_STRING},
{"Ei", "hostpubmd5", ARG_STRING},
- {"Ej", "crlfile", ARG_STRING},
+ {"Ej", "crlfile", ARG_FILENAME},
{"Ek", "tlsuser", ARG_STRING},
{"El", "tlspassword", ARG_STRING},
{"Em", "tlsauthtype", ARG_STRING},
@@ -246,17 +247,17 @@ static const struct LongShort aliases[]= {
{"Eu", "proxy-tlsuser", ARG_STRING},
{"Ev", "proxy-tlspassword", ARG_STRING},
{"Ew", "proxy-tlsauthtype", ARG_STRING},
- {"Ex", "proxy-cert", ARG_STRING},
+ {"Ex", "proxy-cert", ARG_FILENAME},
{"Ey", "proxy-cert-type", ARG_STRING},
- {"Ez", "proxy-key", ARG_STRING},
+ {"Ez", "proxy-key", ARG_FILENAME},
{"E0", "proxy-key-type", ARG_STRING},
{"E1", "proxy-pass", ARG_STRING},
{"E2", "proxy-ciphers", ARG_STRING},
- {"E3", "proxy-crlfile", ARG_STRING},
+ {"E3", "proxy-crlfile", ARG_FILENAME},
{"E4", "proxy-ssl-allow-beast", ARG_BOOL},
{"E5", "login-options", ARG_STRING},
- {"E6", "proxy-cacert", ARG_STRING},
- {"E7", "proxy-capath", ARG_STRING},
+ {"E6", "proxy-cacert", ARG_FILENAME},
+ {"E7", "proxy-capath", ARG_FILENAME},
{"E8", "proxy-insecure", ARG_BOOL},
{"E9", "proxy-tlsv1", ARG_NONE},
{"EA", "socks5-basic", ARG_BOOL},
@@ -277,7 +278,7 @@ static const struct LongShort aliases[]= {
{"j", "junk-session-cookies", ARG_BOOL},
{"J", "remote-header-name", ARG_BOOL},
{"k", "insecure", ARG_BOOL},
- {"K", "config", ARG_STRING},
+ {"K", "config", ARG_FILENAME},
{"l", "list-only", ARG_BOOL},
{"L", "location", ARG_BOOL},
{"Lt", "location-trusted", ARG_BOOL},
@@ -285,10 +286,10 @@ static const struct LongShort aliases[]= {
{"M", "manual", ARG_BOOL},
{"n", "netrc", ARG_BOOL},
{"no", "netrc-optional", ARG_BOOL},
- {"ne", "netrc-file", ARG_STRING},
+ {"ne", "netrc-file", ARG_FILENAME},
{"N", "buffer", ARG_BOOL},
/* 'buffer' listed as --no-buffer in the help */
- {"o", "output", ARG_STRING},
+ {"o", "output", ARG_FILENAME},
{"O", "remote-name", ARG_NONE},
{"Oa", "remote-name-all", ARG_BOOL},
{"p", "proxytunnel", ARG_BOOL},
@@ -300,7 +301,7 @@ static const struct LongShort aliases[]= {
{"s", "silent", ARG_BOOL},
{"S", "show-error", ARG_BOOL},
{"t", "telnet-option", ARG_STRING},
- {"T", "upload-file", ARG_STRING},
+ {"T", "upload-file", ARG_FILENAME},
{"u", "user", ARG_STRING},
{"U", "proxy-user", ARG_STRING},
{"v", "verbose", ARG_BOOL},
@@ -570,7 +571,7 @@ ParameterError getparameter(const char *flag, /* f or -long-flag */
}
}
- if(aliases[hit].desc == ARG_STRING) {
+ if(aliases[hit].desc >= ARG_STRING) {
/* this option requires an extra parameter */
if(!longopt && parse[1]) {
nextarg = (char *)&parse[1]; /* this is the actual extra parameter */
@@ -580,6 +581,13 @@ ParameterError getparameter(const char *flag, /* f or -long-flag */
return PARAM_REQUIRES_PARAMETER;
else
*usedarg = TRUE; /* mark it as used */
+
+ if((aliases[hit].desc == ARG_FILENAME) &&
+ (nextarg[0] == '-') && nextarg[1]) {
+ /* if the file name looks like a command line option */
+ warnf(global, "The file name argument '%s' looks like a flag.\n",
+ nextarg);
+ }
}
else if((aliases[hit].desc == ARG_NONE) && !toggle)
return PARAM_NO_PREFIX;