Milter-greylist

Hello everybody

milter-greylist 1.3.9 is available. It just fixes serious bugs, every other
patches were psotponed for later.

1.3.9:
        Solaris rc script from Christian Pelissier
        Correctly check pthread function return values
        Avoid using a thread for the initial config file load (useless)
        Bug fix: Domain regex were not freed on config file reloads
        Bug fix: Timeout parmaeter was not honoured everywhere

Can you please all check if the thing builds and works correctly on 
your system?

-- 
Emmanuel Dreyfus
manu@...

On Tue, Jun 08, 2004 at 04:15:48PM +0000, Emmanuel Dreyfus wrote:
> milter-greylist 1.3.9 is available. It just fixes serious bugs, every other
> patches were psotponed for later.
> 
> 1.3.9:
>         Solaris rc script from Christian Pelissier
>         Correctly check pthread function return values
>         Avoid using a thread for the initial config file load (useless)
>         Bug fix: Domain regex were not freed on config file reloads
>         Bug fix: Timeout parmaeter was not honoured everywhere
> 
> Can you please all check if the thing builds and works correctly on 
> your system?

The new release works fine on NetBSD 2.0_BETA with the bundled "libpthread".
But I got a scary message after restarting "milter-greylist":

Jun  8 23:32:02 colwyn sm-mta[4372]: i58LPNw7004372: Milter (greylist): write(Q) returned -1, expected 5: Broken pipe
Jun  8 23:32:02 colwyn sm-mta[4372]: i58LPNw7004372: Milter (greylist): to error state

Is that because "sendmail" was talking to it while I restarted it?

	Kind regards

-- 
Matthias Scheler                                  http://scheler.de/~matthias/

Matthias Scheler <tron@...> wrote:

> The new release works fine on NetBSD 2.0_BETA with the bundled
> "libpthread". But I got a scary message after restarting
> "milter-greylist":
(snip) 
> Is that because "sendmail" was talking to it while I restarted it?

Yes, it's not really a bug: you killed the milter while it was being
used by sendmail. 

-- 
Emmanuel Dreyfus
Il y a 10 sortes de personnes dans le monde: ceux qui comprennent 
le binaire et ceux qui ne le comprennent pas.
manu@...

Emmanuel Dreyfus wrote:
> Hello everybody
> 
> milter-greylist 1.3.9 is available. 

> 
> Can you please all check if the thing builds and works correctly on 
> your system?

Looks OK here. I built it with both Solaris Forte 6 compiler and gcc.

There are 2 remaining small problems :

- configure tries to use -ansi -Wall for some tests, but since cc 
doesn't generate a non-zero exit code for unknown options, it goes 
un-noticed.

- there's a warning about the implicit seteuid() function declaration in 
milter-greylist.c with Forte, because -D__EXTENSIONS__ is not pulled by 
configure, since the snprintf() test program doesn't need it. I've 
attached config.log below if you want to fix it.
-- 
Matthieu Herrb

Matthieu Herrb <matthieu.herrb@...> wrote:
 
> - configure tries to use -ansi -Wall for some tests, but since cc 
> doesn't generate a non-zero exit code for unknown options, it goes 
> un-noticed.

I don't see how we could fix that.
 
> - there's a warning about the implicit seteuid() function declaration in
> milter-greylist.c with Forte, because -D__EXTENSIONS__ is not pulled by
> configure, since the snprintf() test program doesn't need it. I've 
> attached config.log below if you want to fix it.

Well, I'd really prefer a patch to configure.ac :)

-- 
Emmanuel Dreyfus
Il y a 10 sortes de personnes dans le monde: ceux qui comprennent 
le binaire et ceux qui ne le comprennent pas.
manu@...