From 39a86c2188b22818c187e860fbf7d77a28a6a116 Mon Sep 17 00:00:00 2001 From: Tim Peters Date: Sun, 12 May 2002 07:19:38 +0000 Subject: SF bug 555042: zip() may trigger MemoryError. NOT a bugfix candidate: this is a fix to an optimization introduced in 2.3. --- Python/bltinmodule.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'Python/bltinmodule.c') diff --git a/Python/bltinmodule.c b/Python/bltinmodule.c index fb6810edb8..d0411e23ae 100644 --- a/Python/bltinmodule.c +++ b/Python/bltinmodule.c @@ -1717,13 +1717,18 @@ builtin_zip(PyObject *self, PyObject *args) /* args must be a tuple */ assert(PyTuple_Check(args)); - /* Guess at result length: the shortest of the input lengths. */ + /* Guess at result length: the shortest of the input lengths. + If some argument refuses to say, we refuse to guess too, lest + an argument like xrange(sys.maxint) lead us astray.*/ len = -1; /* unknown */ for (i = 0; i < itemsize; ++i) { PyObject *item = PyTuple_GET_ITEM(args, i); int thislen = PySequence_Length(item); - if (thislen < 0) + if (thislen < 0) { PyErr_Clear(); + len = -1; + break; + } else if (len < 0 || thislen < len) len = thislen; } -- cgit v1.2.1