commit 3776836f675f3d52b21713a45460f09a03af15df
author Benjamin Peterson <benjamin@python.org> Wed Oct 05 23:29:07 2016 -0700
committer Benjamin Peterson <benjamin@python.org> Wed Oct 05 23:29:07 2016 -0700
tree 923064bbeb013d3d1467db2c593fd3bac050ede6
parent 4c8b2cd12650a05d87f3cef7f457bfddaf79c0e0

do not leak buffer if mmap is not writable

Modules/mmapmodule.c

diff --git a/Modules/mmapmodule.c b/Modules/mmapmodule.c
index 6e2db61..e01de44 100644
--- a/Modules/mmapmodule.c
+++ b/Modules/mmapmodule.c
@@ -383,8 +383,10 @@
     if (!PyArg_ParseTuple(args, "y*:write", &data))
         return(NULL);
 
-    if (!is_writable(self))
+    if (!is_writable(self)) {
+        PyBuffer_Release(&data);
         return NULL;
+    }
 
     if (self->pos > self->size || self->size - self->pos < data.len) {
         PyBuffer_Release(&data);