Use try_fold() for proper error propagation

Inspired by Colin's comment https://github.com/coreos/bootupd/pull/932/files#r2103628222

Author: HuijingHei

src/efi.rs

@@ -79,11 +79,26 @@ impl Efi {
     // Get mounted point for esp
     pub(crate) fn get_mounted_esp(&self, root: &Path) -> Result<Option<PathBuf>> {
         // First check all potential mount points without holding the borrow
-        let found_mount = ESP_MOUNTS.iter().map(|&mnt| root.join(mnt)).find(|mnt| {
-            mnt.exists()
-                && rustix::fs::statfs(mnt).map_or(false, |st| st.f_type == libc::MSDOS_SUPER_MAGIC)
-                && util::ensure_writable_mount(mnt).is_ok()
-        });
+        let found_mount = ESP_MOUNTS
+            .iter()
+            .try_fold(None, |mntpoint, &mnt| -> anyhow::Result<_> {
+                // return for the first match
+                if mntpoint.is_some() {
+                    return Ok(mntpoint);
+                }
+                let path = root.join(mnt);
+                if !path.exists() {
+                    return Ok(None);
+                }
+
+                let st = rustix::fs::statfs(&path)?;
+                if st.f_type == libc::MSDOS_SUPER_MAGIC {
+                    util::ensure_writable_mount(&path)?;
+                    Ok(Some(path))
+                } else {
+                    Ok(None)
+                }
+            })?;
 
         // Only borrow mutably if we found a mount point
         if let Some(mnt) = found_mount {
@@ -266,8 +281,7 @@ impl Component for Efi {
             anyhow::bail!("Failed to find adoptable system")
         };
 
-        // Confirm that esp_devices is Some(value)
-        let esp_devices = esp_devices.unwrap();
+        let esp_devices = esp_devices.unwrap_or_default();
         let mut devices = esp_devices.iter();
         let Some(esp) = devices.next() else {
             anyhow::bail!("Failed to find esp device");
@@ -445,8 +459,7 @@ impl Component for Efi {
             .as_ref()
             .ok_or_else(|| anyhow::anyhow!("No filetree for installed EFI found!"))?;
 
-        // Confirm that esp_devices is Some(value)
-        let esp_devices = esp_devices.unwrap();
+        let esp_devices = esp_devices.unwrap_or_default();
         let mut devices = esp_devices.iter();
         let Some(esp) = devices.next() else {
             anyhow::bail!("Failed to find esp device");