CVSROOT:        /cvs
Module name:    src
Changes by:     gil...@cvs.openbsd.org  2012/01/26 16:18:08

Modified files:
        usr.sbin/smtpd : smtp_session.c 

Log message:
Do not call session_respond() AND session_imsg() from the same handler as
they are exclusive. In the quit handler, it will lead to a corrupted
session as the F_QUIT is set and session_respond() will kill the session
before session_imsg() is done.

For now, comment session_imsg(IMSG_MFA_QUIT) as we don't have filters
enabled anyways. The proper fix will be to have session_imsg() called
from the handler THEN upon return in smtp process do the session_respond()
call.

Fixes a segv reported by Roman Kravchuk. While at it, revert my last commit
in session_error() which was probably needed because of this bug.

Reply via email to