style: format imports and chain method calls in safe_traversal tests

Author: mattsu2020

src/uucore/src/lib/features/safe_traversal.rs

@@ -392,21 +392,23 @@ impl DirFd {
     ///
     /// Uses rustix safe wrappers to avoid raw `unsafe` FFI calls.
     #[cfg(target_os = "linux")]
-    fn chmod_at_via_opath(&self, name: &CStr, mode: u32) -> io::Result<()> {
+    fn chmod_at_via_opath(&self, name: &std::ffi::CStr, mode: u32) -> io::Result<()> {
         use rustix::fs::{Mode, OFlags, chmod, openat};
 
         let fd = openat(
             &self.fd,
             name,
             OFlags::PATH | OFlags::NOFOLLOW | OFlags::CLOEXEC,
             Mode::empty(),
-        )?;
+        )
+        .map_err(|e| io::Error::from_raw_os_error(e.raw_os_error().unwrap_or(libc::EIO)))?;
 
         let proc_path = format!("/proc/self/fd/{}\0", fd.as_raw_fd());
-        let proc_cstr = CStr::from_bytes_with_nul(proc_path.as_bytes())
+        let proc_cstr = std::ffi::CStr::from_bytes_with_nul(proc_path.as_bytes())
             .map_err(|_| io::Error::new(io::ErrorKind::InvalidInput, "invalid proc path"))?;
 
         chmod(proc_cstr, Mode::from_bits_truncate(mode as u32))
+            .map_err(|e| io::Error::from_raw_os_error(e.raw_os_error().unwrap_or(libc::EIO)))
     }
 
     /// Change mode of this directory
@@ -899,8 +901,8 @@ impl std::os::unix::fs::MetadataExt for Metadata {
 mod tests {
     use super::*;
     use std::fs;
-    use std::os::unix::fs::symlink;
     use std::os::unix::fs::MetadataExt;
+    use std::os::unix::fs::symlink;
     use std::os::unix::io::IntoRawFd;
     use tempfile::TempDir;
 
@@ -1337,9 +1339,7 @@ mod tests {
         // Create a sentinel file outside the traversal directory
         let sentinel = temp_dir.path().join("sentinel");
         fs::write(&sentinel, "victim").unwrap();
-        let sentinel_mode = fs::symlink_metadata(&sentinel)
-            .unwrap()
-            .mode();
+        let sentinel_mode = fs::symlink_metadata(&sentinel).unwrap().mode();
 
         // Create a subdirectory with a symlink pointing to the sentinel
         let subdir = temp_dir.path().join("subdir");
@@ -1356,9 +1356,7 @@ mod tests {
         // which is acceptable — the important thing is the target is NOT modified.
         if let Ok(()) = result {
             // fchmodat2 succeeded: verify sentinel mode is unchanged
-            let new_sentinel_mode = fs::symlink_metadata(&sentinel)
-                .unwrap()
-                .mode();
+            let new_sentinel_mode = fs::symlink_metadata(&sentinel).unwrap().mode();
             assert_eq!(
                 new_sentinel_mode, sentinel_mode,
                 "sentinel mode should not change when chmod'ing symlink with NoFollow"