# HG changeset patch # User Chong Yidong # Date 1290183909 18000 # Node ID c392bf1fd90ce4cae00782e59a0d0e4620ae0385 # Parent 32de8ce4db5ba5fb9f52ec9fd509e302a16e249c * pop3.el (pop3-open-server): Read server greeting before starting TLS negotiation. diff -r 32de8ce4db5b -r c392bf1fd90c lisp/gnus/ChangeLog --- a/lisp/gnus/ChangeLog Wed Nov 17 22:44:35 2010 -0500 +++ b/lisp/gnus/ChangeLog Fri Nov 19 11:25:09 2010 -0500 @@ -1,3 +1,8 @@ +2010-11-19 Yuri Karaban (tiny change) + + * pop3.el (pop3-open-server): Read server greeting before starting TLS + negotiation. + 2010-10-12 Juanma Barranquero * nnmail.el (nnmail-fancy-expiry-targets): Fix typo in docstring. diff -r 32de8ce4db5b -r c392bf1fd90c lisp/gnus/pop3.el --- a/lisp/gnus/pop3.el Wed Nov 17 22:44:35 2010 -0500 +++ b/lisp/gnus/pop3.el Fri Nov 19 11:25:09 2010 -0500 @@ -259,21 +259,22 @@ ;; gnutls-cli, openssl don't accept service names (if (equal port "pop3") (setq port 110)) - (let ((process (starttls-open-stream "POP" (current-buffer) - mailhost (or port 110)))) - (pop3-send-command process "STLS") - (let ((response (pop3-read-response process t))) - (if (and response (string-match "+OK" response)) - (starttls-negotiate process) - (pop3-quit process) - (error "POP server doesn't support starttls"))) - process)) - (t + ;; Delay STLS until server greeting is read (Bug#7438). + (starttls-open-stream "POP" (current-buffer) + mailhost (or port 110))) + (t (open-network-stream "POP" (current-buffer) mailhost port)))) (let ((response (pop3-read-response process t))) (setq pop3-timestamp (substring response (or (string-match "<" response) 0) (+ 1 (or (string-match ">" response) -1))))) + (when (eq pop3-stream-type 'starttls) + (pop3-send-command process "STLS") + (let ((response (pop3-read-response process t))) + (if (and response (string-match "+OK" response)) + (starttls-negotiate process) + (pop3-quit process) + (error "POP server doesn't support starttls")))) process))) ;; Support functions