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

C mem function shims: consistently treat "invalid" pointers as UB #2972

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Jul 9, 2023

Depends on rust-lang/rust#113435.

@RalfJung RalfJung marked this pull request as draft July 9, 2023 14:23
@RalfJung RalfJung added the S-blocked Status: blocked on something happening somewhere else label Aug 15, 2023
@RalfJung RalfJung removed the S-blocked Status: blocked on something happening somewhere else label Aug 15, 2023
@RalfJung RalfJung marked this pull request as ready for review August 15, 2023 11:34
@RalfJung
Copy link
Member Author

rust-lang/rust#114382 has landed, so this should be good to go.
@bors r+

@bors
Copy link
Collaborator

bors commented Aug 15, 2023

📌 Commit a71aef1 has been approved by RalfJung

It is now in the queue for this repository.

@RalfJung RalfJung changed the title Draft: C mem function shims: consistently treat "invalid" pointers as UB C mem function shims: consistently treat "invalid" pointers as UB Aug 15, 2023
@bors
Copy link
Collaborator

bors commented Aug 15, 2023

⌛ Testing commit a71aef1 with merge f0b5f98...

@bors
Copy link
Collaborator

bors commented Aug 15, 2023

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing f0b5f98 to master...

@bors bors merged commit f0b5f98 into rust-lang:master Aug 15, 2023
8 checks passed
@bors bors mentioned this pull request Aug 15, 2023
@RalfJung RalfJung deleted the c-mem-functions branch August 15, 2023 13:15
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.

None yet

2 participants