efi: add drop() before unmounting /boot/efi

Fix error:
```
umount: /boot/efi: target is busy.

error: unmount after validate: Failed to unmount "/boot/efi":
Child ["umount" "/boot/efi"] exited: exit status: 32
```

Author: HuijingHei

src/efi.rs

@@ -10,6 +10,7 @@ use std::path::{Path, PathBuf};
 use std::process::Command;
 
 use anyhow::{bail, Context, Result};
+use bootc_utils::CommandRunExt;
 use cap_std::fs::Dir;
 use cap_std_ext::cap_std;
 use fn_error_context::context;
@@ -22,7 +23,7 @@ use widestring::U16CString;
 use crate::bootupd::RootContext;
 use crate::model::*;
 use crate::ostreeutil;
-use crate::util::{self, CommandRunExt};
+use crate::util;
 use crate::{blockdev, filetree};
 use crate::{component::*, packagesystem};
 
@@ -422,7 +423,7 @@ impl Component for Efi {
         for esp in esp_devices.iter() {
             let destpath = &self.ensure_mounted_esp(Path::new("/"), Path::new(&esp))?;
 
-            let efidir = &openat::Dir::open(&destpath.join("EFI"))
+            let efidir = openat::Dir::open(&destpath.join("EFI"))
                 .with_context(|| format!("opening EFI dir {}", destpath.display()))?;
             let diff = currentf.relative_diff_to(&efidir)?;
 
@@ -433,6 +434,7 @@ impl Component for Efi {
                 errs.push(format!("Removed: {}", f));
             }
             assert_eq!(diff.additions.len(), 0);
+            drop(efidir);
             self.unmount().context("unmount after validate")?;
         }
 

src/util.rs

@@ -5,6 +5,7 @@ use std::process::Command;
 use anyhow::{bail, Context, Result};
 use openat_ext::OpenatDirExt;
 
+#[allow(dead_code)]
 pub(crate) trait CommandRunExt {
     fn run(&mut self) -> Result<()>;
 }