On Mon, Nov 24, 2014 at 11:39:34AM +0100, Pavel Reichl wrote:
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.
* master: e0d2777620726f3f9f1f0eee911c5a9c66488443