diff options
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | nis/nss_compat/compat-initgroups.c | 17 |
2 files changed, 19 insertions, 4 deletions
@@ -1,5 +1,11 @@ 2007-11-22 Ulrich Drepper <drepper@redhat.com> + [BZ #5378] + * nis/nss_compat/compat-initgroups.c (getgrent_next_nss): Don't + use result of nss_getgrgid_r if nothing was found. For other + error return with a failure. + Partially based on a patch by Petr Baudis <pasky@suse.cz>. + * locale/programs/ld-collate.c (collate_read): Fix loop to match macro name. diff --git a/nis/nss_compat/compat-initgroups.c b/nis/nss_compat/compat-initgroups.c index fd16475ea9..76ca95d1e6 100644 --- a/nis/nss_compat/compat-initgroups.c +++ b/nis/nss_compat/compat-initgroups.c @@ -303,10 +303,19 @@ getgrent_next_nss (ent_t *ent, char *buffer, size_t buflen, const char *user, else tmpbuf = extend_alloca (tmpbuf, tmplen, 2 * tmplen); - if (!in_blacklist (grpbuf.gr_name, - strlen (grpbuf.gr_name), ent)) - check_and_add_group (user, group, start, size, groupsp, - limit, &grpbuf); + if (__builtin_expect (status != NSS_STATUS_NOTFOUND, 1)) + { + if (__builtin_expect (status != NSS_STATUS_SUCCESS, 0)) + { + free (mygroups); + return status; + } + + if (!in_blacklist (grpbuf.gr_name, + strlen (grpbuf.gr_name), ent)) + check_and_add_group (user, group, start, size, groupsp, + limit, &grpbuf); + } } free (mygroups); |