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.