diff options
| author | Rich Salz <rsalz@akamai.com> | 2020-11-28 22:12:58 +0100 |
|---|---|---|
| committer | Tomas Mraz <tmraz@fedoraproject.org> | 2020-12-15 11:47:17 +0100 |
| commit | 021410ea3fc3876538830839d16b67e610d12785 (patch) | |
| tree | 7178c87097f1083bc285a77f8ec179ceba343464 /apps/info.c | |
| parent | test: document the random test ordering env variable (diff) | |
| download | openssl-021410ea3fc3876538830839d16b67e610d12785.tar.xz openssl-021410ea3fc3876538830839d16b67e610d12785.zip | |
Check non-option arguments
Make sure all commands check to see if there are any "extra" arguments
after the options, and print an error if so.
Made all error messages consistent (which is to say, minimal).
Fixes: #13527
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from https://github.com/openssl/openssl/pull/13563)
Diffstat (limited to 'apps/info.c')
| -rw-r--r-- | apps/info.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/apps/info.c b/apps/info.c index ae6e34c9bc..5099853494 100644 --- a/apps/info.c +++ b/apps/info.c @@ -86,10 +86,8 @@ opthelp: break; } } - if (opt_num_rest() != 0) { - BIO_printf(bio_err, "%s: Extra parameters given.\n", prog); + if (opt_num_rest() != 0) goto opthelp; - } if (dirty > 1) { BIO_printf(bio_err, "%s: Only one item allowed\n", prog); goto opthelp; |