From 21137ab63c83c8f75823daa9a6f9b5d7885ebe59 Mon Sep 17 00:00:00 2001 From: Martin Kosek Date: Tue, 29 Oct 2013 17:19:28 +0100 Subject: Remove ipa-pwd-extop and ipa-enrollment duplicate error strings Some error strings were duplicate which makes it then harder to see what is the real root cause of it. https://fedorahosted.org/freeipa/ticket/3988 --- .../ipa-slapi-plugins/ipa-enrollment/ipa_enrollment.c | 9 ++++++--- .../ipa-slapi-plugins/ipa-pwd-extop/ipa_pwd_extop.c | 19 +++++++++++-------- 2 files changed, 17 insertions(+), 11 deletions(-) (limited to 'daemons') diff --git a/daemons/ipa-slapi-plugins/ipa-enrollment/ipa_enrollment.c b/daemons/ipa-slapi-plugins/ipa-enrollment/ipa_enrollment.c index 22c40f2bc..a3dcf08a6 100644 --- a/daemons/ipa-slapi-plugins/ipa-enrollment/ipa_enrollment.c +++ b/daemons/ipa-slapi-plugins/ipa-enrollment/ipa_enrollment.c @@ -184,7 +184,7 @@ ipa_join(Slapi_PBlock *pb) slapi_pblock_get(pbte, SLAPI_PLUGIN_INTOP_RESULT, &res); if (ret == -1 || res != LDAP_SUCCESS) { LOG_TRACE("Search for host failed, err (%d)\n", res?res:ret); - errMesg = "Host not found.\n"; + errMesg = "Host not found (search failed).\n"; rc = LDAP_NO_SUCH_OBJECT; goto free_and_return; } @@ -193,7 +193,7 @@ ipa_join(Slapi_PBlock *pb) slapi_pblock_get(pbte, SLAPI_PLUGIN_INTOP_SEARCH_ENTRIES, &es); if (!es) { LOG_TRACE("No entries ?!"); - errMesg = "Host not found.\n"; + errMesg = "Host not found (no result returned).\n"; rc = LDAP_NO_SUCH_OBJECT; goto free_and_return; } @@ -204,7 +204,10 @@ ipa_join(Slapi_PBlock *pb) /* if there is none or more than one, freak out */ if (i != 1) { LOG_TRACE("Too many entries, or entry no found (%d)", i); - errMesg = "Host not found.\n"; + if (i == 0) + errMesg = "Host not found.\n"; + else + errMesg = "Host not found (too many entries).\n"; rc = LDAP_NO_SUCH_OBJECT; goto free_and_return; } diff --git a/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipa_pwd_extop.c b/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipa_pwd_extop.c index c3e0ebd9d..688d6995e 100644 --- a/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipa_pwd_extop.c +++ b/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipa_pwd_extop.c @@ -719,7 +719,7 @@ static int ipapwd_setkeytab(Slapi_PBlock *pb, struct ipapwd_krbcfg *krbcfg) bsdn = slapi_be_getsuffix(be, 0); if (bsdn == NULL) { LOG_TRACE("Search for Base DN failed\n"); - errMesg = "PrincipalName not found.\n"; + errMesg = "PrincipalName not found (search for Base DN failed).\n"; rc = LDAP_NO_SUCH_OBJECT; goto free_and_return; } @@ -743,7 +743,7 @@ static int ipapwd_setkeytab(Slapi_PBlock *pb, struct ipapwd_krbcfg *krbcfg) if (ret == -1 || res != LDAP_SUCCESS) { LOG_TRACE("Search for Principal failed, err (%d)\n", res ? res : ret); - errMesg = "PrincipalName not found.\n"; + errMesg = "PrincipalName not found (search failed).\n"; rc = LDAP_NO_SUCH_OBJECT; goto free_and_return; } @@ -752,7 +752,7 @@ static int ipapwd_setkeytab(Slapi_PBlock *pb, struct ipapwd_krbcfg *krbcfg) slapi_pblock_get(pbte, SLAPI_PLUGIN_INTOP_SEARCH_ENTRIES, &es); if (!es) { LOG_TRACE("No entries ?!"); - errMesg = "PrincipalName not found.\n"; + errMesg = "PrincipalName not found (no result returned).\n"; rc = LDAP_NO_SUCH_OBJECT; goto free_and_return; } @@ -763,7 +763,10 @@ static int ipapwd_setkeytab(Slapi_PBlock *pb, struct ipapwd_krbcfg *krbcfg) /* if there is none or more than one, freak out */ if (i != 1) { LOG_TRACE("Too many entries, or entry no found (%d)", i); - errMesg = "PrincipalName not found.\n"; + if (i == 0) + errMesg = "PrincipalName not found.\n"; + else + errMesg = "PrincipalName not found (too many entries).\n"; rc = LDAP_NO_SUCH_OBJECT; goto free_and_return; } @@ -857,7 +860,7 @@ static int ipapwd_setkeytab(Slapi_PBlock *pb, struct ipapwd_krbcfg *krbcfg) rtag = ber_scanf(ber, "{t[{t[i]t[o]}]", &ttmp, &ttmp, &tint, &ttmp, &tval); if (rtag == LBER_ERROR) { LOG_FATAL("ber_scanf failed\n"); - errMesg = "Invalid payload, failed to decode.\n"; + errMesg = "Invalid payload, failed to decode EncryptionKey.\n"; rc = LDAP_PROTOCOL_ERROR; goto free_and_return; } @@ -906,7 +909,7 @@ static int ipapwd_setkeytab(Slapi_PBlock *pb, struct ipapwd_krbcfg *krbcfg) rtag = ber_scanf(ber, "t[{t[i]", &ttmp, &ttmp, &tint); if (rtag == LBER_ERROR) { LOG_FATAL("ber_scanf failed\n"); - errMesg = "Invalid payload, failed to decode.\n"; + errMesg = "Invalid payload, failed to decode KrbSalt type.\n"; rc = LDAP_PROTOCOL_ERROR; goto free_and_return; } @@ -920,7 +923,7 @@ static int ipapwd_setkeytab(Slapi_PBlock *pb, struct ipapwd_krbcfg *krbcfg) rtag = ber_scanf(ber, "t[o]}]", &ttmp, &tval); if (rtag == LBER_ERROR) { LOG_FATAL("ber_scanf failed\n"); - errMesg = "Invalid payload, failed to decode.\n"; + errMesg = "Invalid payload, failed to decode KrbSalt contents.\n"; rc = LDAP_PROTOCOL_ERROR; goto free_and_return; } @@ -948,7 +951,7 @@ static int ipapwd_setkeytab(Slapi_PBlock *pb, struct ipapwd_krbcfg *krbcfg) } if (rtag == LBER_ERROR) { LOG_FATAL("ber_scanf failed\n"); - errMesg = "Invalid payload, failed to decode.\n"; + errMesg = "Invalid payload, failed to decode s2kparams.\n"; rc = LDAP_PROTOCOL_ERROR; goto free_and_return; } -- cgit