cmd: efidebug: fix -Werror=type-limits warning
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Sat, 9 May 2020 15:59:19 +0000 (17:59 +0200)
committerHeinrich Schuchardt <xypron.glpk@gmx.de>
Sat, 9 May 2020 22:01:12 +0000 (00:01 +0200)
Compiling with -Wtype-limits yields:

cmd/efidebug.c:968:32: error: comparison is always false due to limited
range of data type [-Werror=type-limits]
  968 |  if (*endp != '\0' || bootnext > 0xffff) {
      |

Remove the superfluous check.

Fixes: 59df7e7e77e7 ("cmd: add efidebug command")
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
cmd/efidebug.c

index 668e5c4e872e5c6f5dd0d11abb5f298fcca6767a..5cc0a41af3b441f072e7432e596dcbf079fbc357 100644 (file)
@@ -970,7 +970,7 @@ static int do_efi_boot_next(cmd_tbl_t *cmdtp, int flag,
                return CMD_RET_USAGE;
 
        bootnext = (u16)simple_strtoul(argv[1], &endp, 16);
                return CMD_RET_USAGE;
 
        bootnext = (u16)simple_strtoul(argv[1], &endp, 16);
-       if (*endp != '\0' || bootnext > 0xffff) {
+       if (*endp) {
                printf("invalid value: %s\n", argv[1]);
                r = CMD_RET_FAILURE;
                goto out;
                printf("invalid value: %s\n", argv[1]);
                r = CMD_RET_FAILURE;
                goto out;