Skip to content

openbsd: make WIFCONTINUED() safe#2035

Merged
bors merged 1 commit intorust-lang:masterfrom
semarie:openbsd-safesignal
Jan 18, 2021
Merged

openbsd: make WIFCONTINUED() safe#2035
bors merged 1 commit intorust-lang:masterfrom
semarie:openbsd-safesignal

Conversation

@semarie
Copy link
Copy Markdown
Contributor

@semarie semarie commented Jan 16, 2021

it seems it was a missed part of #1891

Found it because rustc -nightly started to use libc::WIFCONTINUED() recently in a safe context and the build fail to build on OpenBSD.

Fixes #2001

@rust-highfive
Copy link
Copy Markdown

r? @JohnTitor

(rust-highfive has picked a reviewer for you, use r? to override)

Copy link
Copy Markdown
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this!

@JohnTitor
Copy link
Copy Markdown
Member

@bors r+

@bors
Copy link
Copy Markdown
Contributor

bors commented Jan 18, 2021

📌 Commit cf0793a has been approved by JohnTitor

@bors
Copy link
Copy Markdown
Contributor

bors commented Jan 18, 2021

⌛ Testing commit cf0793a with merge 7f11cdb...

@bors
Copy link
Copy Markdown
Contributor

bors commented Jan 18, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 7f11cdb to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

WIFCONTINUED is unsafe on OpenBSD

4 participants