From: Nick Porter <nick@portercomputing.co.uk>
Date: Thu, 8 May 2025 07:20:25 +0000 (+0100)
Subject: Remove pair that didn't get populated
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=bc432b18d07cb7f2db1f937fcadfc730ee51b72d;p=thirdparty%2Ffreeradius-server.git

Remove pair that didn't get populated
---

diff --git a/src/modules/rlm_sql/rlm_sql.c b/src/modules/rlm_sql/rlm_sql.c
index a0cfdd6ca28..12e82042a25 100644
--- a/src/modules/rlm_sql/rlm_sql.c
+++ b/src/modules/rlm_sql/rlm_sql.c
@@ -687,6 +687,7 @@ static int _sql_map_proc_get_value(TALLOC_CTX *ctx, fr_pair_list_t *out,
 	if (fr_pair_value_from_str(vp, value, strlen(value), NULL, true) < 0) {
 		RPEDEBUG("Failed parsing value \"%pV\" for attribute %s",
 			 fr_box_strvalue_buffer(value), vp->da->name);
+		fr_pair_delete(out, vp);
 		return -1;
 	}