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.