cp: use stat for dest existence check to support LD_PRELOAD

Author: ChrisDryden

src/uu/cp/src/cp.rs

@@ -1390,7 +1390,8 @@ pub fn copy(sources: &[PathBuf], target: &Path, options: &Options) -> CopyResult
             let dest = construct_dest_path(source, target, target_type, options)
                 .unwrap_or_else(|_| target.to_path_buf());
 
-            if FileInformation::from_path(&dest, false).is_ok_and(|info| !info.is_symlink())
+            if FileInformation::from_path(&dest, true).is_ok()
+                && !fs::symlink_metadata(&dest).is_ok_and(|m| m.file_type().is_symlink())
                 // if both `source` and `dest` are symlinks, it should be considered as an overwrite.
                 || fs::metadata(source).is_ok()
                     && fs::symlink_metadata(source)?.file_type().is_symlink()

src/uucore/src/lib/features/fs.rs

@@ -166,20 +166,6 @@ impl FileInformation {
         #[cfg(any(target_os = "netbsd", not(target_pointer_width = "64")))]
         return self.0.st_ino.into();
     }
-
-    // The concept of symlinks doesn't map exactly to Windows (which has reparse
-    // points, junctions, etc.), so having our own method lets us control what
-    // we consider a symlink across platforms.
-    #[cfg(unix)]
-    pub fn is_symlink(&self) -> bool {
-        (self.0.st_mode as mode_t & S_IFMT) == S_IFLNK
-    }
-
-    #[cfg(windows)]
-    pub fn is_symlink(&self) -> bool {
-        use windows_sys::Win32::Storage::FileSystem::FILE_ATTRIBUTE_REPARSE_POINT;
-        self.0.file_attributes() & u64::from(FILE_ATTRIBUTE_REPARSE_POINT) != 0
-    }
 }
 
 #[cfg(unix)]