Skip to content

Add unistd::close_range() #2153

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
cpick wants to merge 5 commits into from
Closed

Add unistd::close_range() #2153

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
82d3c02
Add `unistd::close_range()`
Sep 29, 2023
5a380d4
Add `close_range` PR link to changelog
Oct 4, 2023
46bd488
Add a double-close warning to `close_range()`
Oct 4, 2023
3e5d881
Mark `close_range` I/O unsafe
Oct 5, 2023
6129a9e
Add `# Safety` to `close_range`
Oct 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Add # Safety to close_range
  • Loading branch information
Chris Pick committed Oct 5, 2023
commit 6129a9e3790e2818a2c5f3a43b767bc73106c274
2 changes: 2 additions & 0 deletions src/unistd.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1112,6 +1112,8 @@ libc_bitflags! {
/// Close a range of raw file descriptors.
/// [close_range(2)](https://man7.org/linux/man-pages/man2/close_range.2.html).
///
/// # Safety
///
/// Be aware that many Rust types implicitly close-on-drop, including
/// `std::fs::File`. Explicitly closing them with this method too can result in
/// a double-close condition, which can cause confusing `EBADF` errors in
Expand Down