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.