From: Alan T. DeKok <aland@freeradius.org>
Date: Wed, 23 Aug 2023 13:12:41 +0000 (-0400)
Subject: no escaping is done, so we don't need func _or_ uctx
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ecc3d1e4160139a1f45ed1d116f8cd6c4939d0b7;p=thirdparty%2Ffreeradius-server.git

no escaping is done, so we don't need func _or_ uctx
---

diff --git a/src/modules/rlm_sql/rlm_sql.c b/src/modules/rlm_sql/rlm_sql.c
index 80a1ab7355e..847d77373d6 100644
--- a/src/modules/rlm_sql/rlm_sql.c
+++ b/src/modules/rlm_sql/rlm_sql.c
@@ -854,14 +854,6 @@ 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
@@ -1139,8 +1131,8 @@ 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 = sql_group_xlat_escape; /* No real escaping done - we do strcmp() on it */
-		sql_xlat_arg[0].uctx = inst;
+		sql_xlat_arg[0].func = NULL; /* No real escaping done - we do strcmp() on it */
+		sql_xlat_arg[0].uctx = NULL;
 		sql_xlat_arg[1] = (xlat_arg_parser_t)XLAT_ARG_PARSER_TERMINATOR;
 
 		xlat_func_mono_set(xlat, sql_xlat_arg);