v4games
/
assimp
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #4963 from avaneyev/fix-fbx-document-duplicate-id-leak 

Fix a leak in FBXDocument when duplicate object IDs are found
pull/4107/head
Kim Kulling 2023-02-23 10:53:40 +01:00 committed by GitHub
parent a14696b9ec eb73464055
commit 46ae8534f7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
code/AssetLib/FBX/FBXDocument.cpp
View File

@ -381,8 +381,10 @@ void Document::ReadObjects() {
DOMError("encountered object with implicitly defined id 0",el.second); DOMError("encountered object with implicitly defined id 0",el.second);
} }
if(objects.find(id) != objects.end()) { const auto foundObject = objects.find(id);
if(foundObject != objects.end()) {
DOMWarning("encountered duplicate object id, ignoring first occurrence",el.second); DOMWarning("encountered duplicate object id, ignoring first occurrence",el.second);
delete foundObject->second;
} }
objects[id] = new LazyObject(id, *el.second, *this); objects[id] = new LazyObject(id, *el.second, *this);