From 60cbddbb5faf75e3c2df1e037eed759997dfa541 Mon Sep 17 00:00:00 2001 From: Charles Pigott Date: Sat, 2 Nov 2019 12:50:42 +0000 Subject: Fix #7606: Game crash when trying to clean up a crashed script Also fix another possible memory leak --- src/script/script_instance.cpp | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'src/script') diff --git a/src/script/script_instance.cpp b/src/script/script_instance.cpp index 2c83e8838..a38a315ff 100644 --- a/src/script/script_instance.cpp +++ b/src/script/script_instance.cpp @@ -89,6 +89,10 @@ void ScriptInstance::Initialize(const char *main_script, const char *instance_na /* Create the main-class */ this->instance = new SQObject(); if (!this->engine->CreateClassInstance(instance_name, this->controller, this->instance)) { + /* If CreateClassInstance has returned false instance has not been + * registered with squirrel, so avoid trying to Release it by clearing it now */ + delete this->instance; + this->instance = nullptr; this->Died(); return; } @@ -154,6 +158,7 @@ void ScriptInstance::Died() this->last_allocated_memory = this->GetAllocatedMemory(); // Update cache if (this->instance != nullptr) this->engine->ReleaseObject(this->instance); + delete this->instance; delete this->engine; this->instance = nullptr; this->engine = nullptr; -- cgit v1.2.3-54-g00ecf