diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2014-03-11 17:37:47 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-07-23 21:27:00 +0200 |
commit | e3c994b7b779e6d6d6d125ee28d30bd139590f49 (patch) | |
tree | 0e982614ddb90c12425e4481e48aa22d2af4ff6b /src | |
parent | 9d876108620931e0941a115adf60bfd8d67459d9 (diff) | |
download | sssd-e3c994b7b779e6d6d6d125ee28d30bd139590f49.tar.gz sssd-e3c994b7b779e6d6d6d125ee28d30bd139590f49.tar.xz sssd-e3c994b7b779e6d6d6d125ee28d30bd139590f49.zip |
No point in searching for gid if we already know the group should be filtered
Reviewed-by: Sumit Bose <sbose@redhat.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/providers/ldap/sdap_async_nested_groups.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/providers/ldap/sdap_async_nested_groups.c b/src/providers/ldap/sdap_async_nested_groups.c index c521e0874..305afbc9d 100644 --- a/src/providers/ldap/sdap_async_nested_groups.c +++ b/src/providers/ldap/sdap_async_nested_groups.c @@ -265,9 +265,11 @@ sdap_nested_group_hash_group(struct sdap_nested_group_ctx *group_ctx, } } - ret = sysdb_attrs_get_uint32_t(group, map[SDAP_AT_GROUP_GID].sys_name, - &gid); - if (ret == ENOENT || (ret == EOK && gid == 0) || !posix_group) { + if (posix_group) { + ret = sysdb_attrs_get_uint32_t(group, map[SDAP_AT_GROUP_GID].sys_name, + &gid); + } + if (!posix_group || ret == ENOENT || (ret == EOK && gid == 0)) { DEBUG(SSSDBG_TRACE_ALL, "The group's gid was %s\n", ret == ENOENT ? "missing" : "zero"); DEBUG(SSSDBG_TRACE_INTERNAL, |