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

Add portable-atomic-util bug to "bugs found" list #3233

Merged
merged 1 commit into from Jan 25, 2024

Conversation

RalfJung
Copy link
Member

At least, reading https://notgull.net/cautionary-unsafe-tale/ it seems fair to say Miri found this bug. @notgull please let me know if you are okay with having this listed here.

@taiki-e
Copy link
Member

taiki-e commented Dec 21, 2023

It is not portable-atomic bug, and it is portable-atomic-util bug. They are not the same crate and have different actual authors.

@RalfJung RalfJung changed the title Add portable-atomic bug to "bugs found" list Add portable-atomic/util bug to "bugs found" list Dec 21, 2023
@RalfJung RalfJung changed the title Add portable-atomic/util bug to "bugs found" list Add portable-atomic-util bug to "bugs found" list Dec 21, 2023
@RalfJung
Copy link
Member Author

Ah, sorry for that. Fixed.

@RalfJung
Copy link
Member Author

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 25, 2024

📌 Commit d919cc8 has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jan 25, 2024

⌛ Testing commit d919cc8 with merge a225cbd...

bors added a commit that referenced this pull request Jan 25, 2024
Add portable-atomic-util bug to "bugs found" list

At least, reading https://notgull.net/cautionary-unsafe-tale/ it seems fair to say Miri found this bug. `@notgull` please let me know if you are okay with having this listed here.
@bors
Copy link
Collaborator

bors commented Jan 25, 2024

💥 Test timed out

@RalfJung
Copy link
Member Author

@bors retry

@bors
Copy link
Collaborator

bors commented Jan 25, 2024

⌛ Testing commit d919cc8 with merge 580f0b7...

@bors
Copy link
Collaborator

bors commented Jan 25, 2024

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing 580f0b7 to master...

@bors bors merged commit 580f0b7 into rust-lang:master Jan 25, 2024
8 checks passed
@RalfJung RalfJung deleted the trophy branch January 28, 2024 17:07
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

3 participants