severity 595756 serious
thanks

Hi Manuel,

On Mon, Sep 06, 2010 at 02:46:37PM +0200, Manuel CISSE wrote:
> With a slow mail server, notify_email_notification() may be called 
> simultaneously by
> 2 or more threads, leading to a crash in libesmtp (due to multiple calls to
> smtp_start_session() on the same session).
> 
> Also, if collectd is stopped when a notification is being dispatched,
> notify_email_shutdown() destroys the session used by 
> notify_email_notification()
> causing a segfault.

Thanks for reporting this! Afaik, Florian already has a (rather
untested) patch available in Git. I'll look into that and prepare an
updated package (targeting Squeeze [through unstable]).

Since segfaults in any plug-in will bring down the whole daemon process,
this a) makes the plugin rather unusable and b) could cause data loss
when having caching enabled in the "rrdtool" plugin. Thus, I've raised
the severity to "serious". (It's perfectly fine, though, that you've
used "normal" -- it's the maintainers job to change the severity if
necessary ;-))

Cheers,
Sebastian

-- 
Sebastian "tokkee" Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/

Those who would give up Essential Liberty to purchase a little Temporary
Safety, deserve neither Liberty nor Safety.         -- Benjamin Franklin

Attachment: signature.asc
Description: Digital signature

Reply via email to