|
|
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 |
|