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

Test getrandom_uninit #474

Closed
wants to merge 2 commits into from
Closed

Test getrandom_uninit #474

wants to merge 2 commits into from

Conversation

josephlr
Copy link
Member

@josephlr josephlr commented Jun 12, 2024

Depends on #471, Alternative to #462

I verified that running the tests with MSAN:

RUSTFLAGS="-Zsanitizer=memory" cargo +nightly test -Zbuild-std --target=x86_64-unknown-linux-gnu

succeeds before this PR and fails with this PR, so we are doing the right thing here.

@josephlr josephlr mentioned this pull request Jun 12, 2024
@josephlr josephlr force-pushed the test_uninit branch 3 times, most recently from 05baf46 to 51989f6 Compare June 19, 2024 07:45
josephlr added 2 commits June 19, 2024 03:25
Signed-off-by: Joe Richey <[email protected]>
Signed-off-by: Joe Richey <[email protected]>
@josephlr josephlr marked this pull request as ready for review June 19, 2024 10:28
@josephlr josephlr requested a review from newpavlov June 19, 2024 10:28
@josephlr
Copy link
Member Author

Closing to keep conversation in #471

@josephlr josephlr closed this Jun 20, 2024
@newpavlov newpavlov deleted the test_uninit branch June 20, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant