diff options
author | kientzle <kientzle@FreeBSD.org> | 2004-05-17 22:15:49 +0000 |
---|---|---|
committer | kientzle <kientzle@FreeBSD.org> | 2004-05-17 22:15:49 +0000 |
commit | 330e353d70b4e66999db3be3f44e56bfe0515cb1 (patch) | |
tree | cb7059c9c4c0cb0b291db3e1b90f902db9e5cff6 /lib | |
parent | 4070a4a8d4f684242bddd0ac9a1e6c999dd11ded (diff) | |
download | FreeBSD-src-330e353d70b4e66999db3be3f44e56bfe0515cb1.zip FreeBSD-src-330e353d70b4e66999db3be3f44e56bfe0515cb1.tar.gz |
getgrent() and friends should set errno if there is an error.
Also, clarify the manpage description of when errno is set and
explain that clients should set errno=0 first if they want useful
error information.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/libc/gen/getgrent.3 | 6 | ||||
-rw-r--r-- | lib/libc/gen/getgrent.c | 3 |
2 files changed, 8 insertions, 1 deletions
diff --git a/lib/libc/gen/getgrent.3 b/lib/libc/gen/getgrent.3 index be37567..838e107 100644 --- a/lib/libc/gen/getgrent.3 +++ b/lib/libc/gen/getgrent.3 @@ -168,9 +168,13 @@ and return a pointer to a group structure on success or .Dv NULL if the entry is not found or if an error occurs. -In the latter case, +If an error does occur, .Va errno will be set. +Note that programs must explicitly set +.Va errno +to zero before calling any of these functions if they need to +distinguish between a non-existent entry and an error. The functions .Fn getgrent_r , .Fn getgrnam_r , diff --git a/lib/libc/gen/getgrent.c b/lib/libc/gen/getgrent.c index d834d09..c9b7474 100644 --- a/lib/libc/gen/getgrent.c +++ b/lib/libc/gen/getgrent.c @@ -307,6 +307,7 @@ getgr(int (*fn)(union key, struct group *, char *, size_t, struct group **), free(grp_storage); if ((grp_storage_size << 1) > GRP_STORAGE_MAX) { grp_storage = NULL; + errno = ERANGE; return (NULL); } grp_storage_size <<= 1; @@ -315,6 +316,8 @@ getgr(int (*fn)(union key, struct group *, char *, size_t, struct group **), return (NULL); } } while (res == NULL && rv == ERANGE); + if (rv != 0) + errno = rv; return (res); } |