diff --git a/src/uu/cp/src/cp.rs b/src/uu/cp/src/cp.rs index 8741646a3..d005bd718 100644 --- a/src/uu/cp/src/cp.rs +++ b/src/uu/cp/src/cp.rs @@ -768,8 +768,8 @@ fn preserve_hardlinks( let mut stat = mem::zeroed(); if libc::lstat(src_path.as_ptr(), &mut stat) < 0 { return Err(format!( - "cannot stat {:?}: {}", - src_path, + "cannot stat {}: {}", + source.quote(), std::io::Error::last_os_error() ) .into()); @@ -849,7 +849,7 @@ fn copy(sources: &[Source], target: &TargetSlice, options: &Options) -> CopyResu let mut found_hard_link = false; if preserve_hard_links { let dest = construct_dest_path(source, target, &target_type, options)?; - preserve_hardlinks(&mut hard_links, source, dest, &mut found_hard_link).unwrap(); + preserve_hardlinks(&mut hard_links, source, dest, &mut found_hard_link)?; } if !found_hard_link { if let Err(error) = @@ -1031,7 +1031,7 @@ fn copy_directory( let mut found_hard_link = false; let source = path.to_path_buf(); let dest = local_to_target.as_path().to_path_buf(); - preserve_hardlinks(&mut hard_links, &source, dest, &mut found_hard_link).unwrap(); + preserve_hardlinks(&mut hard_links, &source, dest, &mut found_hard_link)?; if !found_hard_link { match copy_file( path.as_path(), diff --git a/tests/by-util/test_cp.rs b/tests/by-util/test_cp.rs index 07597cf15..92637dfbe 100644 --- a/tests/by-util/test_cp.rs +++ b/tests/by-util/test_cp.rs @@ -43,6 +43,8 @@ static TEST_MOUNT_COPY_FROM_FOLDER: &str = "dir_with_mount"; static TEST_MOUNT_MOUNTPOINT: &str = "mount"; #[cfg(any(target_os = "linux", target_os = "freebsd"))] static TEST_MOUNT_OTHER_FILESYSTEM_FILE: &str = "mount/DO_NOT_copy_me.txt"; +#[cfg(unix)] +static TEST_NONEXISTENT_FILE: &str = "nonexistent_file.txt"; #[test] fn test_cp_cp() { @@ -1429,3 +1431,16 @@ fn test_copy_through_dangling_symlink() { .fails() .stderr_only("cp: not writing through dangling symlink 'target'"); } + +#[test] +#[cfg(unix)] +fn test_cp_archive_on_nonexistent_file() { + new_ucmd!() + .arg("-a") + .arg(TEST_NONEXISTENT_FILE) + .arg(TEST_EXISTING_FILE) + .fails() + .stderr_only( + "cp: cannot stat 'nonexistent_file.txt': No such file or directory (os error 2)", + ); +}