[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