On 2013-10-07, Jeremy Harris <[email protected]> wrote:
> On 05/10/13 21:12, Jasen Betts wrote:
>> currently (unless it's been changed since 4.80)
>> there's no way to use expansion to disable the transport_filter
>> transport option.
>
> transport_filter is an expanded string.  You should be able
> to have it expand to an empty string when you don't
> want a filter.

The problem is the code doesn't check for an empty string 

I changed this:
  if (transport_filter_argv == NULL)
to this:
  if (transport_filter_argv == NULL || ! *transport_filter_argv || ! 
**transport_filter_argv )
to get that effect.


However it might make more sense to move the test for an empty string to the 
place 
where transport_filter is being expanded and transport_filter_argv is being 
set, 
as then it could avoid expanding the rest of the values.

-- 
⚂⚃ 100% natural

-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

Reply via email to