efi: Correct assumption about if_type
authorSimon Glass <sjg@chromium.org>
Fri, 12 Aug 2022 01:34:58 +0000 (19:34 -0600)
committerTom Rini <trini@konsulko.com>
Fri, 16 Sep 2022 15:05:16 +0000 (11:05 -0400)
efi_set_blk_dev_to_system_partition() assumes that 0 is an invalid
if_type. This is true now but is about to be false. Fix this bug to avoid
a test failure.

Signed-off-by: Simon Glass <sjg@chromium.org>
lib/efi_loader/efi_var_file.c

index 76a2ff9..994e663 100644 (file)
@@ -38,7 +38,7 @@ static efi_status_t __maybe_unused efi_set_blk_dev_to_system_partition(void)
        char part_str[PART_STR_LEN];
        int r;
 
-       if (!efi_system_partition.if_type) {
+       if (efi_system_partition.if_type == UCLASS_INVALID) {
                log_err("No EFI system partition\n");
                return EFI_DEVICE_ERROR;
        }