Skip to content

Add a case to list of 'StackedBorrows violations' #1445

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jun 8, 2020

Conversation

JOE1994
Copy link
Contributor

@JOE1994 JOE1994 commented Jun 8, 2020

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 👍

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](rust-lang#1225 (comment)))

Thank you for reviewing 👍
@RalfJung
Copy link
Member

RalfJung commented Jun 8, 2020

Thanks!
@bors r+

@bors
Copy link
Contributor

bors commented Jun 8, 2020

📌 Commit ea4a558 has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Jun 8, 2020

⌛ Testing commit ea4a558 with merge 09b5755...

@bors
Copy link
Contributor

bors commented Jun 8, 2020

☀️ Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 09b5755 to master...

@bors bors merged commit 09b5755 into rust-lang:master Jun 8, 2020
@JOE1994 JOE1994 deleted the patch-1 branch June 9, 2020 01:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants