diff options
| author | Joe Orton <jorton@apache.org> | 2021-12-03 16:40:18 +0000 |
|---|---|---|
| committer | Joe Orton <jorton@apache.org> | 2021-12-03 16:40:18 +0000 |
| commit | d318188682d3577a0feea348657d25192c7f513f (patch) | |
| tree | 5fc0e387bee2ad9d9e2439ca107885b24e7a4d03 /support | |
| parent | 34da2e78b72221e2ae7a801c718e826ec33e1959 (diff) | |
| download | httpd-d318188682d3577a0feea348657d25192c7f513f.tar.gz | |
* support/ab.c (main): Check apr_getopt() returned APR_EOF, fixing clang
warning:
support/ab.c:2343:13: warning[deadcode.DeadStores]: Although the value stored to 'status' is used in the enclosing expression, the value is never actually read from 'status'
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1895559 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'support')
| -rw-r--r-- | support/ab.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/support/ab.c b/support/ab.c index f4a68662f6..55d1e4bb88 100644 --- a/support/ab.c +++ b/support/ab.c @@ -2592,8 +2592,8 @@ int main(int argc, const char * const argv[]) } } - if (opt->ind != argc - 1) { - fprintf(stderr, "%s: wrong number of arguments\n", argv[0]); + if (status != APR_EOF || opt->ind != argc - 1) { + fprintf(stderr, "%s: Invalid or missing arguments\n", argv[0]); usage(argv[0]); } |