From 4ac34cb134ca6445e0fa49536bb1395cac69e465 Mon Sep 17 00:00:00 2001 From: Jean-Francois Dockes Date: Thu, 15 May 2014 15:15:01 +0200 Subject: [PATCH] Off by one error in maximum embedding depth test caused overflow of FileInterner m_tmpflgs temp flags array and possibly bus error depending on arch (only seen on 32 bits arch) --- src/internfile/internfile.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/internfile/internfile.cpp b/src/internfile/internfile.cpp index f6a46eff..23aeeaf6 100644 --- a/src/internfile/internfile.cpp +++ b/src/internfile/internfile.cpp @@ -661,7 +661,7 @@ int FileInterner::addHandler() } // We need to stack another handler. Check stack size - if (m_handlers.size() > MAXHANDLERS) { + if (m_handlers.size() >= MAXHANDLERS) { // Stack too big. Skip this and go on to check if there is // something else in the current back() LOGERR(("FileInterner::addHandler: stack too high\n"));