On Wed, Jan 08, 2014 at 10:07:08AM +0200, Alexander Bokovoy wrote:
On Wed, 08 Jan 2014, Jakub Hrozek wrote:
>Hi,
>
>the attached patch fixes #2191.
>From f12c4bb11ff35cdd785026aa571db8f659127763 Mon Sep 17 00:00:00 2001
>From: Jakub Hrozek <jhrozek(a)redhat.com>
>Date: Wed, 8 Jan 2014 08:11:46 +0100
>Subject: [PATCH] LDAP: Fix typo and use the right attribute map
>
>https://fedorahosted.org/sssd/ticket/2191
>
>There was a copy-n-paste bug in the code that resulted in using a wrong
>attribute map. This could lead to the primary name not being selected
>correctly.
>---
>src/providers/ldap/sdap.c | 2 +-
>1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/src/providers/ldap/sdap.c b/src/providers/ldap/sdap.c
>index
078326ad3614ed2cd41659ea279642a46a0e24e1..ddcf199b61311b69fde54c3ee25f2338ceb05576 100644
>--- a/src/providers/ldap/sdap.c
>+++ b/src/providers/ldap/sdap.c
>@@ -1246,7 +1246,7 @@ errno_t sdap_get_user_primary_name(TALLOC_CTX *memctx,
> const char **_user_name)
>{
> return sdap_get_primary_name(memctx,
>- opts->group_map[SDAP_AT_USER_NAME].name,
>+ opts->user_map[SDAP_AT_USER_NAME].name,
> attrs, dom, _user_name);
>}
ACK.
Pushed to master and sssd-1-11 (yesterday already, I just forgot to send
the push mail as I rushed to include the fix in downstream)