From: Victor Stinner <victor.stinner@gmail.com>
Date: Tue, 16 May 2017 23:02:35 +0000 (-0700)
Subject: bpo-30329: Catch Windows error 10022 on shutdown() (#1538) (#1621)
X-Git-Tag: v3.5.4rc1~146
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c9ba45d1b71d86321e5422e8a2cbe6e52aaba6f4;p=python

bpo-30329: Catch Windows error 10022 on shutdown() (#1538) (#1621)

Catch the Windows socket WSAEINVAL error (code 10022) in imaplib and
poplib on shutdown(SHUT_RDWR): An invalid operation was attempted

This error occurs sometimes on SSL connections.
(cherry picked from commit 83a2c2879839da2e10037f5e4af1bd1dafbf1a52)
---

diff --git a/Lib/imaplib.py b/Lib/imaplib.py
index 2f7e9331fb..ef2c6f814c 100644
--- a/Lib/imaplib.py
+++ b/Lib/imaplib.py
@@ -310,9 +310,12 @@ class IMAP4:
         self.file.close()
         try:
             self.sock.shutdown(socket.SHUT_RDWR)
-        except OSError as e:
-            # The server might already have closed the connection
-            if e.errno != errno.ENOTCONN:
+        except OSError as exc:
+            # The server might already have closed the connection.
+            # On Windows, this may result in WSAEINVAL (error 10022):
+            # An invalid operation was attempted.
+            if (exc.errno != errno.ENOTCONN
+               and getattr(exc, 'winerror', 0) != 10022):
                 raise
         finally:
             self.sock.close()
diff --git a/Lib/poplib.py b/Lib/poplib.py
index f6723904e8..516b6f060d 100644
--- a/Lib/poplib.py
+++ b/Lib/poplib.py
@@ -288,9 +288,12 @@ class POP3:
             if sock is not None:
                 try:
                     sock.shutdown(socket.SHUT_RDWR)
-                except OSError as e:
-                    # The server might already have closed the connection
-                    if e.errno != errno.ENOTCONN:
+                except OSError as exc:
+                    # The server might already have closed the connection.
+                    # On Windows, this may result in WSAEINVAL (error 10022):
+                    # An invalid operation was attempted.
+                    if (exc.errno != errno.ENOTCONN
+                       and getattr(exc, 'winerror', 0) != 10022):
                         raise
                 finally:
                     sock.close()
diff --git a/Misc/NEWS b/Misc/NEWS
index 6b4b401aa6..16cb60c9e3 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -49,6 +49,10 @@ Extension Modules
 Library
 -------
 
+- bpo-30329: imaplib and poplib now catch the Windows socket WSAEINVAL error
+  (code 10022) on shutdown(SHUT_RDWR): An invalid operation was attempted.
+  This error occurs sometimes on SSL connections.
+
 - bpo-30375: Warnings emitted when compile a regular expression now always
   point to the line in the user code.  Previously they could point into inners
   of the re module if emitted from inside of groups or conditionals.