Skip to content
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.

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

Fix getsockopt zeroing memory #901

Merged
merged 1 commit into from Oct 26, 2023

Conversation

alexcrichton
Copy link
Member

This commit fixes an accidental regression from #858 where a previous call to core::mem::zeroed was replaced with MaybeUninit::uninit instead of MaybeUninit::zeroed.

This commit fixes an accidental regression from bytecodealliance#858 where a previous
call to `core::mem::zeroed` was replaced with `MaybeUninit::uninit`
instead of `MaybeUninit::zeroed`.
@sunfishcode
Copy link
Member

Thanks for tracking this down and fixing this!

@sunfishcode sunfishcode merged commit b01b482 into bytecodealliance:main Oct 26, 2023
43 checks passed
@alexcrichton alexcrichton deleted the fix-zeroed branch October 26, 2023 14:01
alexcrichton added a commit to alexcrichton/wasmtime that referenced this pull request Oct 26, 2023
github-merge-queue bot pushed a commit to bytecodealliance/wasmtime that referenced this pull request Oct 26, 2023
* Update some crates to align on latest `rustix`

This commit addresses some dependabot warnings showing up on the
Wasmtime repository by updating all dependencies to using the latest
`rustix` release.

* Debug CI

prtest:full

* Downgrade rustix to 0.38.14

Works around bytecodealliance/rustix#901
sunfishcode added a commit that referenced this pull request Oct 26, 2023
Following up on #900, add illumos to the platforms that may return a
Unix-domain socket address with a shorter string than the socklen
value.

And following up on #901, a another test for TCP_NODELAY.
sunfishcode added a commit that referenced this pull request Oct 26, 2023
Following up on #900, add illumos to the platforms that may return a
Unix-domain socket address with a shorter string than the socklen
value.

And following up on #901, a another test for TCP_NODELAY.
@sunfishcode
Copy link
Member

This is now released in rustix 0.38.21.

alexcrichton added a commit to alexcrichton/wasmtime that referenced this pull request Oct 26, 2023
Completes the upgrade from bytecodealliance#7370 with bytecodealliance/rustix#901 being
published to crates.io.
github-merge-queue bot pushed a commit to bytecodealliance/wasmtime that referenced this pull request Oct 26, 2023
Completes the upgrade from #7370 with bytecodealliance/rustix#901 being
published to crates.io.
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.

None yet

2 participants