-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 10/20/2009 08:24 AM, Sumit Bose wrote: > Hi, > > this set of patches adds the new option handling to krb5 and ipa auth: > > 0001: new option handling for krb5 > 0002: new option handling for ipa auth > 0003: fix a compiler warning that was introduced by extending > krb5_common.h > > 0001 and 0002 depend on Simo's "Start implementing ipa specific > options." patch. > > bye, > Sumit > > > > _______________________________________________ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org > https://fedorahosted.org/mailman/listinfo/sssd-devel
Nack. You commented out some code that should have been removed (elements of krb5_ctx and their references in krb5_utils.c. Please remove them completely. This also causes the unit tests to fail: 'make check' fails: ../../server/tests/krb5_utils-tests.c: In function ‘setup_talloc_context’: ../../server/tests/krb5_utils-tests.c:67: error: ‘struct krb5_ctx’ has no member named ‘realm’ ../../server/tests/krb5_utils-tests.c:68: error: ‘struct krb5_ctx’ has no member named ‘ccache_dir’ ../../server/tests/krb5_utils-tests.c:46: warning: unused variable ‘ret’ Also, you use 'krb5_changepw_princ', but the manpage lists 'krb5_changepw_principle'. Please fix one or the other :) - -- Stephen Gallagher RHCE 804006346421761 Looking to carve out IT costs? www.redhat.com/carveoutcosts/ -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/ iEYEARECAAYFAkrfA8QACgkQeiVVYja6o6P0kACgpixvyW40kerJX11liowRQOsd AGAAnjU8WqLNLs7wHFi9uYFwZyCjWHOx =ubAE -----END PGP SIGNATURE----- _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel