cp: move copy_xattrs_skip_selinux to uucore

Author: ChrisDryden

src/uu/cp/src/cp.rs

@@ -16,7 +16,7 @@ use std::os::unix::net::UnixListener;
 use std::path::{Path, PathBuf, StripPrefixError};
 use std::{fmt, io};
 #[cfg(all(unix, not(target_os = "android")))]
-use uucore::fsxattr::copy_xattrs;
+use uucore::fsxattr::{copy_xattrs, copy_xattrs_skip_selinux};
 use uucore::translate;
 
 use clap::{Arg, ArgAction, ArgMatches, Command, builder::ValueParser, value_parser};
@@ -1748,21 +1748,6 @@ fn copy_extended_attrs(source: &Path, dest: &Path, skip_selinux: bool) -> CopyRe
     Ok(())
 }
 
-/// Copy extended attributes but skip security.selinux
-#[cfg(all(unix, not(target_os = "android")))]
-fn copy_xattrs_skip_selinux(source: &Path, dest: &Path) -> std::io::Result<()> {
-    for attr_name in xattr::list(source)? {
-        // Skip security.selinux when -Z is used to set default context
-        if attr_name.to_string_lossy() == "security.selinux" {
-            continue;
-        }
-        if let Some(value) = xattr::get(source, &attr_name)? {
-            xattr::set(dest, &attr_name, &value)?;
-        }
-    }
-    Ok(())
-}
-
 /// Copy the specified attributes from one path to another.
 /// If `skip_selinux_xattr` is true, the security.selinux xattr will not be copied
 /// (used when -Z is specified to set the default context instead).

src/uucore/src/lib/features/fsxattr.rs

@@ -32,6 +32,19 @@ pub fn copy_xattrs<P: AsRef<Path>>(source: P, dest: P) -> std::io::Result<()> {
     Ok(())
 }
 
+/// Like `copy_xattrs`, but skips the security.selinux attribute.
+#[cfg(unix)]
+pub fn copy_xattrs_skip_selinux<P: AsRef<Path>>(source: P, dest: P) -> std::io::Result<()> {
+    for attr_name in xattr::list(&source)? {
+        if attr_name.to_string_lossy() != "security.selinux" {
+            if let Some(value) = xattr::get(&source, &attr_name)? {
+                xattr::set(&dest, &attr_name, &value)?;
+            }
+        }
+    }
+    Ok(())
+}
+
 /// Retrieves the extended attributes (xattrs) of a given file or directory.
 ///
 /// # Arguments