From: Alan T. DeKok <aland@freeradius.org>
Date: Thu, 27 Mar 2025 18:52:49 +0000 (-0400)
Subject: rely on fr_value_box_is_safe_for() rather then checking safe_for value
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=790154d062ee3270a55c692c85638be9ce32afac;p=thirdparty%2Ffreeradius-server.git

rely on fr_value_box_is_safe_for() rather then checking safe_for value
---

diff --git a/src/lib/unlang/xlat_eval.c b/src/lib/unlang/xlat_eval.c
index 1427de2dae4..c6c0a63e5d3 100644
--- a/src/lib/unlang/xlat_eval.c
+++ b/src/lib/unlang/xlat_eval.c
@@ -291,7 +291,7 @@ static xlat_action_t xlat_process_arg_list(TALLOC_CTX *ctx, fr_value_box_list_t
 
 #define ESCAPE(_arg, _vb, _arg_num) \
 do { \
-	if ((_arg)->func && (!(_vb)->safe_for || !fr_value_box_is_safe_for((_vb), (_arg)->safe_for) || (_arg)->always_escape)) { \
+	if ((_arg)->func && (!fr_value_box_is_safe_for((_vb), (_arg)->safe_for) || (_arg)->always_escape)) { \
 		if ((_arg)->func(request, _vb, (_arg)->uctx) < 0) { \
 			RPEDEBUG("Function \"%s\" failed escaping argument %u", name, _arg_num); \
 			return XLAT_ACTION_FAIL; \