efi_loader: ListPackageLists() return EFI_NOT_FOUND
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Mon, 17 Jun 2019 18:46:29 +0000 (20:46 +0200)
committerHeinrich Schuchardt <xypron.glpk@gmx.de>
Thu, 20 Jun 2019 22:26:19 +0000 (22:26 +0000)
If no matching package list is found in ListPackageLists(), return
EFI_NOT_FOUND.

If we do not support a package type, we will not find a matching package
list. Remove the unreachable EFI_PRINTF() statements.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
lib/efi_loader/efi_hii.c

index 61b71dec621b0be2bb474da55a5a6e456e6f0b25..77e330285a72f61592ba2f0bc09dce4d5e819096 100644 (file)
@@ -581,18 +581,22 @@ list_package_lists(const struct efi_hii_database_protocol *this,
        struct efi_hii_packagelist *hii =
                                (struct efi_hii_packagelist *)handle;
        int package_cnt, package_max;
-       efi_status_t ret = EFI_SUCCESS;
+       efi_status_t ret = EFI_NOT_FOUND;
 
        EFI_ENTRY("%p, %u, %pUl, %p, %p", this, package_type, package_guid,
                  handle_buffer_length, handle);
 
        if (!handle_buffer_length ||
-           (*handle_buffer_length && !handle))
-               return EFI_EXIT(EFI_INVALID_PARAMETER);
+           (*handle_buffer_length && !handle)) {
+               ret = EFI_INVALID_PARAMETER;
+               goto out;
+       }
 
        if ((package_type != EFI_HII_PACKAGE_TYPE_GUID && package_guid) ||
-           (package_type == EFI_HII_PACKAGE_TYPE_GUID && !package_guid))
-               return EFI_EXIT(EFI_INVALID_PARAMETER);
+           (package_type == EFI_HII_PACKAGE_TYPE_GUID && !package_guid)) {
+               ret = EFI_INVALID_PARAMETER;
+               goto out;
+       }
 
        EFI_PRINT("package type=%x, guid=%pUl, length=%zu\n", (int)package_type,
                  package_guid, *handle_buffer_length);
@@ -607,53 +611,28 @@ list_package_lists(const struct efi_hii_database_protocol *this,
                        if (!list_empty(&hii->guid_list))
                                break;
                        continue;
-               case EFI_HII_PACKAGE_FORMS:
-                       EFI_PRINT("Form package not supported\n");
-                       ret = EFI_INVALID_PARAMETER;
-                       continue;
                case EFI_HII_PACKAGE_STRINGS:
                        if (!list_empty(&hii->string_tables))
                                break;
                        continue;
-               case EFI_HII_PACKAGE_FONTS:
-                       EFI_PRINT("Font package not supported\n");
-                       ret = EFI_INVALID_PARAMETER;
-                       continue;
-               case EFI_HII_PACKAGE_IMAGES:
-                       EFI_PRINT("Image package not supported\n");
-                       ret = EFI_INVALID_PARAMETER;
-                       continue;
-               case EFI_HII_PACKAGE_SIMPLE_FONTS:
-                       EFI_PRINT("Simple font package not supported\n");
-                       ret = EFI_INVALID_PARAMETER;
-                       continue;
-               case EFI_HII_PACKAGE_DEVICE_PATH:
-                       EFI_PRINT("Device path package not supported\n");
-                       ret = EFI_INVALID_PARAMETER;
-                       continue;
                case EFI_HII_PACKAGE_KEYBOARD_LAYOUT:
                        if (!list_empty(&hii->keyboard_packages))
                                break;
                        continue;
-               case EFI_HII_PACKAGE_ANIMATIONS:
-                       EFI_PRINT("Animation package not supported\n");
-                       ret = EFI_INVALID_PARAMETER;
-                       continue;
-               case EFI_HII_PACKAGE_END:
-               case EFI_HII_PACKAGE_TYPE_SYSTEM_BEGIN:
-               case EFI_HII_PACKAGE_TYPE_SYSTEM_END:
                default:
                        continue;
                }
 
                package_cnt++;
-               if (package_cnt <= package_max)
+               if (package_cnt <= package_max) {
                        *handle++ = hii;
-               else
+                       ret = EFI_SUCCESS;
+               } else {
                        ret = EFI_BUFFER_TOO_SMALL;
+               }
        }
        *handle_buffer_length = package_cnt * sizeof(*handle);
-
+out:
        return EFI_EXIT(ret);
 }