Nixyri/godot
1
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2024-11-24 21:22:48 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #97315 from lawnjelly/fix_character_platform

Browse Source 
Fix physics platform behaviour regression
This commit is contained in:
Rémi Verschelde 2024-09-23 12:28:17 +02:00
commit b9b793237c
No known key found for this signature in database
GPG Key ID: C3336907360768E1
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
scene/3d/physics/character_body_3d.cpp
View File

@ -60,8 +60,13 @@ bool CharacterBody3D::move_and_slide() {
// We need to check the platform_rid object still exists before accessing. // We need to check the platform_rid object still exists before accessing.
// A valid RID is no guarantee that the object has not been deleted. // A valid RID is no guarantee that the object has not been deleted.
if (ObjectDB::get_instance(platform_object_id)) {
//this approach makes sure there is less delay between the actual body velocity and the one we saved // We can only perform the ObjectDB lifetime check on Object derived objects.
// Note that physics also creates RIDs for non-Object derived objects, these cannot
// be lifetime checked through ObjectDB, and therefore there is a still a vulnerability
// to dangling RIDs (access after free) in this scenario.
if (platform_object_id.is_null() || ObjectDB::get_instance(platform_object_id)) {
// This approach makes sure there is less delay between the actual body velocity and the one we saved.
bs = PhysicsServer3D::get_singleton()->body_get_direct_state(platform_rid); bs = PhysicsServer3D::get_singleton()->body_get_direct_state(platform_rid);
} }