<div dir="ltr"><div>Hi List,</div><div><br></div><div>The symbol add_key is used by</div><div>src/providers/krb5/krb5_delayed_online_authentication.c</div><div>which is part of libsss_krb5_common.so</div><div><br></div><div>
Fixes following error:</div><div>[sssd[be[default]]] [load_backend_module]</div><div>  (0x0010): Unable to load ad module with path</div><div>  (/usr/lib64/sssd/libsss_ad.so), error:</div><div>  /usr/lib64/sssd/libsss_krb5_common.so: undefined symbol: add_key</div>
<div><br></div><div>-lkeyutils was passed to the libraries libsss_{krb5,ipa,ad}.so,</div><div>but when compiling with -Wl,--as-needed this flag will be ignored,</div><div>since it is not used directly. So it was unavailable to</div>
<div>libsss_krb5_common.so which actually needs it.</div><div><br></div><div>This patch removes $(KEYUTILS_LIBS) from those libraries and adds it to</div><div>libsss_krb5_common.so</div><div><br></div><div>Maybe libsss_krb5_common.so should be added to dlopen-tests?</div>
<div>But then other libraries and functions are needed as well,</div><div>which it currently inherits from libsss_{krb5,ipa,ad}.so.</div><div><br></div><div>BTW: are these common libraries (i mean ldap too) convenience build libraries, or to save disk space?</div>
<div>If they&#39;re just for convencience maybe they should not be installed?</div><div><br></div><div>Regards, Ben</div></div>