the /dev/null at source case has same issue

Author: cre4ture

src/uu/install/src/install.rs

@@ -18,7 +18,7 @@ use std::os::unix::fs::MetadataExt;
 use std::os::unix::prelude::OsStrExt;
 use std::path::{Path, PathBuf, MAIN_SEPARATOR};
 use std::process;
-use std::{fs, io};
+use std::fs;
 use uucore::backup_control::{self, BackupMode};
 use uucore::display::Quotable;
 use uucore::entries::{grp2gid, usr2uid};
@@ -736,18 +736,6 @@ fn perform_backup(to: &Path, b: &Behavior) -> UResult<Option<PathBuf>> {
     }
 }
 
-fn copy_file_source_not_dev_null(from: &Path, to: &Path) -> io::Result<()> {
-    let dest_meta_res = fs::symlink_metadata(to);
-    if let Ok(dest_meta) = dest_meta_res {
-        if dest_meta.is_symlink() {
-            // fs::copy fails if destination is a invalid symlink
-            // so lets just remove all symlinks at destination before copy
-            fs::remove_file(to)?;
-        }
-    }
-    fs::copy(from, to).map(|_| ())
-}
-
 /// Copy a file from one path to another.
 ///
 /// # Parameters
@@ -760,6 +748,15 @@ fn copy_file_source_not_dev_null(from: &Path, to: &Path) -> io::Result<()> {
 /// Returns an empty Result or an error in case of failure.
 ///
 fn copy_file(from: &Path, to: &Path) -> UResult<()> {
+    let dest_meta_res = fs::symlink_metadata(to);
+    if let Ok(dest_meta) = dest_meta_res {
+        if dest_meta.is_symlink() {
+            // fs::copy fails if destination is a invalid symlink
+            // so lets just remove all symlinks at destination before copy
+            fs::remove_file(to)?;
+        }
+    }
+
     if from.as_os_str() == "/dev/null" {
         /* workaround a limitation of fs::copy
          * https://github.com/rust-lang/rust/issues/79390
@@ -769,13 +766,8 @@ fn copy_file(from: &Path, to: &Path) -> UResult<()> {
                 InstallError::InstallFailed(from.to_path_buf(), to.to_path_buf(), err).into(),
             );
         }
-    } else {
-        let result = copy_file_source_not_dev_null(from, to);
-        if let Err(err) = result {
-            return Err(
-                InstallError::InstallFailed(from.to_path_buf(), to.to_path_buf(), err).into(),
-            );
-        }
+    } else if let Err(err) = fs::copy(from, to) {
+        return Err(InstallError::InstallFailed(from.to_path_buf(), to.to_path_buf(), err).into());
     }
     Ok(())
 }

tests/by-util/test_install.rs

@@ -733,6 +733,37 @@ fn test_install_on_invalid_link_at_destination() {
         .no_stdout();
 }
 
+#[cfg(all(unix, feature = "chmod"))]
+#[test]
+// FixME: Freebsd fails on 'No such file or directory'
+#[cfg(not(target_os = "freebsd"))]
+fn test_install_on_invalid_link_at_destination_and_dev_null_at_source() {
+    let scene = TestScenario::new(util_name!());
+
+    let at = &scene.fixtures;
+    at.mkdir("src");
+    at.mkdir("dest");
+    let src_dir = at.plus("src");
+    let dst_dir = at.plus("dest");
+
+    at.touch("test.sh");
+    at.symlink_file(
+        "/opt/FakeDestination",
+        &dst_dir.join("test.sh").to_string_lossy(),
+    );
+    scene.ccmd("chmod").arg("+x").arg("test.sh").succeeds();
+    at.symlink_file("test.sh", &src_dir.join("test.sh").to_string_lossy());
+
+    scene
+        .ucmd()
+        .current_dir(&src_dir)
+        .arg("/dev/null")
+        .arg(dst_dir.join("test.sh"))
+        .succeeds()
+        .no_stderr()
+        .no_stdout();
+}
+
 #[test]
 #[cfg(not(windows))]
 fn test_install_and_strip_with_invalid_program() {