From e3c994b7b779e6d6d6d125ee28d30bd139590f49 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Tue, 11 Mar 2014 17:37:47 +0100 Subject: No point in searching for gid if we already know the group should be filtered Reviewed-by: Sumit Bose --- src/providers/ldap/sdap_async_nested_groups.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'src/providers') 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, -- cgit