Swiftdiff options
| author | Edwin Mons <edwin.mons@isode.com> | 2014-05-23 12:52:30 (GMT) |
|---|---|---|
| committer | Swift Review <review@swift.im> | 2014-06-08 09:32:50 (GMT) |
| commit | eb6fdb866e5ffd467ad0f08fb4bcc50a5af37a22 (patch) | |
| tree | bd30aec78656a0d9b49775e4070f3f428dd94cb6 | |
| parent | 58396750ef7639701b3cd2c5c48f7e867ccfd8c7 (diff) | |
| download | swift-eb6fdb866e5ffd467ad0f08fb4bcc50a5af37a22.zip swift-eb6fdb866e5ffd467ad0f08fb4bcc50a5af37a22.tar.bz2 | |
Fix crash in Sluift DOMElementConvertor
DOMElementConvertor crashed because the stack wasn't large enough for
deeper nested element trees. Calling lua_checkstack grows the stack if
there's need for it. Normally Lua does this for you, but the
DOMElementConverter can spend quite some time in non-Lua space.
Change-Id: I23e563d49191b1db1204d2f6fa7d30e2e9d05c1f
| -rw-r--r-- | Sluift/ElementConvertors/DOMElementConvertor.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/Sluift/ElementConvertors/DOMElementConvertor.cpp b/Sluift/ElementConvertors/DOMElementConvertor.cpp index fb1f658..784fcfd 100644 --- a/Sluift/ElementConvertors/DOMElementConvertor.cpp +++ b/Sluift/ElementConvertors/DOMElementConvertor.cpp @@ -35,6 +35,7 @@ namespace { virtual void handleStartElement( const std::string& element, const std::string& ns, const AttributeMap& attributes) SWIFTEN_OVERRIDE { + lua_checkstack(L, 6); lua_pushnumber(L, currentIndex); lua_newtable(L); lua_pushstring(L, element.c_str()); @@ -78,6 +79,7 @@ namespace { } virtual void handleCharacterData(const std::string& data) SWIFTEN_OVERRIDE { + lua_checkstack(L, 2); lua_pushnumber(L, currentIndex); lua_pushstring(L, data.c_str()); lua_settable(L, -3); |