Skip to content

Conversation

asomers
Copy link
Member

@asomers asomers commented Jul 18, 2025

This test deliberately makes an invalid syscall. Rust no longer allows us to even construct an invalid enum (even with unsafe), so alter the test to get EBADF instead of EINVAL.

What does this PR do

Checklist:

  • I have read CONTRIBUTING.md
  • I have written necessary tests and rustdoc comments
  • A change log has been added if this PR modifies nix's API

This test deliberately makes an invalid syscall.  Rust no longer allows
us to even construct an invalid enum (even with `unsafe`), so alter the
test to get EBADF instead of EINVAL.
@asomers asomers requested a review from SteveLauC July 20, 2025 00:07
@asomers asomers added this pull request to the merge queue Aug 7, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 7, 2025
This unresolved link was only revealed by a recent change to the nightly
compiler.
@asomers asomers added this pull request to the merge queue Aug 17, 2025
Merged via the queue into nix-rust:master with commit ca3a5b6 Aug 17, 2025
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant