[SSSD] [PATCH] Hint about removing sysdb if initializing ID map fails

Pavel Reichl preichl at redhat.com
Mon Nov 24 10:39:34 UTC 2014


On 11/22/2014 03:56 PM, Jakub Hrozek wrote:
> diff --git a/src/providers/ipa/ipa_init.c b/src/providers/ipa/ipa_init.c
> index 45abd0e2ae3d5909e33fd0fca2027a238264b455..160d15cac42a1c1083722d656d369442e6ee4dd8 100644
> --- a/src/providers/ipa/ipa_init.c
> +++ b/src/providers/ipa/ipa_init.c
> @@ -225,7 +225,13 @@ int sssm_ipa_id_init(struct be_ctx *bectx,
>   
>       /* Set up the ID mapping object */
>       ret = ipa_idmap_init(sdap_ctx, sdap_ctx, &sdap_ctx->opts->idmap_ctx);
> -    if (ret != EOK) goto done;
> +    if (ret != EOK) {
> +        DEBUG(SSSDBG_FATAL_FAILURE,
> +              "Could not initialize ID mapping. In case ID mapping properties "
> +              "changed on the server, please remove the SSSD database\n");
> +        goto done;
> +    }
> +
>   
>       ret = ldap_id_setup_tasks(sdap_ctx);
>       if (ret != EOK) {
> -- 1.9.3
Thanks, builds OK, so ack by me.

But I would not protest if you removed the extra empty line in the patch 
before pushing it.



More information about the sssd-devel mailing list