xattr

Signed-off-by: Hiroyuki Moriya <[email protected]>

experiment/selinux/Cargo.toml

@@ -13,4 +13,6 @@ keywords = ["youki", "container", "selinux"]
 
 [dependencies]
 nix = { version = "0.29.0", features = ["process", "fs"] }
+rustix = { version = "0.38.34", features = ["fs"] }
+tempfile = "3.10.1"
 thiserror = "1.0.61"

experiment/selinux/src/lib.rs

@@ -1,2 +1,2 @@
 pub mod selinux;
-pub mod xattr;
+pub mod xattrs;

experiment/selinux/src/selinux.rs

@@ -1,4 +1,4 @@
-use crate::xattr::*;
+use crate::xattrs::*;
 use nix::unistd::gettid;
 use nix::sys::statfs;
 use nix::errno::Errno;
@@ -75,10 +75,10 @@ impl SELinux {
         }
 
         loop {
-            match set_xattr(fpath, XATTR_NAME_SELINUX, label.as_bytes(), 0) {
+            match set_xattr(fpath, XATTR_NAME_SELINUX, label.as_bytes()) {
                 Ok(_) => break,
-                // TODO: This line will be fixed after implementing set_xattr.
-                Err(EINTR) => continue,
+                // When a system call is interrupted by a signal, it needs to be retried.
+                Err(XattrError::EINTR(_)) => continue,
                 Err(e) => {
                     return Err(SELinuxError::SetFileLabel(
                         format!("set_xattr failed: {}", e),
@@ -98,10 +98,10 @@ impl SELinux {
         }
 
         loop {
-            match lset_xattr(fpath, XATTR_NAME_SELINUX, label.as_bytes(), 0) {
+            match lset_xattr(fpath, XATTR_NAME_SELINUX, label.as_bytes()) {
                 Ok(_) => break,
-                // TODO: This line will be fixed after implementing lset_xattr.
-                Err(EINTR) => continue,
+                // When a system call is interrupted by a signal, it needs to be retried.
+                Err(XattrError::EINTR(_)) => continue,
                 Err(e) => {
                     return Err(SELinuxError::LSetFileLabel(format!("lset_xattr failed: {}", e)));
                 }
@@ -366,7 +366,7 @@ mod tests {
     use crate::selinux::*;
     use std::fs::{self, File};
     use std::path::Path;
-    use std::io::Write;
+    use std::io::Write;  
 
     fn create_temp_file(content: &[u8], file_name: &str) {
         let path = Path::new(file_name);
@@ -465,5 +465,5 @@ mod tests {
                 assert!(result.is_err(), "Expected Err, but got Ok");                
             }
         }
-    }
+    }   
 }