https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7324

Mark Martinec <mark.marti...@ijs.si> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|Undefined                   |3.4.2

--- Comment #1 from Mark Martinec <mark.marti...@ijs.si> ---
> stackoverflow ... says this is asking for trouble.

Yes, ignoring status return is asking for trouble.
When return code is intentionally ignored, at least it should be
clearly documented why this is so.

In this particular case (in libspamc_log) it is not so terrible,
as all we can lose is a log output (and if stderr is not writable
there is probably no place to send an error message to).

It is more troublesome that calls to SSL_write in that same module
are not checked for status. I think there is already a bug open somewhere,
claiming that mail can be lost/truncated when spamc uses SSL.

Should be fixed - patches welcome...

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to