From: Nick Porter <nick@portercomputing.co.uk>
Date: Wed, 23 Aug 2023 09:15:25 +0000 (+0100)
Subject: Use a dummy escape function to satisfy xlat arg validation
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0c7ef1c08dac8fdd0fc8eb39a7a33c40bc5a288e;p=thirdparty%2Ffreeradius-server.git

Use a dummy escape function to satisfy xlat arg validation
---

diff --git a/src/modules/rlm_sql/rlm_sql.c b/src/modules/rlm_sql/rlm_sql.c
index 43d76297da5..80a1ab7355e 100644
--- a/src/modules/rlm_sql/rlm_sql.c
+++ b/src/modules/rlm_sql/rlm_sql.c
@@ -854,6 +854,14 @@ static int sql_groupcmp(void *instance, request_t *request, fr_pair_t const *che
 	return 1;
 }
 
+/** Dummy escaping function for group membership xlat
+ *
+ */
+static int sql_group_xlat_escape(UNUSED request_t *request, UNUSED fr_value_box_t *vb, UNUSED void *uctx)
+{
+	return 0;
+}
+
 /** Check if the user is a member of a particular group
  *
 @verbatim
@@ -1131,7 +1139,7 @@ static int mod_bootstrap(module_inst_ctx_t const *mctx)
 		sql_xlat_arg[0].type = FR_TYPE_STRING;
 		sql_xlat_arg[0].required = true;
 		sql_xlat_arg[0].concat = true;
-		sql_xlat_arg[0].func = NULL; /* no escaping, we do strcmp() on it */
+		sql_xlat_arg[0].func = sql_group_xlat_escape; /* No real escaping done - we do strcmp() on it */
 		sql_xlat_arg[0].uctx = inst;
 		sql_xlat_arg[1] = (xlat_arg_parser_t)XLAT_ARG_PARSER_TERMINATOR;