Skip to content

Commit

Permalink
Unrolled build for rust-lang#129409
Browse files Browse the repository at this point in the history
Rollup merge of rust-lang#129409 - grinapo:patch-1, r=Amanieu

Expand std::os::unix::fs::chown() doc with a warning

Include warning about losing setuid/gid when chowning, per POSIX.

It is about the underlying system call but it is rather useful to mention it in the help in case someone accidentally forgets (don't look at me :)).
  • Loading branch information
rust-timer authored Nov 28, 2024
2 parents eddb717 + 451c8cd commit 63e6ea3
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions library/std/src/os/unix/fs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -987,6 +987,11 @@ impl DirBuilderExt for fs::DirBuilder {
/// Changing the group typically requires either being the owner and a member of the group, or
/// having privileges.
///
/// Be aware that changing owner clears the `suid` and `sgid` permission bits in most cases
/// according to POSIX, usually even if the user is root. The sgid is not cleared when
/// the file is non-group-executable. See: <https://www.man7.org/linux/man-pages/man2/chown.2.html>
/// This call may also clear file capabilities, if there was any.
///
/// If called on a symbolic link, this will change the owner and group of the link target. To
/// change the owner and group of the link itself, see [`lchown`].
///
Expand Down

0 comments on commit 63e6ea3

Please sign in to comment.