From: Victor Stinner <victor.stinner@gmail.com>
Date: Tue, 4 Jul 2017 14:20:06 +0000 (+0200)
Subject: bpo-30319: socket.close() now ignores ECONNRESET (#2565)
X-Git-Tag: v3.7.0a1~465
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=67e1478dba6efe60b8e1890192014b8b06dd6bd9;p=python

bpo-30319: socket.close() now ignores ECONNRESET (#2565)

socket.close() was modified in Python 3.6 to raise OSError on
failure: see bpo-26685.
---

diff --git a/Misc/NEWS.d/next/Library/2017-07-04-13-48-21.bpo-30319.hg_3TX.rst b/Misc/NEWS.d/next/Library/2017-07-04-13-48-21.bpo-30319.hg_3TX.rst
new file mode 100644
index 0000000000..1112d2f373
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2017-07-04-13-48-21.bpo-30319.hg_3TX.rst
@@ -0,0 +1 @@
+socket.close() now ignores ECONNRESET error.
diff --git a/Modules/socketmodule.c b/Modules/socketmodule.c
index a1d829f9cb..e18dd32d90 100644
--- a/Modules/socketmodule.c
+++ b/Modules/socketmodule.c
@@ -2696,7 +2696,9 @@ sock_close(PySocketSockObject *s)
         Py_BEGIN_ALLOW_THREADS
         res = SOCKETCLOSE(fd);
         Py_END_ALLOW_THREADS
-        if (res < 0) {
+        /* bpo-30319: The peer can already have closed the connection.
+           Python ignores ECONNRESET on close(). */
+        if (res < 0 && errno != ECONNRESET) {
             return s->errorhandler();
         }
     }