On Tue, Feb 22, 2011 at 08:02:56AM -0500, Stephen Gallagher wrote:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 02/22/2011 06:13 AM, Sumit Bose wrote:
On Mon, Feb 21, 2011 at 02:04:52PM -0500, Stephen Gallagher wrote: Un-withdrawing this patch (re-attaching for posterity).
I thought we were not honoring krb5_realm elsewhere, because we were just doing dp_get_option(KRB5_REALM), but I discovered that ipa_get_id_options() and ipa_get_auth_options() forcibly sets these values to be correct.
So the only piece that was incorrect was ipa_service_init() (which only uses the realm for writing the krb5info file.
ACK
Pushed to master and sssd-1-5.
Sorry, I just realized that my testing was flawed. You access options->id->basic in ipa_service_init() before it is initialized by ipa_get_id_options().
bye, Sumit
Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/ -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
iEYEARECAAYFAk1jtAAACgkQeiVVYja6o6NcJgCfZWWgaMlwRh2InpxWSJS0ivfj OOQAnAqj2Llswe7ZlCYlAbF3E6ppcHAE =Wdh5 -----END PGP SIGNATURE----- _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel