NotificationsYou must be signed in to change notification settings
Fork768
Star5.4k

Commit927f980

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 parentf506d65 commit927f980Copy full SHA for 927f980

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!

Commit927f980

File tree

1 file changed

1 file changed

`‎src/runtime/classderived.cs‎`

0 commit comments