Răsfoiți Sursa

Fixed ~CSharpScript() holding on to a mutex longer than necessary, creating potential for a deadlock.

(cherry picked from commit 9d50a486bf00592888664828b35d703c46eaa9b0)
Michael Wörner 1 an în urmă
părinte
comite
167b4c071c
1 a modificat fișierele cu 5 adăugiri și 3 ștergeri
  1. 5 3
      modules/mono/csharp_script.cpp

+ 5 - 3
modules/mono/csharp_script.cpp

@@ -2819,15 +2819,17 @@ CSharpScript::CSharpScript() {
 #ifdef DEBUG_ENABLED
 	{
 		MutexLock lock(CSharpLanguage::get_singleton()->script_instances_mutex);
-		CSharpLanguage::get_singleton()->script_list.add(&this->script_list);
+		CSharpLanguage::get_singleton()->script_list.add(&script_list);
 	}
 #endif
 }
 
 CSharpScript::~CSharpScript() {
 #ifdef DEBUG_ENABLED
-	MutexLock lock(CSharpLanguage::get_singleton()->script_instances_mutex);
-	CSharpLanguage::get_singleton()->script_list.remove(&this->script_list);
+	{
+		MutexLock lock(CSharpLanguage::get_singleton()->script_instances_mutex);
+		CSharpLanguage::get_singleton()->script_list.remove(&script_list);
+	}
 #endif
 
 	if (GDMonoCache::godot_api_cache_updated) {