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 1427de2dae43213a82a7c1837a5007bf8f121b56..c6c0a63e5d368f03845e5f47192afb05c37e53e3 100644 (file)
--- 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; \