fixed leak in NewReference.Move

lostmsu · lostmsu · commit 469ec671fc9d · 2022-07-14T11:40:59.000-07:00
fixes #1872
diff --git a/src/runtime/Native/NewReference.cs b/src/runtime/Native/NewReference.cs
@@ -47,7 +47,7 @@ public PyObject MoveToPyObject()
         /// </summary>
         public NewReference Move()
         {
-            var result = new NewReference(this);
+            var result = DangerousFromPointer(this.DangerousGetAddress());
             this.pointer = default;
             return result;
         }
diff --git a/tests/test_constructors.py b/tests/test_constructors.py
@@ -3,6 +3,7 @@
 """Test CLR class constructor support."""
 
 import pytest
+import sys
 
 import System
 
@@ -71,6 +72,19 @@ def test_default_constructor_fallback():
         ob = DefaultConstructorMatching("2")
 
 
+def test_constructor_leak():
+    from System import Uri
+    from Python.Runtime import Runtime
+
+    uri = Uri("http://www.python.org")
+    Runtime.TryCollectingGarbage(20)
+    ref_count = sys.getrefcount(uri)
+
+    # check disabled due to GC uncertainty
+    # assert ref_count == 1
+
+
+
 def test_string_constructor():
     from System import String, Char, Array
 

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public PyObject MoveToPyObject()`
`47`	`47`	`/// </summary>`
`48`	`48`	`public NewReference Move()`
`49`	`49`	`{`
`50`		`- var result = new NewReference(this);`
	`50`	`+ var result = DangerousFromPointer(this.DangerousGetAddress());`
`51`	`51`	`this.pointer = default;`
`52`	`52`	`return result;`
`53`	`53`	`}`