summaryrefslogtreecommitdiffstats
path: root/core
diff options
context:
space:
mode:
authorDavid Snopek <dsnopek@gmail.com>2024-05-10 12:07:21 -0500
committerDavid Snopek <dsnopek@gmail.com>2024-05-10 13:17:36 -0500
commit14506a4282b6b9079280ec54f50f46ba041c0d10 (patch)
tree566585e6fc1bd30b155414cc87b7d476b295c691 /core
parent2ba22d1554ded08094e3869a83bd87487266ad38 (diff)
downloadredot-engine-14506a4282b6b9079280ec54f50f46ba041c0d10.tar.gz
GDExtension: Prevent crash during shutdown as singletons are deleted
Diffstat (limited to 'core')
-rw-r--r--core/config/engine.cpp6
-rw-r--r--core/config/engine.h2
-rw-r--r--core/extension/gdextension_manager.cpp3
-rw-r--r--core/object/object.cpp10
4 files changed, 17 insertions, 4 deletions
diff --git a/core/config/engine.cpp b/core/config/engine.cpp
index f2f8aebe8b..3574430cf7 100644
--- a/core/config/engine.cpp
+++ b/core/config/engine.cpp
@@ -384,6 +384,12 @@ Engine::Engine() {
singleton = this;
}
+Engine::~Engine() {
+ if (singleton == this) {
+ singleton = nullptr;
+ }
+}
+
Engine::Singleton::Singleton(const StringName &p_name, Object *p_ptr, const StringName &p_class_name) :
name(p_name),
ptr(p_ptr),
diff --git a/core/config/engine.h b/core/config/engine.h
index 8dece803e3..7e617d8773 100644
--- a/core/config/engine.h
+++ b/core/config/engine.h
@@ -187,7 +187,7 @@ public:
bool notify_frame_server_synced();
Engine();
- virtual ~Engine() {}
+ virtual ~Engine();
};
#endif // ENGINE_H
diff --git a/core/extension/gdextension_manager.cpp b/core/extension/gdextension_manager.cpp
index a4d032f22f..1ee9de0776 100644
--- a/core/extension/gdextension_manager.cpp
+++ b/core/extension/gdextension_manager.cpp
@@ -295,6 +295,9 @@ GDExtensionManager::GDExtensionManager() {
}
GDExtensionManager::~GDExtensionManager() {
+ if (singleton == this) {
+ singleton = nullptr;
+ }
#ifndef DISABLE_DEPRECATED
GDExtensionCompatHashes::finalize();
#endif
diff --git a/core/object/object.cpp b/core/object/object.cpp
index dfc8e2a29a..5125ab8a6e 100644
--- a/core/object/object.cpp
+++ b/core/object/object.cpp
@@ -2095,9 +2095,13 @@ Object::~Object() {
_extension_instance = nullptr;
}
#ifdef TOOLS_ENABLED
- else if (_instance_bindings != nullptr && Engine::get_singleton()->is_extension_reloading_enabled()) {
- for (uint32_t i = 0; i < _instance_binding_count; i++) {
- GDExtensionManager::get_singleton()->untrack_instance_binding(_instance_bindings[i].token, this);
+ else if (_instance_bindings != nullptr) {
+ Engine *engine = Engine::get_singleton();
+ GDExtensionManager *gdextension_manager = GDExtensionManager::get_singleton();
+ if (engine && gdextension_manager && engine->is_extension_reloading_enabled()) {
+ for (uint32_t i = 0; i < _instance_binding_count; i++) {
+ gdextension_manager->untrack_instance_binding(_instance_bindings[i].token, this);
+ }
}
}
#endif