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
jail: convert several functions from int to bool #663
Conversation
An earlier version of this Pull Request was approved by jamie@, but it was never merged. |
sys/sys/jail.h
Outdated
@@ -413,17 +413,17 @@ struct vfsconf; | |||
*/ | |||
#define jailed(cred) (cred->cr_prison != &prison0) | |||
|
|||
int jailed_without_vnet(struct ucred *); | |||
boolt jailed_without_vnet(struct ucred *); |
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.
There seems to be a typo here. Should be just bool
, not boolt
.
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.
thanks for catching that
fixed!
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.
You are welcome, now we just need the changes to make it compile.
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.
looking okay now.
And would look a lot better with #668 merged.
47f46e6
to
4af6d0c
Compare
these functions exclusively return (0) and (1), so convert them to bool We also convert some networking related jail functions from int to bool some of which were returning an error that was never used. Differential Revision: https://reviews.freebsd.org/D29659
4af6d0c
to
6f3680a
Compare
looks good... sorry it took so long to get into the tree... |
these functions exclusively return (0) and (1), so convert them to bool We also convert some networking related jail functions from int to bool some of which were returning an error that was never used. Differential Revision: https://reviews.freebsd.org/D29659 Reviewed by: imp, jamie (earlier version) Pull Request: #663
these functions exclusively return (0) and (1), so convert them to bool We also convert some networking related jail functions from int to bool some of which were returning an error that was never used. Differential Revision: https://reviews.freebsd.org/D29659 Reviewed by: imp, jamie (earlier version) Pull Request: freebsd/freebsd-src#663
these functions exclusively return (0) and (1), so convert them to bool
We also convert some networking related jail functions from int to bool some of which were returning an error that was never used.
Differential Revision: https://reviews.freebsd.org/D29659