-
Notifications
You must be signed in to change notification settings - Fork 1.2k
solaris based systems priv api flags updates #2647
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
Conversation
|
r? @Amanieu (rust-highfive has picked a reviewer for you, use r? to override) |
src/unix/solarish/mod.rs
Outdated
| pub const NET_MAC_AWARE_INHERIT: ::c_uint = 0x0020; | ||
| pub const PRIV_AWARE_RESET: ::c_uint = 0x0040; | ||
| pub const PRIV_XPOLICY: ::c_uint = 0x0080; | ||
| pub const PRIV_PFEXEC: ::c_uint = 0x0080; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be 0x0100?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch bad c/p :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you run the tests? They generally catch transcription errors like this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it does I do in other platforms but in my illumos VM tests get stuck and mostly I have to reboot the VM.
cff60d9 to
bc405e5
Compare
|
@bors r+ |
|
📌 Commit bc405e5 has been approved by |
|
☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13 |
No description provided.