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

fix UB in a test #63260

Merged
merged 3 commits into from
Aug 6, 2019
Merged

fix UB in a test #63260

merged 3 commits into from
Aug 6, 2019

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Aug 4, 2019

We used to compare two mutable references that were supposed to point to the same thing. That's no good.

Compare them as raw pointers instead.

@rust-highfive
Copy link
Collaborator

r? @Kimundi

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 4, 2019
src/libcore/tests/ptr.rs Outdated Show resolved Hide resolved
Co-Authored-By: Aleksey Kladov <aleksey.kladov@gmail.com>
@matklad
Copy link
Member

matklad commented Aug 5, 2019

r? matklad

@bors r+ rollup

(I am not an expert in this area, but change seems simple and fine to me, so I want to r+. Please tell me if I am using r+ right incorrectly :) )

@bors
Copy link
Contributor

bors commented Aug 5, 2019

📌 Commit 4e51ef7 has been approved by matklad

@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 5, 2019
@matklad
Copy link
Member

matklad commented Aug 5, 2019

r? @matklad

@rust-highfive

This comment has been minimized.

src/libcore/tests/ptr.rs Outdated Show resolved Hide resolved
@matklad
Copy link
Member

matklad commented Aug 5, 2019

bors r-

(I don't know if bors's smart enough to not schedule a merge is PR checks fail)

@matklad
Copy link
Member

matklad commented Aug 5, 2019

@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 5, 2019
@RalfJung
Copy link
Member Author

RalfJung commented Aug 5, 2019

(I don't know if bors's smart enough to not schedule a merge is PR checks fail)

It is not.

Co-Authored-By: Aleksey Kladov <aleksey.kladov@gmail.com>
@RalfJung
Copy link
Member Author

RalfJung commented Aug 5, 2019

@bors r=matklad

@bors
Copy link
Contributor

bors commented Aug 5, 2019

📌 Commit 90b95cf has been approved by matklad

@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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 5, 2019
@Centril
Copy link
Contributor

Centril commented Aug 5, 2019

r? @matklad

@rust-highfive rust-highfive assigned matklad and unassigned Kimundi Aug 5, 2019
Centril added a commit to Centril/rust that referenced this pull request Aug 5, 2019
fix UB in a test

We used to compare two mutable references that were supposed to point to the same thing. That's no good.

Compare them as raw pointers instead.
Centril added a commit to Centril/rust that referenced this pull request Aug 6, 2019
fix UB in a test

We used to compare two mutable references that were supposed to point to the same thing. That's no good.

Compare them as raw pointers instead.
Centril added a commit to Centril/rust that referenced this pull request Aug 6, 2019
fix UB in a test

We used to compare two mutable references that were supposed to point to the same thing. That's no good.

Compare them as raw pointers instead.
Centril added a commit to Centril/rust that referenced this pull request Aug 6, 2019
fix UB in a test

We used to compare two mutable references that were supposed to point to the same thing. That's no good.

Compare them as raw pointers instead.
bors added a commit that referenced this pull request Aug 6, 2019
Rollup of 14 pull requests

Successful merges:

 - #61457 (Implement DoubleEndedIterator for iter::{StepBy, Peekable, Take})
 - #63017 (Remove special code-path for handing unknown tokens)
 - #63184 (Explaining the reason why validation is performed in to_str of path.rs)
 - #63230 (Make use of possibly uninitialized data [E0381] a hard error)
 - #63260 (fix UB in a test)
 - #63264 (Revert "Rollup merge of #62696 - chocol4te:fix_#62194, r=estebank")
 - #63272 (Some more libsyntax::attr cleanup)
 - #63285 (Remove leftover AwaitOrigin)
 - #63287 (Don't store &Span)
 - #63293 (Clarify align_to's requirements and obligations)
 - #63295 (improve align_offset docs)
 - #63299 (Make qualify consts in_projection use PlaceRef)
 - #63312 (doc: fix broken sentence)
 - #63315 (Fix #63313)

Failed merges:

r? @ghost
@bors bors merged commit 90b95cf into rust-lang:master Aug 6, 2019
@RalfJung RalfJung deleted the ptr-test branch August 9, 2019 16:43
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.

6 participants