From: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Date: Wed, 26 Jun 2019 21:20:09 +0000 (-0700)
Subject: bpo-37419: Fix possible segfaults when passing large sequences to os.posix_spawn... 
X-Git-Tag: v3.8.0b2~46
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=04d4692579cc4e0204c7fbced3692f8aa4bbb857;p=python

bpo-37419: Fix possible segfaults when passing large sequences to os.posix_spawn() (GH-14409)


Use Py_ssize_t instead of int for i.
(cherry picked from commit d52a83a3d471ff3c7e9ebfa1731765e5334f7c24)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
---

diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index 2d68c9dbb4..70b15360a2 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -5380,7 +5380,7 @@ parse_file_actions(PyObject *file_actions,
         return -1;
     }
 
-    for (int i = 0; i < PySequence_Fast_GET_SIZE(seq); ++i) {
+    for (Py_ssize_t i = 0; i < PySequence_Fast_GET_SIZE(seq); ++i) {
         file_action = PySequence_Fast_GET_ITEM(seq, i);
         Py_INCREF(file_action);
         if (!PyTuple_Check(file_action) || !PyTuple_GET_SIZE(file_action)) {