[Python-checkins] bpo-37316: mmap.mmap() passes the wrong variable to PySys_Audit() (GH-14152)
Steve Dower
webhook-mailer at python.org
Fri Jun 21 12:45:17 EDT 2019
https://github.com/python/cpython/commit/6c7947713a40851e123493ca0fe8f2791d7ed2a6
commit: 6c7947713a40851e123493ca0fe8f2791d7ed2a6
branch: 3.8
author: Steve Dower <steve.dower at python.org>
committer: GitHub <noreply at github.com>
date: 2019-06-21T09:45:13-07:00
summary:
bpo-37316: mmap.mmap() passes the wrong variable to PySys_Audit() (GH-14152)
Also, add a missing call to va_end() in PySys_Audit().
files:
A Misc/NEWS.d/next/Core and Builtins/2019-06-17-03-53-16.bpo-37316.LytDX_.rst
M Lib/test/audit-tests.py
M Lib/test/test_audit.py
M Modules/mmapmodule.c
M Python/sysmodule.c
diff --git a/Lib/test/audit-tests.py b/Lib/test/audit-tests.py
index 7a7725f7c212..ddeff22030a4 100644
--- a/Lib/test/audit-tests.py
+++ b/Lib/test/audit-tests.py
@@ -261,6 +261,13 @@ def trace(frame, event, *args):
assertSequenceEqual(["call"] * 4, traced)
+def test_mmap():
+ import mmap
+ with TestHook() as hook:
+ mmap.mmap(-1, 8)
+ assertEqual(hook.seen[0][1][:2], (-1, 8))
+
+
if __name__ == "__main__":
from test.libregrtest.setup import suppress_msvcrt_asserts
suppress_msvcrt_asserts(False)
diff --git a/Lib/test/test_audit.py b/Lib/test/test_audit.py
index f629b7b3d073..2fc41bddcb8a 100644
--- a/Lib/test/test_audit.py
+++ b/Lib/test/test_audit.py
@@ -74,6 +74,9 @@ def test_open(self):
def test_cantrace(self):
self.do_test("test_cantrace")
+ def test_mmap(self):
+ self.do_test("test_mmap")
+
if __name__ == "__main__":
unittest.main()
diff --git a/Misc/NEWS.d/next/Core and Builtins/2019-06-17-03-53-16.bpo-37316.LytDX_.rst b/Misc/NEWS.d/next/Core and Builtins/2019-06-17-03-53-16.bpo-37316.LytDX_.rst
new file mode 100644
index 000000000000..40436d467c85
--- /dev/null
+++ b/Misc/NEWS.d/next/Core and Builtins/2019-06-17-03-53-16.bpo-37316.LytDX_.rst
@@ -0,0 +1 @@
+Fix the :c:func:`PySys_Audit` call in :class:`mmap.mmap`.
diff --git a/Modules/mmapmodule.c b/Modules/mmapmodule.c
index 755f1669d8d3..9e3414f94e31 100644
--- a/Modules/mmapmodule.c
+++ b/Modules/mmapmodule.c
@@ -1154,7 +1154,7 @@ new_mmap_object(PyTypeObject *type, PyObject *args, PyObject *kwdict)
}
if (PySys_Audit("mmap.__new__", "ini" _Py_PARSE_OFF_T,
- fileno, map_size, access, offset) < 0) {
+ fd, map_size, access, offset) < 0) {
return NULL;
}
diff --git a/Python/sysmodule.c b/Python/sysmodule.c
index 12b1bd7711d5..fbccea3c14fe 100644
--- a/Python/sysmodule.c
+++ b/Python/sysmodule.c
@@ -160,6 +160,7 @@ PySys_Audit(const char *event, const char *argFormat, ...)
va_list args;
va_start(args, argFormat);
eventArgs = Py_VaBuildValue(argFormat, args);
+ va_end(args);
if (eventArgs && !PyTuple_Check(eventArgs)) {
PyObject *argTuple = PyTuple_Pack(1, eventArgs);
Py_DECREF(eventArgs);
More information about the Python-checkins
mailing list