UnityMirror/spine-runtimes
1
0
Fork 0
mirror of https://github.com/EsotericSoftware/spine-runtimes.git synced 2025-12-20 17:26:01 +08:00
Code Issues Packages Projects Releases Wiki Activity

[cpp] Change traversal order of Json::~Json(). Old order would grow stack much more than needed, leading to crashes when freeing big Json trees.

Browse Source
This commit is contained in:
badlogic 2019-10-30 15:48:55 +01:00
parent d2da9e795c
commit 479c236835
1 changed files with 15 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
spine-cpp/spine-cpp/src/spine/Json.cpp
View File

@ -120,17 +120,23 @@ Json::Json(const char *value) :
}
Json::~Json() {
delete _child;
spine::Json* curr = nullptr;
spine::Json* next = _child;
do {
curr = next;
if (curr) {
next = curr->_next;
}
delete curr;
} while(next);
if (_valueString) {
SpineExtension::free(_valueString, __FILE__, __LINE__);
}
if (_valueString) {
SpineExtension::free(_valueString, __FILE__, __LINE__);
}
if (_name) {
SpineExtension::free(_name, __FILE__, __LINE__);
}
delete _next;
if (_name) {
SpineExtension::free(_name, __FILE__, __LINE__);
}
}
const char *Json::skip(const char *inValue) {