From 41b5f830c6d74dca69cfea1c259f7e21d4ad9bb9 Mon Sep 17 00:00:00 2001
From: Alejandro Sirgo Rica <asirgo@soleta.eu>
Date: Mon, 22 Apr 2024 13:55:21 +0200
Subject: utils: consolidate code to find efi loader

Add find_windows_efi_loader and find_linux_efi_loader to reduce
code duplication and to centralize efi loader path modifications.
---
 src/utils/uefi.py | 33 +++++++++++++++++++++++----------
 1 file changed, 23 insertions(+), 10 deletions(-)

(limited to 'src/utils/uefi.py')

diff --git a/src/utils/uefi.py b/src/utils/uefi.py
index 0906ae9..48fb6c3 100644
--- a/src/utils/uefi.py
+++ b/src/utils/uefi.py
@@ -103,6 +103,28 @@ def efibootmgr_create_bootentry(disk, part, loader, label, add_to_bootorder=True
         raise OgError(f'Unexpected error adding boot entry to nvram. UEFI firmware might be buggy') from e
 
 
+def _find_efi_loader(loader_paths):
+    for efi_app in loader_paths:
+        logging.info(f'Searching for {efi_app}')
+        if os.path.exists(efi_app):
+            logging.info(f'Found bootloader at ESP partition: {efi_app}')
+            return efi_app
+    else:
+        raise OgError(f'Unable to locate Windows EFI bootloader bootmgfw.efi')
+
+
+def find_windows_efi_loader(esp_mountpoint, bootlabel):
+    loader_paths = [f'{esp_mountpoint}/EFI/{bootlabel}/Boot/bootmgfw.efi',
+                    f'{esp_mountpoint}/EFI/Microsoft/Boot/bootmgfw.efi']
+    return _find_efi_loader(loader_paths)
+
+
+def find_linux_efi_loader(esp_mountpoint, bootlabel):
+    loader_paths = [f'{esp_mountpoint}/EFI/{bootlabel}/Boot/shimx64.efi',
+                    f'{esp_mountpoint}/EFI/ubuntu/shimx64.efi']
+    return _find_efi_loader(loader_paths)
+
+
 def copy_windows_efi_bootloader(disk, partition):
     device = get_partition_device(disk, partition)
     mountpoint = device.replace('dev', 'mnt')
@@ -122,17 +144,8 @@ def copy_windows_efi_bootloader(disk, partition):
         umount(mountpoint)
         raise OgError(f'Unable to mount detected EFI System Partition at {esp} into {esp_mountpoint}')
 
-    loader_paths = [f'{esp_mountpoint}/EFI/{bootlabel}/Boot/bootmgfw.efi',
-                    f'{esp_mountpoint}/EFI/Microsoft/Boot/bootmgfw.efi']
     try:
-        for efi_app in loader_paths:
-            logging.info(f'Searching for {efi_app}')
-            if os.path.exists(efi_app):
-                loader = efi_app
-                logging.info(f'Found bootloader at ESP partition: {loader}')
-                break
-        else:
-            raise OgError(f'Unable to locate Windows EFI bootloader bootmgfw.efi')
+        loader = find_windows_efi_loader(esp_mountpoint, bootlabel)
 
         loader_dir = os.path.dirname(loader)
         destination_dir = f'{mountpoint}/ogBoot'
-- 
cgit v1.2.3-18-g5258