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

Use non-null pointer for size 0 posix memalign #3600

Merged
merged 1 commit into from
May 12, 2024

Conversation

tiif
Copy link
Contributor

@tiif tiif commented May 11, 2024

Fixes #3576

tests/pass-dep/libc/libc-mem.rs Outdated Show resolved Hide resolved
@tiif tiif marked this pull request as ready for review May 11, 2024 22:07
@tiif
Copy link
Contributor Author

tiif commented May 11, 2024

@rustbot ready

@rustbot rustbot added the S-waiting-on-review Status: Waiting for a review to complete label May 11, 2024
@RalfJung
Copy link
Member

LGTM, thanks, just one comment nit.

Then please squash the commits.

@tiif tiif force-pushed the non-null-posix-memalign branch from df91db1 to e1222e3 Compare May 12, 2024 07:02
@tiif
Copy link
Contributor Author

tiif commented May 12, 2024

As always, thanks for the prompt review :)

@RalfJung
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented May 12, 2024

📌 Commit e1222e3 has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented May 12, 2024

⌛ Testing commit e1222e3 with merge cda82c7...

@bors
Copy link
Collaborator

bors commented May 12, 2024

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

@bors bors merged commit cda82c7 into rust-lang:master May 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Waiting for a review to complete
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make malloc(0) and posix_memalign(0) return a non-null pointer
4 participants