Whoops! One the "redundant" initializations removed by Vladimir in

the previous patch wasn't -- there was a path through the code that
bypassed all initializations.  Thanks to Just for reporting the bug!

diff --git a/Python/ceval.c b/Python/ceval.c
index e3660727b2da3f5df127d27a7e502f2f605c5fe2..bcc2288d8b26321dca117cfe2edef8d51c3fc11f 100644 (file)
--- a/Python/ceval.c
+++ b/Python/ceval.c
@@ -364,7 +364,7 @@ eval_code2(co, globals, locals,
        register PyObject *t;
        register PyFrameObject *f; /* Current frame */
        register PyObject **fastlocals;
-       PyObject *retval;       /* Return value */
+       PyObject *retval = NULL;        /* Return value */
        PyThreadState *tstate = PyThreadState_Get();
        unsigned char *first_instr;
 #ifdef LLTRACE
@@ -1769,7 +1769,7 @@ eval_code2(co, globals, locals,
                        if (tstate->sys_profilefunc)
                                call_exc_trace(&tstate->sys_profilefunc,
                                               (PyObject**)0, f);
-}
+               }
                
                /* For the rest, treat WHY_RERAISE as WHY_EXCEPTION */