[SSSD] [PATCH] AD: Write out domain-realm mappings
Jakub Hrozek
jhrozek at redhat.com
Thu Jun 27 16:44:35 UTC 2013
On Thu, Jun 27, 2013 at 03:03:51PM +0200, Sumit Bose wrote:
> On Thu, Jun 27, 2013 at 01:44:41PM +0200, Jakub Hrozek wrote:
> > On Thu, Jun 27, 2013 at 09:53:42AM +0200, Sumit Bose wrote:
> > > > +static errno_t
> > > > +ipa_write_subdom_mappings(struct sss_domain_info *domain)
> > > > +{
> > > > + errno_t ret;
> > > > +
> > > > + ret = sss_write_domain_mappings(domain);
> > > > + if (ret) {
> > > > + DEBUG(SSSDBG_OP_FAILURE, ("couldn't write domain-realm mappings\n"));
> > > > + }
> > > > +
> > > > + /* Some mappings might have been written, touch krb5.conf */
> > > > +
> > > > + /* touch krb5.conf to ensure that new mappings are loaded */
> > > > + ret = sss_krb5_touch_config();
> > > > + if (ret != EOK) {
> > > > + DEBUG(SSSDBG_CRIT_FAILURE, ("Unable to change last modification time "
> > > > + "of krb5.conf. Created mappings may not be loaded.\n"));
> > > > + return ret;
> > > > + }
> > > > +
> > > > + return EOK;
> > > > +}
> > > > +
> > >
> > > Would it be possible to move this to src/util/domain_info_utils.c as
> > > well? The corresponding AD call looks quite similar.
> > >
> > > bye,
> > > Sumit
> >
> > Sure, I moved everything (including touching the config) to
> > domain_info_utils.c and make it a single patch.
>
> ACK
>
> bye,
> Sumit
Pushed to master.
More information about the sssd-devel
mailing list