Print more useful error

diff --git a/src/lib/pair.c b/src/lib/pair.c
index d132f16b257b9ecdf634fb6c11c6d25bd90028ce..aa4a6fac24787d373c5521bcfdb74efeb6e793ed 100644 (file)
--- 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 000c9061e20fbd2727395f95b34fd3f90041dc03..76bf3da146f6fec340d26de89236186edbcd12d7 100644 (file)
--- 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;