Use AsFd in copy_file_range

Author: asomers

CHANGELOG.md

@@ -15,7 +15,7 @@ This project adheres to [Semantic Versioning](https://semver.org/).
   ([#1966])(https://github.com/nix-rust/nix/pull/1966)
 - Added `LocalPeerPid` to `nix::sys::socket::sockopt` for macOS. ([#1967](https://github.com/nix-rust/nix/pull/1967))
 - Exposed `copy_file_range` on FreeBSD
-  (#[???](https://github.com/nix-rust/nix/pull/???))
+  (#[1906](https://github.com/nix-rust/nix/pull/1906))
 
 ### Changed
 
@@ -27,6 +27,15 @@ This project adheres to [Semantic Versioning](https://semver.org/).
   users no longer need to manually close the file descriptors in these types.
   ([#1921](https://github.com/nix-rust/nix/pull/1921))
 
+- Implemented I/O safety.  Many public functions argument and return types have
+  changed:
+  | Original Type | New Type              |
+  | ------------- | --------------------- |
+  | AsRawFd       | AsFd                  |
+  | RawFd         | BorrowedFd or OwnedFd |
+
+  (#[1906](https://github.com/nix-rust/nix/pull/1906))
+
 ### Fixed
 - Fix `SockaddrIn6` bug that was swapping flowinfo and scope_id byte ordering.
   ([#1964](https://github.com/nix-rust/nix/pull/1964))

src/fcntl.rs

@@ -5,11 +5,15 @@ use std::ffi::OsString;
 use std::os::raw;
 use std::os::unix::ffi::OsStringExt;
 use std::os::unix::io::RawFd;
+// For splice and copy_file_range
+#[cfg(any(target_os = "android", target_os = "freebsd", target_os = "linux"))]
+use std::{
+    os::unix::io::{AsRawFd, AsFd},
+    ptr
+};
 
 #[cfg(feature = "fs")]
 use crate::{sys::stat::Mode, NixPath, Result};
-#[cfg(any(target_os = "android", target_os = "freebsd", target_os = "linux"))]
-use std::ptr; // For splice and copy_file_range
 
 #[cfg(any(
     target_os = "linux",
@@ -627,10 +631,10 @@ type type_of_off = libc::off_t;
 /// On successful completion the number of bytes actually copied will be
 /// returned.
 #[cfg(any(target_os = "android", target_os = "freebsd", target_os = "linux"))]
-pub fn copy_file_range(
-    fd_in: RawFd,
+pub fn copy_file_range<Fd1: AsFd, Fd2: AsFd>(
+    fd_in: Fd1,
     off_in: Option<&mut type_of_off>,
-    fd_out: RawFd,
+    fd_out: Fd2,
     off_out: Option<&mut type_of_off>,
     len: usize,
 ) -> Result<usize> {
@@ -643,9 +647,9 @@ pub fn copy_file_range(
 
     let ret = unsafe {
         libc::copy_file_range(
-            fd_in,
+            fd_in.as_fd().as_raw_fd(),
             off_in,
-            fd_out,
+            fd_out.as_fd().as_raw_fd(),
             off_out,
             len,
             0,

test/test_fcntl.rs

@@ -243,7 +243,7 @@ fn test_readlink() {
 // QEMU does not support copy_file_range. Skip under qemu
 #[cfg_attr(qemu, ignore)]
 fn test_copy_file_range() {
-    use std::os::unix::io::AsRawFd;
+    use std::os::unix::io::AsFd;
     use nix::fcntl::copy_file_range;
 
     const CONTENTS: &[u8] = b"foobarbaz";
@@ -256,9 +256,9 @@ fn test_copy_file_range() {
 
     let mut from_offset: i64 = 3;
     copy_file_range(
-        tmp1.as_raw_fd(),
+        tmp1.as_fd(),
         Some(&mut from_offset),
-        tmp2.as_raw_fd(),
+        tmp2.as_fd(),
         None,
         3,
     )