Re: [exim-dev] making transport_filter conditional

Góra strony
Delete this message
Reply to this message
Autor: Jasen Betts
Data:  
Dla: exim-dev
Temat: Re: [exim-dev] making transport_filter conditional
On 2013-10-07, Jeremy Harris <jgh@???> 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