dpward / rpms / sssd

Forked from rpms/sssd 3 years ago
Clone

Blame SOURCES/0068-SDAP-sysdb_search_users-does-not-set-users_count-for.patch

b2d430
From 4e1111d869c8980f81a17b58844c48f1a342d774 Mon Sep 17 00:00:00 2001
b2d430
From: Lukas Slebodnik <lslebodn@redhat.com>
b2d430
Date: Thu, 4 Aug 2016 08:50:50 +0200
b2d430
Subject: [PATCH 68/74] SDAP: sysdb_search_users does not set users_count for
b2d430
 failures
b2d430
MIME-Version: 1.0
b2d430
Content-Type: text/plain; charset=UTF-8
b2d430
Content-Transfer-Encoding: 8bit
b2d430
b2d430
==32577== Conditional jump or move depends on uninitialised value(s)
b2d430
==32577==    at 0x140DCE10: sdap_process_missing_member_2307 (sdap_async_groups.c:1556)
b2d430
==32577==    by 0x140DCE10: sdap_process_group_members_2307 (sdap_async_groups.c:1625)
b2d430
==32577==    by 0x140DCE10: sdap_process_group_send (sdap_async_groups.c:1298)
b2d430
==32577==    by 0x140DCE10: sdap_get_groups_process (sdap_async_groups.c:2130)
b2d430
==32577==    by 0x140CFDA8: generic_ext_search_handler.isra.3 (sdap_async.c:1688)
b2d430
==32577==    by 0x140D2416: sdap_get_generic_op_finished (sdap_async.c:1578)
b2d430
==32577==    by 0x140D0DFC: sdap_process_message (sdap_async.c:353)
b2d430
==32577==    by 0x140D0DFC: sdap_process_result (sdap_async.c:197)
b2d430
==32577==    by 0x8BF1B4E: tevent_common_loop_timer_delay (tevent_timed.c:341)
b2d430
==32577==    by 0x8BF2B59: epoll_event_loop_once (tevent_epoll.c:911)
b2d430
==32577==    by 0x8BF1256: std_event_loop_once (tevent_standard.c:114)
b2d430
==32577==    by 0x8BED40C: _tevent_loop_once (tevent.c:533)
b2d430
==32577==    by 0x8BED5AA: tevent_common_loop_wait (tevent.c:637)
b2d430
==32577==    by 0x8BF11F6: std_event_loop_wait (tevent_standard.c:140)
b2d430
==32577==    by 0x529DD02: server_loop (server.c:702)
b2d430
==32577==    by 0x110951: main (data_provider_be.c:587)
b2d430
b2d430
Resolves:
b2d430
https://fedorahosted.org/sssd/ticket/3121
b2d430
b2d430
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
b2d430
---
b2d430
 src/providers/ldap/sdap_async_groups.c | 2 +-
b2d430
 1 file changed, 1 insertion(+), 1 deletion(-)
b2d430
b2d430
diff --git a/src/providers/ldap/sdap_async_groups.c b/src/providers/ldap/sdap_async_groups.c
b2d430
index f19b68b8c403734f88b51a411ba0d009977d3491..72760b75acae4cb6ce15c72f16dae8e859d89847 100644
b2d430
--- a/src/providers/ldap/sdap_async_groups.c
b2d430
+++ b/src/providers/ldap/sdap_async_groups.c
b2d430
@@ -1553,7 +1553,7 @@ sdap_process_missing_member_2307(struct sdap_process_group_state *state,
b2d430
         if (ret != EOK) {
b2d430
             DEBUG(SSSDBG_OP_FAILURE, "Could not add group member %s\n", username);
b2d430
         }
b2d430
-    } else if (ret == ENOENT || count == 0) {
b2d430
+    } else if (ret == ENOENT) {
b2d430
         /* The entry really does not exist, add a ghost */
b2d430
         DEBUG(SSSDBG_TRACE_FUNC, "Adding a ghost entry\n");
b2d430
         ret = sdap_add_group_member_2307(state->ghost_dns, member_name);
b2d430
-- 
b2d430
2.4.11
b2d430