fix install owner flag with invalid uid

Author: zhw2101024

src/uu/install/src/install.rs

@@ -392,7 +392,10 @@ fn behavior(matches: &ArgMatches) -> UResult<Behavior> {
     } else {
         match usr2uid(&owner) {
             Ok(u) => Some(u),
-            Err(_) => return Err(InstallError::InvalidUser(owner.clone()).into()),
+            Err(_) => match owner.parse::<u32>() {
+                Ok(u) => Some(u),
+                Err(_) => return Err(InstallError::InvalidUser(owner.clone()).into()),
+            },
         }
     };
 

tests/by-util/test_install.rs

@@ -9,7 +9,7 @@ use filetime::FileTime;
 use std::fs;
 use std::os::unix::fs::{MetadataExt, PermissionsExt};
 #[cfg(not(windows))]
-use std::process::Command;
+use std::process::{Command, Stdio};
 #[cfg(any(target_os = "linux", target_os = "android"))]
 use std::thread::sleep;
 use uucore::process::{getegid, geteuid};
@@ -1807,4 +1807,39 @@ fn test_install_symlink_same_file() {
         .stderr_contains(format!(
             "'{target_dir}/{file}' and '{target_link}/{file}' are the same file"
         ));
+/// This test requires user with userid 500 nonexistent
+#[test]
+#[cfg(unix)]
+fn test_install_set_owner_nonexistent_userid() {
+    match Command::new("id")
+        .arg("500")
+        .stdout(Stdio::null())
+        .stderr(Stdio::null())
+        .status()
+    {
+        Ok(ret) if ret.success() => {
+            println!("Test skipped; requires user with userid 500 nonexistent");
+            return;
+        }
+        Ok(_) => {}
+        Err(e) => {
+            println!("failed to run id command: {}", e);
+            return;
+        }
+    }
+
+    let ts = TestScenario::new(util_name!());
+    let at = &ts.fixtures;
+    at.touch("a");
+    at.touch("c");
+
+    if let Ok(result) = run_ucmd_as_root(&ts, &["-o500", "a", "b"]) {
+        result.success();
+        assert!(at.file_exists("b"));
+
+        let metadata = fs::metadata(at.plus("b")).unwrap();
+        assert_eq!(metadata.uid(), 500);
+    } else {
+        print!("Test skipped; requires root user");
+    }
 }