[Mimedefang] return values

Dianne Skoll dfs at roaringpenguin.com
Fri Oct 6 13:13:33 EDT 2017


> Shouldn't filter() and filter_multipart() simply use "return;" (with
> no return value)?  If not, please help me understand.

The return value of filter and filter_multipart is ignored.
I sometimes write:

  return action_accept();

as shorthand for:


> Issue 2:  bounce or discard, which is it?

Yeah, that makes no sense. :)

> In the "suggested-minimum-filter." filter function, there is this:

>     # Block message/partial parts
>     if (lc($type) eq "message/partial") {
>         md_graphdefang_log('message/partial');
>         action_bounce("MIME type message/partial not accepted here");
>         return action_discard();
>     }

Replace return action_discard(); with return;

I will fix the filter.



More information about the MIMEDefang mailing list