[SSSD] [PATCH] Extend the LDAP attribute map with custom options

Lukas Slebodnik lslebodn at redhat.com
Fri May 2 21:23:19 UTC 2014


On (25/04/14 11:42), Jakub Hrozek wrote:
>On Wed, Apr 23, 2014 at 11:28:51AM -0400, Simo Sorce wrote:
>> On Wed, 2014-04-23 at 17:12 +0200, Jakub Hrozek wrote:
>> > 
>> > OK, new patches that use ':' as a separator are attached. This version
>> > uses key:value, but that can be swapped.
>> 
>> Why do you consider the 'key' to be the remote attribute name ?
>> I can't make sense of it :-)
>> 
>> The "key" sssd looks for in it's cache is the sysdb name, no ?
>> 
>> Simo.
>> 
>
>I don't really mind too much and appears Jan does not either. So I
>simply swapped the order in the attached patches, now it's sysdb:LDAP.
>
>Man page was changed accordingly.
>
>Jan, could you check if the man page change (src/man/sssd-ldap.5.xml in
>the second patch) reads good to you?

>From 92acd3f95ef5ae8ce49f398f9ef616a79887193f Mon Sep 17 00:00:00 2001
>From: Jakub Hrozek <jhrozek at redhat.com>
>Date: Mon, 21 Apr 2014 21:33:36 +0200
>Subject: [PATCH 1/3] LDAP: Fix off-by-one bug in sdap_copy_opts
>
>The sdap_copy_opts function copied all the arguments except for the
>sentinel.
>---
> Makefile.am                    |  3 +++
> src/providers/ldap/sdap.c      |  5 +++-
> src/tests/ipa_ldap_opt-tests.c | 55 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 62 insertions(+), 1 deletion(-)
>
>diff --git a/Makefile.am b/Makefile.am
>index 5dc359596416f1202c5bbf5cc2585b60eeeea6f1..56d8e1df648822a1fe8b3ca5fc8995f3c78a8566 100644
>--- a/Makefile.am
>+++ b/Makefile.am
>@@ -1215,6 +1215,9 @@ auth_tests_LDADD = \
> 
> ipa_ldap_opt_tests_SOURCES = \
>     src/providers/data_provider_opts.c \
>+    src/providers/ldap/sdap.c \
>+    src/providers/ldap/sdap_range.c \
>+    src/util/sss_ldap.c \
>     src/tests/ipa_ldap_opt-tests.c
> ipa_ldap_opt_tests_CFLAGS = \
>     $(AM_CFLAGS) \
I appologize for late response. I didn't have a time to review this patch.
This simple change causes probelm with linking this test on systems with
disabled link_all_deplibs (debian)

Attached patch fix this issue.

LS
-------------- next part --------------
>From 1031ffe3e85213517279e21077949202dcc86fe4 Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik <lslebodn at redhat.com>
Date: Fri, 2 May 2014 16:13:58 +0200
Subject: [PATCH] TEST: Link ipa_ldap_opt test with openldap libs

The test ipa_ldap_opt has undefined symbols from libldap and liblber,
but it was not directly linked with openldap libraries.

sh-4.2$ nm --undefined-only .libs/ipa_ldap_opt-tests | grep -E "ldap|ber"
                 U ber_free
                 U ldap_err2string

It causes linker failure on systems with disabled link_all_deplibs (debian)

/usr/bin/ld: src/providers/ldap/ipa_ldap_opt_tests-sdap.o: undefined reference
to symbol 'ber_free'
/usr/bin/ld: note: 'ber_free' is defined in DSO /lib64/liblber-2.4.so.2 so try
adding it to the linker command line
/lib64/liblber-2.4.so.2: could not read symbols: Invalid operation
clang: error: linker command failed with exit code 1 (use -v to see invocation)
---
 Makefile.am | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Makefile.am b/Makefile.am
index 56d8e1df648822a1fe8b3ca5fc8995f3c78a8566..4f9c11f47ce49751e585a174b15d8f30d14e1877 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -1226,6 +1226,7 @@ ipa_ldap_opt_tests_LDADD = \
     $(CHECK_LIBS) \
     $(TALLOC_LIBS) \
     $(SSSD_INTERNAL_LTLIBS) \
+    $(OPENLDAP_LIBS) \
     libsss_test_common.la
 
 ad_ldap_opt_tests_SOURCES = \
-- 
1.9.0



More information about the sssd-devel mailing list