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.
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to