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

Fix dirty read of ObjectID counter when threads are involved

Browse Source
This commit is contained in:
Marc Gilleron 2018-09-29 15:45:09 +01:00
parent 6cbdeedf57
commit a392d553c2
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
core/object.cpp
View File

@ -2014,11 +2014,13 @@ ObjectID ObjectDB::add_instance(Object *p_object) {
ERR_FAIL_COND_V(p_object->get_instance_id() != 0, 0); ERR_FAIL_COND_V(p_object->get_instance_id() != 0, 0);
rw_lock->write_lock(); rw_lock->write_lock();
instances[++instance_counter] = p_object; ObjectID instance_id = ++instance_counter;
instance_checks[p_object] = instance_counter; instances[instance_id] = p_object;
instance_checks[p_object] = instance_id;
rw_lock->write_unlock(); rw_lock->write_unlock();
return instance_counter; return instance_id;
} }
void ObjectDB::remove_instance(Object *p_object) { void ObjectDB::remove_instance(Object *p_object) {
@ -2095,6 +2097,5 @@ void ObjectDB::cleanup() {
instances.clear(); instances.clear();
instance_checks.clear(); instance_checks.clear();
rw_lock->write_unlock(); rw_lock->write_unlock();
memdelete(rw_lock); memdelete(rw_lock);
} }