[Python-checkins] bpo-32710: Fix leak in Overlapped_WSASend() (GH-11469)

Tue Jan 8 08:40:53 EST 2019

https://github.com/python/cpython/commit/88ad48bc98980a40591cc5521703dbb0ad3a9b17
commit: 88ad48bc98980a40591cc5521703dbb0ad3a9b17
branch: 3.7
author: Miss Islington (bot) <31488909+miss-islington at users.noreply.github.com>
committer: GitHub <noreply at github.com>
date: 2019年01月08日T05:40:50-08:00
summary:
bpo-32710: Fix leak in Overlapped_WSASend() (GH-11469)
Fix a memory leak in asyncio in the ProactorEventLoop when ReadFile()
or WSASend() overlapped operation fail immediately: release the
internal buffer.
(cherry picked from commit a234e148394c2c7419372ab65b773d53a57f3625)
Co-authored-by: Victor Stinner <vstinner at redhat.com>
files:
A Misc/NEWS.d/next/Library/2019-01-08-14-00-52.bpo-32710.Sn5Ujj.rst
M Modules/overlapped.c

diff --git a/Misc/NEWS.d/next/Library/2019-01-08-14-00-52.bpo-32710.Sn5Ujj.rst b/Misc/NEWS.d/next/Library/2019-01-08-14-00-52.bpo-32710.Sn5Ujj.rst
new file mode 100644
index 000000000000..5c3961c33d96
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2019-01-08-14-00-52.bpo-32710.Sn5Ujj.rst
@@ -0,0 +1,3 @@
+Fix a memory leak in asyncio in the ProactorEventLoop when ``ReadFile()`` or
+``WSASend()`` overlapped operation fail immediately: release the internal
+buffer.
diff --git a/Modules/overlapped.c b/Modules/overlapped.c
index ae7cddadd02d..4ef5a96b1e4d 100644
--- a/Modules/overlapped.c
+++ b/Modules/overlapped.c
@@ -723,6 +723,7 @@ do_ReadFile(OverlappedObject *self, HANDLE handle,
 case ERROR_IO_PENDING:
 Py_RETURN_NONE;
 default:
+ PyBuffer_Release(&self->user_buffer);
 self->type = TYPE_NOT_STARTED;
 return SetFromWindowsErr(err);
 }
@@ -1011,6 +1012,7 @@ Overlapped_WSASend(OverlappedObject *self, PyObject *args)
 case ERROR_IO_PENDING:
 Py_RETURN_NONE;
 default:
+ PyBuffer_Release(&self->user_buffer);
 self->type = TYPE_NOT_STARTED;
 return SetFromWindowsErr(err);
 }
