From 4503506a2361357b790b489eeebc6f2d4e7dbb17 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B6rg=20Mechnich?= Date: Thu, 11 Apr 2024 14:59:47 +0200 Subject: [PATCH] Fixed comparison that caused control_executor_mailfilter_*_configured to never be set The arguments to isgreaterthan() were reversed. With 0 being the first argument, it won't ever be greater than the length of the list. Ticket: CFE-4374 Changelog: Title (cherry picked from commit e3718ef3e141ae3c0477f35035f97c82ae40fa83) --- controls/def.cf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/controls/def.cf b/controls/def.cf index 90839ab59c..5cfdf552b6 100644 --- a/controls/def.cf +++ b/controls/def.cf @@ -309,13 +309,13 @@ bundle common def " emails sent by cf-execd."); "control_executor_mailfilter_exclude_configured" -> { "ENT-10210" } - expression => isgreaterthan( 0, length( "control_executor_mailfilter_exclude" ) ), + expression => isgreaterthan( length( "control_executor_mailfilter_exclude" ), 0 ), comment => concat( "If default:def.control_executor_mailfilter_exclude is not", " an empty list, we want to use it's value for", " stripping lines from emails sent by cf-execd."); "control_executor_mailfilter_include_configured" -> { "ENT-10210" } - expression => isgreaterthan( 0, length( "control_executor_mailfilter_include" ) ), + expression => isgreaterthan( length( "control_executor_mailfilter_include" ), 0 ), comment => concat( "If default:def.control_executor_mailfilter_include is not", " an empty list, we want to use it's value for", " including lines from emails sent by cf-execd.");