refactor: I/O safety for add_dup2

Signed-off-by: tison <[email protected]>

Author: tisonkun

src/spawn.rs

@@ -1,14 +1,14 @@
 //! Safe wrappers around posix_spawn* functions found in the libc "spawn.h" header.
 
-use std::{ffi::CStr, mem, os::fd::RawFd};
-
 #[cfg(any(feature = "fs", feature = "term"))]
 use crate::fcntl::OFlag;
 #[cfg(feature = "signal")]
 use crate::sys::signal::SigSet;
 #[cfg(feature = "fs")]
 use crate::sys::stat::Mode;
 use crate::{errno::Errno, unistd::Pid, NixPath, Result};
+use std::os::fd::AsRawFd;
+use std::{ffi::CStr, mem, os::fd::RawFd};
 
 /// A spawn attributes object. See [posix_spawnattr_t](https://pubs.opengroup.org/onlinepubs/9699919799/functions/posix_spawnattr_init.html).
 #[repr(transparent)]
@@ -277,7 +277,14 @@ impl PosixSpawnFileActions {
     /// Add a [dup2](https://pubs.opengroup.org/onlinepubs/9699919799/functions/dup2.html) action. See
     /// [posix_spawn_file_actions_adddup2](https://pubs.opengroup.org/onlinepubs/9699919799/functions/posix_spawn_file_actions_adddup2.html).
     #[doc(alias("posix_spawn_file_actions_adddup2"))]
-    pub fn add_dup2(&mut self, fd: RawFd, newfd: RawFd) -> Result<()> {
+    pub fn add_dup2<Fd, NewFd>(&mut self, fd: Fd, newfd: NewFd) -> Result<()>
+    where
+        Fd: std::os::fd::AsFd,
+        NewFd: std::os::fd::AsFd,
+    {
+        let fd = fd.as_fd().as_raw_fd();
+        let newfd = newfd.as_fd().as_raw_fd();
+
         let res = unsafe {
             libc::posix_spawn_file_actions_adddup2(
                 &mut self.fa as *mut libc::posix_spawn_file_actions_t,