]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
Merge patch series "cmd: hash: correct parameter count check"
authorTom Rini <trini@konsulko.com>
Thu, 14 Nov 2024 16:49:30 +0000 (10:49 -0600)
committerTom Rini <trini@konsulko.com>
Thu, 14 Nov 2024 16:49:51 +0000 (10:49 -0600)
Heinrich Schuchardt <heinrich.schuchardt@canonical.com> says:

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.

Fix the parameter check.

Provide a unit test.

Link: https://lore.kernel.org/r/20241102100836.103005-1-heinrich.schuchardt@canonical.com
1  2 
test/cmd/Makefile

index 5e4a2040d6ce4b284013af38ddb165d77ad1c38c,1fd3e89efffb8685b6b2a302d578af810e399cd6..583e7c2eec42ee640a09add674f1875f9f2652b6
@@@ -16,9 -15,9 +16,10 @@@ obj-y += exit.o mem.
  obj-$(CONFIG_X86) += cpuid.o msr.o
  obj-$(CONFIG_CMD_ADDRMAP) += addrmap.o
  obj-$(CONFIG_CMD_BDI) += bdinfo.o
 +obj-$(CONFIG_COREBOOT_SYSINFO) += coreboot.o
  obj-$(CONFIG_CMD_FDT) += fdt.o
  obj-$(CONFIG_CONSOLE_TRUETYPE) += font.o
+ obj-$(CONFIG_CMD_HASH) += hash.o
  obj-$(CONFIG_CMD_HISTORY) += history.o
  obj-$(CONFIG_CMD_LOADM) += loadm.o
  obj-$(CONFIG_CMD_MEMINFO) += meminfo.o