Message ID | 20241102100836.103005-2-heinrich.schuchardt@canonical.com |
---|---|
State | Accepted |
Commit | 6831415e397a95c3220295a6a7d6a686359de35f |
Delegated to: | Tom Rini |
Headers | show |
Series | cmd: hash: correct parameter count check | expand |
Hi Heinrich, On Sat, Nov 2, 2024 at 11:09 AM Heinrich Schuchardt <heinrich.schuchardt@canonical.com> wrote: > > Since commit 348ea878508d ("cmd: hash: fix param count check") the hash > command cannot be used without the optional variable name parameter if > CONFIG_HASH_VERIFY=y. 'hash sha1 $loadaddr $filesize' returns > CMD_RET_USAGE. > > The minimum number of arguments is four no matter if verification is > enabled or not. > > Fixes: 348ea878508d ("cmd: hash: fix param count check") > Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> > --- > cmd/hash.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/cmd/hash.c b/cmd/hash.c > index 60d482b7f87..5b40982b098 100644 > --- a/cmd/hash.c > +++ b/cmd/hash.c > @@ -25,7 +25,7 @@ static int do_hash(struct cmd_tbl *cmdtp, int flag, int argc, > char *s; > int flags = HASH_FLAG_ENV; > > - if (argc < (HARGS - 1)) > + if (argc < 4) > return CMD_RET_USAGE; > > #if IS_ENABLED(CONFIG_HASH_VERIFY) > -- > 2.45.2 > Reviewed-by: Igor Opaniuk <igor.opaniuk@gmail.com>
diff --git a/cmd/hash.c b/cmd/hash.c index 60d482b7f87..5b40982b098 100644 --- a/cmd/hash.c +++ b/cmd/hash.c @@ -25,7 +25,7 @@ static int do_hash(struct cmd_tbl *cmdtp, int flag, int argc, char *s; int flags = HASH_FLAG_ENV; - if (argc < (HARGS - 1)) + if (argc < 4) return CMD_RET_USAGE; #if IS_ENABLED(CONFIG_HASH_VERIFY)
Since commit 348ea878508d ("cmd: hash: fix param count check") the hash command cannot be used without the optional variable name parameter if CONFIG_HASH_VERIFY=y. 'hash sha1 $loadaddr $filesize' returns CMD_RET_USAGE. The minimum number of arguments is four no matter if verification is enabled or not. Fixes: 348ea878508d ("cmd: hash: fix param count check") Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> --- cmd/hash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)