ln: use rustix::fs::symlink instead of libc + CString on WASI

Author: DePasqualeOrg

Cargo.lock

@@ -20,7 +20,7 @@ path = "src/ln.rs"
 
 [dependencies]
 clap = { workspace = true }
-libc = { workspace = true }
+rustix = { workspace = true, features = ["fs"] }
 uucore = { workspace = true, features = ["backup-control", "fs"] }
 thiserror = { workspace = true }
 fluent = { workspace = true }

src/uu/ln/Cargo.toml

@@ -19,8 +19,6 @@ use std::ffi::OsString;
 use std::fs;
 use thiserror::Error;
 
-#[cfg(target_os = "wasi")]
-use std::ffi::CString;
 #[cfg(target_os = "wasi")]
 use std::io;
 #[cfg(any(unix, target_os = "redox"))]
@@ -495,16 +493,5 @@ pub fn symlink<P1: AsRef<Path>, P2: AsRef<Path>>(src: P1, dst: P2) -> std::io::R
 
 #[cfg(target_os = "wasi")]
 pub fn symlink<P1: AsRef<Path>, P2: AsRef<Path>>(src: P1, dst: P2) -> io::Result<()> {
-    use std::os::wasi::ffi::OsStrExt;
-
-    let src_c = CString::new(src.as_ref().as_os_str().as_bytes())
-        .map_err(|e| io::Error::new(io::ErrorKind::InvalidInput, e))?;
-    let dst_c = CString::new(dst.as_ref().as_os_str().as_bytes())
-        .map_err(|e| io::Error::new(io::ErrorKind::InvalidInput, e))?;
-
-    if unsafe { libc::symlink(src_c.as_ptr(), dst_c.as_ptr()) } == 0 {
-        Ok(())
-    } else {
-        Err(io::Error::last_os_error())
-    }
+    rustix::fs::symlink(src.as_ref(), dst.as_ref()).map_err(io::Error::from)
 }