mirror of
				https://github.com/python/cpython.git
				synced 2025-10-31 13:41:24 +00:00 
			
		
		
		
	[3.11] gh-94215: Fix error handling for line-tracing events (GH-94681) (GH-94688)
* Re-enable crasher
* Fix error handling for line-tracing events
* blurb add
(cherry picked from commit 23ee4a8067)
Co-authored-by: Brandt Bucher <brandtbucher@microsoft.com>
			
			
This commit is contained in:
		
							parent
							
								
									73a1800b55
								
							
						
					
					
						commit
						36a3372d51
					
				
					 3 changed files with 17 additions and 6 deletions
				
			
		|  | @ -5642,16 +5642,25 @@ _PyEval_EvalFrameDefault(PyThreadState *tstate, _PyInterpreterFrame *frame, int | |||
|                     err = maybe_call_line_trace(tstate->c_tracefunc, | ||||
|                                                 tstate->c_traceobj, | ||||
|                                                 tstate, frame, instr_prev); | ||||
|                     // Reload possibly changed frame fields:
 | ||||
|                     stack_pointer = _PyFrame_GetStackPointer(frame); | ||||
|                     frame->stacktop = -1; | ||||
|                     // next_instr is only reloaded if tracing *does not* raise.
 | ||||
|                     // This is consistent with the behavior of older Python
 | ||||
|                     // versions. If a trace function sets a new f_lineno and
 | ||||
|                     // *then* raises, we use the *old* location when searching
 | ||||
|                     // for an exception handler, displaying the traceback, and
 | ||||
|                     // so on:
 | ||||
|                     if (err) { | ||||
|                         /* trace function raised an exception */ | ||||
|                         // next_instr wasn't incremented at the start of this
 | ||||
|                         // instruction. Increment it before handling the error,
 | ||||
|                         // so that it looks the same as a "normal" instruction:
 | ||||
|                         next_instr++; | ||||
|                         goto error; | ||||
|                     } | ||||
|                     /* Reload possibly changed frame fields */ | ||||
|                     // Reload next_instr. Don't increment it, though, since
 | ||||
|                     // we're going to re-dispatch to the "true" instruction now:
 | ||||
|                     next_instr = frame->prev_instr; | ||||
| 
 | ||||
|                     stack_pointer = _PyFrame_GetStackPointer(frame); | ||||
|                     frame->stacktop = -1; | ||||
|                 } | ||||
|             } | ||||
|         } | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Christian Heimes
						Christian Heimes