Merge pull request #91 from noeljackson/feat/mount-data

feat(mount): add mount data and improve bind target handling

Author: bleggett

src/config.rs

@@ -250,6 +250,9 @@ pub struct MountSpec {
 
     /// Whether the mount point should be mounted readonly.
     pub read_only: bool,
+
+    /// Optional mount data (e.g., "size=64m" for tmpfs).
+    pub data: Option<String>,
 }
 
 pub trait Mountable {

src/mount.rs

@@ -146,7 +146,22 @@ impl Mountable for MountSpec {
         let target_p = target.as_ptr();
 
         if self.create_mountpoint {
-            fs::create_dir_all(&self.target)?;
+            let source_is_file = self.bind
+                && self
+                    .source
+                    .as_deref()
+                    .and_then(|s| std::path::Path::new(s).metadata().ok())
+                    .map(|m| !m.is_dir())
+                    .unwrap_or(false);
+
+            if source_is_file {
+                if let Some(parent) = std::path::Path::new(&self.target).parent() {
+                    fs::create_dir_all(parent)?;
+                }
+                fs::File::create(&self.target)?;
+            } else {
+                fs::create_dir_all(&self.target)?;
+            }
         }
 
         let mut flags: c_ulong = libc::MS_SILENT;
@@ -163,10 +178,28 @@ impl Mountable for MountSpec {
             flags |= libc::MS_REC;
         }
 
+        let data_cstr = self
+            .data
+            .as_ref()
+            .map(|d| CString::new(d.as_str()).unwrap());
+        let data_ptr = data_cstr
+            .as_ref()
+            .map(|c| c.as_ptr() as *const libc::c_void)
+            .unwrap_or(ptr::null());
+
         unsafe {
-            let rc = libc::mount(source_p, target_p, fstype_p, flags, ptr::null());
+            let rc = libc::mount(source_p, target_p, fstype_p, flags, data_ptr);
             if rc < 0 {
-                bail!("unable to mount");
+                let err = io::Error::last_os_error();
+                bail!(
+                    "unable to mount: source={:?} target={:?} fstype={:?} bind={} flags=0x{:x}: {}",
+                    self.source,
+                    self.target,
+                    self.fstype,
+                    self.bind,
+                    flags,
+                    err
+                );
             }
         }
 

src/wrap.rs

@@ -346,6 +346,7 @@ impl CreateRequest {
             safe: false,
             create_mountpoint: false,
             read_only: false,
+            data: None,
         };
 
         oldroot
@@ -370,6 +371,7 @@ impl CreateRequest {
                 safe: true,
                 create_mountpoint: true,
                 read_only: false,
+                data: None,
             };
             stage_tmpfs
                 .mount()
@@ -390,6 +392,7 @@ impl CreateRequest {
                 safe: false,
                 create_mountpoint: false,
                 read_only: false,
+                data: None,
             };
             stage_bind
                 .mount()
@@ -409,6 +412,7 @@ impl CreateRequest {
             safe: false,
             create_mountpoint: false,
             read_only: false,
+            data: None,
         };
 
         newroot
@@ -432,6 +436,7 @@ impl CreateRequest {
             safe: true,
             create_mountpoint: false,
             read_only: false,
+            data: None,
         };
 
         procfs
@@ -451,6 +456,7 @@ impl CreateRequest {
                     safe: mount.safe,
                     create_mountpoint: mount.create_mountpoint,
                     read_only: mount.read_only,
+                    data: None,
                 };
 
                 parented_mount