From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Sat, 5 Mar 2016 03:37:34 +0000 (-0500)
Subject: Print more useful error
X-Git-Tag: release_3_0_12~197
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f8aa4e88800e8b9c5b80381a31493ea920eca81a;p=thirdparty%2Ffreeradius-server.git

Print more useful error
---

diff --git a/src/lib/pair.c b/src/lib/pair.c
index d132f16b257..aa4a6fac247 100644
--- a/src/lib/pair.c
+++ b/src/lib/pair.c
@@ -1614,11 +1614,13 @@ int fr_pair_mark_xlat(VALUE_PAIR *vp, char const *value)
 	 *	valuepair should not already have a value.
 	 */
 	if (vp->type != VT_NONE) {
+		fr_strerror_printf("Pair already has a value");
 		return -1;
 	}
 
 	raw = talloc_typed_strdup(vp, value);
 	if (!raw) {
+		fr_strerror_printf("Out of memory");
 		return -1;
 	}
 
diff --git a/src/modules/rlm_sql/sql.c b/src/modules/rlm_sql/sql.c
index 000c9061e20..76bf3da146f 100644
--- a/src/modules/rlm_sql/sql.c
+++ b/src/modules/rlm_sql/sql.c
@@ -189,7 +189,7 @@ int sql_fr_pair_list_afrom_str(TALLOC_CTX *ctx, REQUEST *request, VALUE_PAIR **h
 
 	if (do_xlat) {
 		if (fr_pair_mark_xlat(vp, value) < 0) {
-			REDEBUG("Error marking pair for xlat");
+			REDEBUG("Error marking pair for xlat: %s", fr_strerror());
 
 			talloc_free(vp);
 			return -1;