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

Merge pull request #91806 from dsnopek/gdextension-shutdown-crash 

GDExtension: Prevent crash during shutdown as singletons are deleted
This commit is contained in:
Rémi Verschelde 2024-05-11 12:38:30 +02:00
parent 497a66e38b 14506a4282
commit 5e28260a10
No known key found for this signature in database
GPG Key ID: C3336907360768E1
4 changed files with 17 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
core/config/engine.cpp
View File

@ -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),

2
core/config/engine.h
View File

@ -187,7 +187,7 @@ public:
bool notify_frame_server_synced();
Engine();
virtual ~Engine() {}
virtual ~Engine();
};
#endif // ENGINE_H

3
core/extension/gdextension_manager.cpp
View File

@ -295,6 +295,9 @@ GDExtensionManager::GDExtensionManager() {
}
GDExtensionManager::~GDExtensionManager() {
if (singleton == this) {
singleton = nullptr;
}
#ifndef DISABLE_DEPRECATED
GDExtensionCompatHashes::finalize();
#endif

10
core/object/object.cpp
View File

@ -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