I think the series looks good now and I believe your ok with the changes I've made. The series will sit in my patch queue until I commit them and push them to the repo. I will most likely test the function of them too but that will take a while to do. Changes since v1: - in do_bind() don't free the passed in result in ldap_parse_result(), explicitly handle the free in the function that declares the result variable, do_bind(). --- Thomas Reim (4): autofs-5.1.8 - restore gcc flags after autoconf Kerberos 5 check autofs-5.1.8 - prepare for OpenLDAP SASL binding autofs-5.1.8 - let OpenLDAP handle SASL binding autofs-5.1.8 - configure: LDAP function checks ignore implicit declarations CHANGELOG | 4 ++ aclocal.m4 | 52 +++++++++++++++ configure | 55 +++++++++++++++- configure.in | 5 +- include/config.h.in | 3 + include/lookup_ldap.h | 6 ++ modules/cyrus-sasl.c | 150 +++++++++++++++++++++++++++++++++++++++++- modules/lookup_ldap.c | 137 +++++++++++++++++++++++++++++++++++++- 8 files changed, 408 insertions(+), 4 deletions(-) -- Ian