diff options
| author | Joe Hershberger <joe.hershberger@ni.com> | 2015-05-05 12:23:53 -0500 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2015-05-10 09:59:37 -0400 |
| commit | a69bdba9c6275b5673133ad057e2195ab48f3a53 (patch) | |
| tree | 4ac3ba8aa645313f7815f10e177df98579cf41dd /common | |
| parent | 849337bb22ca87355963236f784b7f3e57c0329a (diff) | |
| download | talos-obmc-uboot-a69bdba9c6275b5673133ad057e2195ab48f3a53.tar.gz talos-obmc-uboot-a69bdba9c6275b5673133ad057e2195ab48f3a53.zip | |
Fix broken verify functionality in crc32 command
Introduced in change d20a40de9db07de1f1f06a79a4da1cdda5379b75
"Roll crc32 into hash infrastructure"
The crc32 command with no -v expects an optional 3rd argument to be an
address to store the result in. With the -v switch, the last argument
is a crc, not an address. In the case where -v is set, we should set the
HASH_FLAG_ENV flag since that will first look for the value to be a
digest value, which matches the expected API for the crc32 command.
Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'common')
| -rw-r--r-- | common/cmd_mem.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/cmd_mem.c b/common/cmd_mem.c index b91adb29c9..2e85d53dd2 100644 --- a/common/cmd_mem.c +++ b/common/cmd_mem.c @@ -1227,7 +1227,7 @@ static int do_mem_crc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) ac = argc - 1; #ifdef CONFIG_HASH_VERIFY if (strcmp(*av, "-v") == 0) { - flags |= HASH_FLAG_VERIFY; + flags |= HASH_FLAG_VERIFY | HASH_FLAG_ENV; av++; ac--; } |
