Yahoo Groups archive

Milter-greylist

Index last updated: 2026-04-28 23:32 UTC

Message

Re: [milter-greylist] Solution for non executed final_dump()

2006-10-10 by AIDA Shinra

At Mon, 9 Oct 2006 15:47:06 +0200,
<attila.bruncsak@...> wrote:
> 
> [1  <text/plain; ISO-8859-1 (7bit)>]
> > > There is a problem with that patch: it violates the libmilter API. This says
> > > that signals are to be handled by libmilter itself:
> > > http://www.sendmail.org/doc/sendmail-current/libmilter/docs/overview.html
> > > 
> 
> OK, attached an other patch, replacing the previous one.
> This has absolutely no any reference to signal handling stuff.
> BTW, by my opinion it is time now to update both the libmilter
> code and documentation.
> 
> > Anyway, libmilter blocks the signals in the smfi_main() thread. The
> > mask is inherited by all descendant threads. Libmilter does not catch
> > the signals but picks them by sigwait().
> > 

The new patch breaks our sync mechanism. The sync listener thread does
not start before a client connects to the server.

In contrast, I can't find any reason why the previous patch "break"s
the API. To be paranoid, we just need to unblock signals once exactly
before smfi_main().

Attachments

Move to quarantaine

This moves the raw source file on disk only. The archive index is not changed automatically, so you still need to run a manual refresh afterward.