You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2005/01/26 14:22:58 UTC

[Bug 4107] New: let spamd not crash on accepting a non-ssl connection if spamd is configured for ssl

http://bugzilla.spamassassin.org/show_bug.cgi?id=4107

           Summary: let spamd not crash on accepting a non-ssl connection if
                    spamd is configured for ssl
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P5
         Component: spamc/spamd
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: nico.prenzel@pn-systeme.de


I think it's an unwanted behaviour if spamd is configured to accept only ssl
connections, but crashes itself if a non-ssl connection arrives.

crash log:


debug: prefork: ordered 20559 to accept
debug: log: SSL failure: SSL accept attempt failederror:140760FC:SSL
routines:SSL23_GET_CLIENT_HELLO:unknown
protocolerror:00000000:lib(0):func(0):reason(0)
debug: prefork: child 20559: entering state 1
prefork: ordered child to accept, but child reported state '1' at
/usr/local/share/perl/5.8.4/Mail/SpamAssassin/SpamdForkScaling.pm line 263,
<GEN2> line 2.
empty order from parent at
/usr/local/share/perl/5.8.4/Mail/SpamAssassin/SpamdForkScaling.pm line 309.
debian:/home#
debian:/home#
debian:/home#
debian:/home#
debian:/home#
debian:/home#
debian:/home# empty order from parent at
/usr/local/share/perl/5.8.4/Mail/SpamAssassin/SpamdForkScaling.pm line 309,
<GEN3> line 1.



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

[Bug 4107] let spamd not crash on accepting a non-ssl connection if spamd is configured for ssl

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4107


jm@jmason.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




------- Additional Comments From jm@jmason.org  2005-06-13 00:04 -------
ok, fixed.  
Sending        MANIFEST
Sending        spamd/spamd.raw
Adding         t/spamd_ssl_accept_fail.t
Transmitting file data ...
Committed revision 190364.


ps: I'd say we're ready for a 3.1.0 prerelease...



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

[Bug 4107] let spamd not crash on accepting a non-ssl connection if spamd is configured for ssl

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4107





------- Additional Comments From nico.prenzel@pn-systeme.de  2005-06-02 15:59 -------
Hi Justin,

you're right now. As I reported the error the behaviour was a complete crash. 
SA version was current trunk at report time. (i don't remember)

btw. it shouldn't also crash a child.




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

[Bug 4107] let spamd not crash on accepting a non-ssl connection if spamd is configured for ssl

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4107


jm@jmason.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P5                          |P3




------- Additional Comments From jm@jmason.org  2005-06-07 20:42 -------
Nico, if the child crashes but then is restarted by the master spamd, that's OK.
anyway, raising pri -- this is about the biggest bug left in the 3.1.0 queue....



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

[Bug 4107] let spamd not crash on accepting a non-ssl connection if spamd is configured for ssl

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4107


Bob@Menschel.net changed:

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




------- Additional Comments From Bob@Menschel.net  2005-05-26 21:46 -------
Triage: Guessing this should be easy enough to reproduce and then fix for 3.1



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

[Bug 4107] let spamd not crash on accepting a non-ssl connection if spamd is configured for ssl

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4107





------- Additional Comments From nico.prenzel@pn-systeme.de  2005-01-26 09:54 -------
Also the other way round:

If you connect to spamd which only accepts non-ssl communication from spamc, 
configured to use ssl, than spamd gives a
debug: log: connection from x.x.x.x [x.x.x.x] at port xxxx
and hangs after the output.





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

[Bug 4107] let spamd not crash on accepting a non-ssl connection if spamd is configured for ssl

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4107





------- Additional Comments From jm@jmason.org  2005-06-01 20:34 -------
btw I presume this is not a crash bug -- ie. the entire spamd server isn't
killed, just that child process?



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