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

miri: detect too large dynamically sized objects #64014

Merged
merged 5 commits into from
Aug 31, 2019

Conversation

RalfJung
Copy link
Member

Needed to make rust-lang/miri#929 pass.

r? @oli-obk

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 30, 2019
@oli-obk
Copy link
Contributor

oli-obk commented Aug 30, 2019

Is it possible to write a rustc test for this?

@RalfJung
Copy link
Member Author

I tried, but then I ran into #63952.

@RalfJung
Copy link
Member Author

Ah, I found a way to do it. Does this fit into one of the existing files?

@oli-obk
Copy link
Contributor

oli-obk commented Aug 30, 2019

ub-wide-ptr.rs looks like a good match to me.

r=me either as is or with the test merged into ub-wide-ptr.rs

@RalfJung
Copy link
Member Author

ub-wide-ptr.rs looks like a good match to me.

That won't work; that's a allow(const_err) test but this test here is a const_err (the error happens before we even start validation).

@bors r=oli-obk

@bors
Copy link
Contributor

bors commented Aug 30, 2019

📌 Commit bb34749 has been approved by oli-obk

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 30, 2019
Centril added a commit to Centril/rust that referenced this pull request Aug 30, 2019
 miri: detect too large dynamically sized objects

Needed to make rust-lang/miri#929 pass.

r? @oli-obk
bors added a commit that referenced this pull request Aug 30, 2019
Rollup of 7 pull requests

Successful merges:

 - #62957 (Match the loop examples)
 - #63600 (Merge oli-obk mail addresses)
 - #63684 (Constify LinkedList new function)
 - #63847 ([rustdoc] Fix system theme detection)
 - #63999 (Add missing links on AsRef trait)
 - #64014 ( miri: detect too large dynamically sized objects )
 - #64015 (some const-eval test tweaks)

Failed merges:

r? @ghost
@bors bors merged commit bb34749 into rust-lang:master Aug 31, 2019
@RalfJung RalfJung deleted the miri-slice branch September 2, 2019 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants