Merge pull request #9613 from Ecordonnier/eco/nohup-command-exec

nohup: use Command::exec() instead of libc::execvp()

Author: RenjiSann

src/uu/nohup/src/nohup.rs

@@ -3,17 +3,17 @@
 // For the full copyright and license information, please view the LICENSE
 // file that was distributed with this source code.
 
-// spell-checker:ignore (ToDO) execvp SIGHUP cproc vprocmgr cstrs homeout
+// spell-checker:ignore (ToDO) SIGHUP cproc vprocmgr homeout
 
 use clap::{Arg, ArgAction, Command};
-use libc::{SIG_IGN, SIGHUP};
-use libc::{c_char, dup2, execvp, signal};
+use libc::{SIG_IGN, SIGHUP, dup2, signal};
 use std::env;
-use std::ffi::CString;
 use std::fs::{File, OpenOptions};
-use std::io::{Error, IsTerminal};
+use std::io::{Error, ErrorKind, IsTerminal};
 use std::os::unix::prelude::*;
+use std::os::unix::process::CommandExt;
 use std::path::{Path, PathBuf};
+use std::process;
 use thiserror::Error;
 use uucore::display::Quotable;
 use uucore::error::{UError, UResult, set_exit_code};
@@ -68,17 +68,14 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
         return Err(NohupError::CannotDetach.into());
     }
 
-    let cstrs: Vec<CString> = matches
-        .get_many::<String>(options::CMD)
-        .unwrap()
-        .map(|x| CString::new(x.as_bytes()).unwrap())
-        .collect();
-    let mut args: Vec<*const c_char> = cstrs.iter().map(|s| s.as_ptr()).collect();
-    args.push(std::ptr::null());
-
-    let ret = unsafe { execvp(args[0], args.as_mut_ptr()) };
-    match ret {
-        libc::ENOENT => set_exit_code(EXIT_ENOENT),
+    let mut cmd_iter = matches.get_many::<String>(options::CMD).unwrap();
+    let cmd = cmd_iter.next().unwrap();
+    let args: Vec<&String> = cmd_iter.collect();
+
+    let err = process::Command::new(cmd).args(args).exec();
+
+    match err.kind() {
+        ErrorKind::NotFound => set_exit_code(EXIT_ENOENT),
         _ => set_exit_code(EXIT_CANNOT_INVOKE),
     }
     Ok(())