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 case of libstd fix from Stacked Borrows #234

Closed
wants to merge 1 commit into from
Closed

add case of libstd fix from Stacked Borrows #234

wants to merge 1 commit into from

Conversation

JOE1994
Copy link
Contributor

@JOE1994 JOE1994 commented Jun 8, 2020

This PR adds a new case to the list of
'adjustments made to libstd (due to Stacked Borrows violation)'.

A small fix was made to libstd in rust-lang/rust#70479 (back in March).
(Miri reported UB due to Stacked Borrows violation - link to Miri error log)

Thank you for reviewing 👍

@RalfJung
Copy link
Member

RalfJung commented Jun 8, 2020

Oh, I forgot about this list, good catch!

But we also have a list at https://github.com/rust-lang/miri#bugs-found-by-miri, which is better maintained. Maybe it would make more sense to link there instead? (Looks like this one is also missing from that list, though.)

@JOE1994
Copy link
Contributor Author

JOE1994 commented Jun 8, 2020

Oh, I forgot about this list, good catch!

But we also have a list at https://github.com/rust-lang/miri#bugs-found-by-miri, which is better maintained. Maybe it would make more sense to link there instead? (Looks like this one is also missing from that list, though.)

Then I will close this PR and make a PR to Miri 👍

@JOE1994 JOE1994 closed this Jun 8, 2020
@RalfJung
Copy link
Member

RalfJung commented Jun 8, 2020

@JOE1994 thanks. :) Can you also do a PR here to link to Miri instead of having a separate list?

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.

2 participants