A NOTE has been added to this issue. ====================================================================== http://www.dbmail.org/mantis/view.php?id=260 ====================================================================== Reported By: mavetju Assigned To: paul ====================================================================== Project: DBMail Issue ID: 260 Category: General Reproducibility: always Severity: crash Priority: low Status: assigned ====================================================================== Date Submitted: 23-Aug-05 02:27 CEST Last Modified: 21-Sep-05 02:32 CEST ====================================================================== Summary: pop3/imap daemon dies in stress test Description: The suggested patch in http://www.dbmail.org/mantis/view.php?id=258 is that we use a static string instead of a dynamically allocated one in trace(). Unfortunately, trace() itself still uses syslog() which calls vsyslog() which calls fprintf() which calls [....] which calls malloc().
Solution? Don't call trace() from a signal handler. ====================================================================== ---------------------------------------------------------------------- paul - 24-Aug-05 13:38 ---------------------------------------------------------------------- I've changed the signal handler code in server.c and serverchild.c removing all unsafe code (hopefully). Would you mind testing again? ---------------------------------------------------------------------- mavetju - 24-Aug-05 13:53 ---------------------------------------------------------------------- Of course, but can you tell me which revisions of code I should compare to get a patch? (I'm sorry my SVN skills aren't as good as my CVS ones) ---------------------------------------------------------------------- kouta - 30-Aug-05 00:34 ---------------------------------------------------------------------- IMHO: svn diff -r 1861:1871 server.c svn diff -r 1757:1873 serverchild.c ---------------------------------------------------------------------- mavetju - 21-Sep-05 02:32 ---------------------------------------------------------------------- This works fine in 2.0.7. Issue History Date Modified Username Field Change ====================================================================== 23-Aug-05 02:27 mavetju New Issue 24-Aug-05 13:38 paul Note Added: 0000877 24-Aug-05 13:53 mavetju Note Added: 0000878 30-Aug-05 00:33 kouta Note Added: 0000892 30-Aug-05 00:34 kouta Note Edited: 0000892 31-Aug-05 14:12 paul Assigned To => paul 31-Aug-05 14:12 paul Priority normal => low 31-Aug-05 14:12 paul Status new => assigned 31-Aug-05 14:12 paul Resolution open => fixed 21-Sep-05 02:32 mavetju Note Added: 0000923 ======================================================================
