diff options
| author | Marco Felsch <m.felsch@pengutronix.de> | 2023-07-04 14:16:25 +0200 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2023-07-28 08:40:48 +0200 |
| commit | 28d8664f116e5c69ae73a704967793380d9d0a5b (patch) | |
| tree | 8ee8060bc73d8027b3fd90b568d3327a1906ea1b /commands | |
| parent | a6406b8cda2b384490446b11c786b35d971944b0 (diff) | |
| download | barebox-28d8664f116e5c69ae73a704967793380d9d0a5b.tar.gz barebox-28d8664f116e5c69ae73a704967793380d9d0a5b.tar.xz | |
commands: test: simplify argv handling
Decrement argc first before check the closing ']' to avoid the
*argv[argc - 1]. No functional change.
Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
Link: https://lore.barebox.org/20230704121626.835344-1-m.felsch@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'commands')
| -rw-r--r-- | commands/test.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/commands/test.c b/commands/test.c index c845cec017..c1b84c42ef 100644 --- a/commands/test.c +++ b/commands/test.c @@ -75,11 +75,11 @@ static int do_test(int argc, char *argv[]) struct stat statbuf; if (*argv[0] == '[') { - if (*argv[argc - 1] != ']') { + argc--; + if (*argv[argc] != ']') { printf("[: missing `]'\n"); return 1; } - argc--; } /* args? */ |