diff options
author | Ed Schouten <ed@FreeBSD.org> | 2016-06-01 20:45:21 +0000 |
---|---|---|
committer | Ed Schouten <ed@FreeBSD.org> | 2016-06-01 20:45:21 +0000 |
commit | 2ca9ffa9bb2da931f3055a5bdbf58ce956d36a5f (patch) | |
tree | 3dca84308e48eb52bddfcccbc1ac4ecfb0a1dd01 /lib/libcasper | |
parent | ea580d0b45f1ab5573c1fc1221a1b1629ef9fa49 (diff) | |
download | src-2ca9ffa9bb2da931f3055a5bdbf58ce956d36a5f.tar.gz src-2ca9ffa9bb2da931f3055a5bdbf58ce956d36a5f.zip |
Don't call setgrent() in an unportable way.
For FreeBSD 12, I'm considering updating setgrent() to have a function
prototype that conforms to POSIX. FreeBSD seems to be the only operating
system that lets setgrent() return an integer. It's also inconsistent
with setpwent().
It looks like our libcasper depends on setgrent() returning an integer.
Get rid of that.
Reviewed by: oshogbo
Differential Revision: https://reviews.freebsd.org/D6659
Notes
Notes:
svn path=/head/; revision=301167
Diffstat (limited to 'lib/libcasper')
-rw-r--r-- | lib/libcasper/services/cap_grp/cap_grp.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/libcasper/services/cap_grp/cap_grp.c b/lib/libcasper/services/cap_grp/cap_grp.c index d358cea95882..54a022f76cec 100644 --- a/lib/libcasper/services/cap_grp/cap_grp.c +++ b/lib/libcasper/services/cap_grp/cap_grp.c @@ -703,7 +703,9 @@ grp_setgrent(const nvlist_t *limits __unused, const nvlist_t *nvlin __unused, nvlist_t *nvlout __unused) { - return (setgrent() == 0 ? EFAULT : 0); + setgrent(); + + return (0); } static int |