[SSSD] [PATCH] Check our libraries properly resolve symbols

Simo Sorce simo at redhat.com
Fri Sep 6 01:48:27 UTC 2013


While reviewing the recent big patchset I sent Sumit found out I did not
add some .c files to the ldap provider (which I didn't test as I was
working on the krb5 provider), and that broke the provider's shared
library as one symbols was missing.

We allow modules to have unresolved symbols because a number of the
symbols they use come from our sssd_be binary so they can't be fully
resolved at build time, only at runtime.

The attached patch adds a test that simulates loading most of the shared
modules we build in order to verify that there are no unresolved
symbols, that is done by manually dlopen()ing eash .so library with
RTLD_NOW so all symbols are immediately resolved.

I tested that this test properly caught the issue Sumit found before
fixing it.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-tests-Add-dlopen-test-to-make-sure-modules-works.patch
Type: text/x-patch
Size: 4594 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130905/b12637c4/attachment.bin>


More information about the sssd-devel mailing list