Commit93fcc1f

authored

pythonGH-97752: Clear theprevious member of newly-created generator/coroutine frames (pythonGH-97795)

1 parentd053c47 commit93fcc1fCopy full SHA for 93fcc1f

File tree

+24

-0

lines changed

+24

-0

lines changed

Lines changed: 19 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -206,6 +206,25 @@ def __del__(self):`
`206`	`206`	`finally:`
`207`	`207`	`gc.set_threshold(*thresholds)`
`208`	`208`
	`209`	`+deftest_ag_frame_f_back(self):`
	`210`	`+asyncdeff():`
	`211`	`+yield`
	`212`	`+ag=f()`
	`213`	`+self.assertIsNone(ag.ag_frame.f_back)`
	`214`	`+`
	`215`	`+deftest_cr_frame_f_back(self):`
	`216`	`+asyncdeff():`
	`217`	`+pass`
	`218`	`+cr=f()`
	`219`	`+self.assertIsNone(cr.cr_frame.f_back)`
	`220`	`+cr.close()# Suppress RuntimeWarning.`
	`221`	`+`
	`222`	`+deftest_gi_frame_f_back(self):`
	`223`	`+deff():`
	`224`	`+yield`
	`225`	`+gi=f()`
	`226`	`+self.assertIsNone(gi.gi_frame.f_back)`
	`227`	`+`
`209`	`228`
`210`	`229`
`211`	`230`	`classExceptionTest(unittest.TestCase):`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	+Fix possible data corruption or crashes when accessing the ``f_back`` member
	`2`	`+of newly-created generator or coroutine frames.`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -54,6 +54,9 @@ _PyFrame_Copy(_PyInterpreterFrame src, _PyInterpreterFrame dest)`
`54`	`54`	`assert(src->stacktop >=src->f_code->co_nlocalsplus);`
`55`	`55`	`Py_ssize_tsize= ((char)&src->localsplus[src->stacktop])- (char)src;`
`56`	`56`	`memcpy(dest,src,size);`
	`57`	`+// Don't leave a dangling pointer to the old frame when creating generators`
	`58`	`+// and coroutines:`
	`59`	`+dest->previous=NULL;`
`57`	`60`	`}`
`58`	`61`
`59`	`62`

Comments

(0)