Skip to content

remove wrong packed struct test#39692

Merged
bors merged 2 commits into
rust-lang:masterfrom
oli-obk:patch-2
Feb 11, 2017
Merged

remove wrong packed struct test#39692
bors merged 2 commits into
rust-lang:masterfrom
oli-obk:patch-2

Conversation

@oli-obk

@oli-obk oli-obk commented Feb 9, 2017

Copy link
Copy Markdown
Contributor

This UB was found by running the test under Miri which rejects these unsafe unaligned loads. 😄

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@aturon

aturon commented Feb 10, 2017

Copy link
Copy Markdown
Contributor

Oy! Good catch :-)

@bors: r+ rollup

@bors

bors commented Feb 10, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit c7f9811 has been approved by aturon

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 10, 2017
remove wrong packed struct test

This UB was found by running the test under [Miri](https://github.com/solson/miri) which rejects these unsafe unaligned loads. 😄
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 11, 2017
remove wrong packed struct test

This UB was found by running the test under [Miri](https://github.com/solson/miri) which rejects these unsafe unaligned loads. 😄
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 11, 2017
remove wrong packed struct test

This UB was found by running the test under [Miri](https://github.com/solson/miri) which rejects these unsafe unaligned loads. 😄
bors added a commit that referenced this pull request Feb 11, 2017
Rollup of 9 pull requests

- Successful merges: #39174, #39660, #39676, #39692, #39701, #39710, #39721, #39724, #39725
- Failed merges:
@bors bors merged commit c7f9811 into rust-lang:master Feb 11, 2017
@oli-obk oli-obk deleted the patch-2 branch June 15, 2020 15:25
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.

4 participants