Skip to content

Commit

Permalink
Rollup merge of #71921 - RalfJung:open-mode, r=hanna-kruppe
Browse files Browse the repository at this point in the history
explain the types used in the open64 call

Fixes #71915, where I learned about this quirk. I don't actually know what I am talking about here. ;)
  • Loading branch information
Dylan-DPC authored May 6, 2020
2 parents f29a923 + fbf791b commit b86620a
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/libstd/sys/unix/fs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -703,6 +703,10 @@ impl File {
| opts.get_access_mode()?
| opts.get_creation_mode()?
| (opts.custom_flags as c_int & !libc::O_ACCMODE);
// The third argument of `open64` is documented to have type `mode_t`. On
// some platforms (like macOS, where `open64` is actually `open`), `mode_t` is `u16`.
// However, since this is a variadic function, C integer promotion rules mean that on
// the ABI level, this still gets passed as `c_int` (aka `u32` on Unix platforms).
let fd = cvt_r(|| unsafe { open64(path.as_ptr(), flags, opts.mode as c_int) })?;
let fd = FileDesc::new(fd);

Expand Down

0 comments on commit b86620a

Please sign in to comment.