diff options
| author | Guido van Rossum <guido@python.org> | 2002-06-12 03:45:21 (GMT) |
|---|---|---|
| committer | Guido van Rossum <guido@python.org> | 2002-06-12 03:45:21 (GMT) |
| commit | c5fe5eb8d2650026ae7ce9195cd0dfc153514d2a (patch) | |
| tree | a3244f08eeb082ad3c54f572340d426db313a573 /Python | |
| parent | 969de458aa12e831942637bbcd9994b29dc86252 (diff) | |
| download | cpython-c5fe5eb8d2650026ae7ce9195cd0dfc153514d2a.zip cpython-c5fe5eb8d2650026ae7ce9195cd0dfc153514d2a.tar.gz cpython-c5fe5eb8d2650026ae7ce9195cd0dfc153514d2a.tar.bz2 | |
SF bug 567538: Generator can crash the interpreter (Finn Bock).
This was a simple typo. Strange that the compiler didn't catch it!
Instead of WHY_CONTINUE, two tests used CONTINUE_LOOP, which isn't a
why_code at all, but an opcode; but even though 'why' is declared as
an enum, comparing it to an int is apparently not even worth a
warning -- not in gcc, and not in VC++. :-(
Will fix in 2.2 too.
Diffstat (limited to 'Python')
| -rw-r--r-- | Python/ceval.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Python/ceval.c b/Python/ceval.c index fd30e8f..b9c0d5d 100644 --- a/Python/ceval.c +++ b/Python/ceval.c @@ -1543,7 +1543,7 @@ eval_frame(PyFrameObject *f) why = (enum why_code) PyInt_AsLong(v); if (why == WHY_RETURN || why == WHY_YIELD || - why == CONTINUE_LOOP) + why == WHY_CONTINUE) retval = POP(); } else if (PyString_Check(v) || PyClass_Check(v)) { @@ -2293,7 +2293,7 @@ eval_frame(PyFrameObject *f) } else { if (why == WHY_RETURN || - why == CONTINUE_LOOP) + why == WHY_CONTINUE) PUSH(retval); v = PyInt_FromLong((long)why); PUSH(v); |