Good morning,

I  find something incomprehensible in clamav-milter.c code. When clamav-
milter daemon do the  scanning  work  itself  after  getting  the  first
SIGUSR2 it installs a new signal handler, SIGHUP:

        sigusr2(int sig)
        {
                signal(SIGUSR2, sighup);
        
                logg("^SIGUSR2 caught: scheduling database reload\n");
                reload++;
        }

IMHO I don't see any reason to do that, shouldn't it be changed to:

                signal(SIGUSR2, sigusr2);


Best regards,
Ernest Wypierowski
_______________________________________________
http://lurker.clamav.net/list/clamav-devel.html
Please submit your patches to our Bugzilla: http://bugs.clamav.net

Reply via email to