NotificationsYou must be signed in to change notification settings
Fork753
Star5.1k

Commit1fd6dc2

committed

fixed crash in finalizer of CLR types defined in Python, that survive engine shutdown

#1256 #1256During engine shutdown all links from Python to .NET instances are severed. If an instance of CLR class defined in Python survives the shutdown (for example, a reference is stored in static field) and later gets finalized, it will attempt to severe link again, which is an invalid operation.The fix is to check if the link has already been severed and skip that step during finalization.

1 parent44a36dc commit1fd6dc2Copy full SHA for 1fd6dc2

File tree

1 file changed

-2

lines changed

src/runtime
- classderived.cs

1 file changed

-2

lines changed

`‎src/runtime/classderived.cs`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -857,7 +857,7 @@ public static void Finalize(IPythonDerivedType obj)`
`857`	`857`	`{`
`858`	`858`	`if(0==Runtime.Py_IsInitialized()\|\|Runtime.IsFinalizing)`
`859`	`859`	`{`
`860`		`-self.gcHandle.Free();`
	`860`	`+if(self.gcHandle.IsAllocated)self.gcHandle.Free();`
`861`	`861`	`return;`
`862`	`862`	`}`
`863`	`863`	`}`
`@@ -872,7 +872,7 @@ public static void Finalize(IPythonDerivedType obj)`
`872`	`872`	`// If python's been terminated then just free the gchandle.`
`873`	`873`	`if(0==Runtime.Py_IsInitialized()\|\|Runtime.IsFinalizing)`
`874`	`874`	`{`
`875`		`-self.gcHandle.Free();`
	`875`	`+if(self.gcHandle.IsAllocated)self.gcHandle.Free();`
`876`	`876`	`return;`
`877`	`877`	`}`
`878`	`878`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit1fd6dc2

File tree

1 file changed

1 file changed

`‎src/runtime/classderived.cs`

0 commit comments