godotengine/godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Mono/C#: Fix memory leak with new Reference instances created from C#

Browse Source
This commit is contained in:
Ignacio Etcheverry 2019-12-25 19:33:59 +01:00
parent 78f1513928
commit 0ef635c0e3
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
modules/mono/mono_gd/gd_mono_internals.cpp
View File

@ -73,7 +73,7 @@ void tie_managed_to_unmanaged(MonoObject *managed, Object *unmanaged) {
script_binding.inited = true; script_binding.inited = true;
script_binding.type_name = NATIVE_GDMONOCLASS_NAME(klass); script_binding.type_name = NATIVE_GDMONOCLASS_NAME(klass);
script_binding.wrapper_class = klass; script_binding.wrapper_class = klass;
script_binding.gchandle = MonoGCHandle::create_strong(managed); script_binding.gchandle = ref ? MonoGCHandle::create_weak(managed) : MonoGCHandle::create_strong(managed);
script_binding.owner = unmanaged; script_binding.owner = unmanaged;
if (ref) { if (ref) {
@ -81,7 +81,9 @@ void tie_managed_to_unmanaged(MonoObject *managed, Object *unmanaged) {
// This way if the unmanaged world has no references to our owner // This way if the unmanaged world has no references to our owner
// but the managed instance is alive, the refcount will be 1 instead of 0. // but the managed instance is alive, the refcount will be 1 instead of 0.
// See: godot_icall_Reference_Dtor(MonoObject *p_obj, Object *p_ptr) // See: godot_icall_Reference_Dtor(MonoObject *p_obj, Object *p_ptr)
ref->reference();
// May not me referenced yet, so we must use init_ref() instead of reference()
ref->init_ref();
} }
// The object was just created, no script instance binding should have been attached // The object was just created, no script instance binding should have been attached