Hi,

we have a segfault in master caused by the ldap attribute map extension
patches. The attached patch fixes the segfault.

I'm not happy with how the current sdap_attr_map is represented, so I
filed https://fedorahosted.org/sssd/ticket/2336.
>From 03985423deb18e058f2a77e23006b446be59e9e8 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <[email protected]>
Date: Tue, 20 May 2014 17:04:51 +0200
Subject: [PATCH] AD: Initialize user_map_cnt in server mode

user_map_cnt was initialized when all the traditional back ends are
initialized. However, for the server mode, we simply copy the defaults
and the count was left zeroed, which led to crashes.

Down the road, we should consider tying the map and the attribute count
together (see ticket #2336)
---
 src/providers/ad/ad_common.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/providers/ad/ad_common.c b/src/providers/ad/ad_common.c
index 
2607e766dcbb754f20be1aa91e93de8f95a89f2c..67ded36ed45134fb79c0f8490e49406d428982ba
 100644
--- a/src/providers/ad/ad_common.c
+++ b/src/providers/ad/ad_common.c
@@ -71,6 +71,7 @@ ad_create_default_sdap_options(TALLOC_CTX *mem_ctx)
     if (ret != EOK) {
         goto fail;
     }
+    id_opts->user_map_cnt = SDAP_OPTS_USER;
 
     /* Group map */
     ret = sdap_copy_map(id_opts,
-- 
1.9.0

_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to