wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Backed out changeset 0c0a5a6b57d9 (bug 616491) for jsreftest failures

Browse Source
This commit is contained in:
Wes Kocher 2014-04-21 16:11:20 -07:00
parent b86f7298a0
commit c669a16e2d
1 changed files with 13 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
js/src/yarr/YarrPattern.cpp
View File

@ -304,22 +304,6 @@ public:
m_pattern.m_disjunctions.append(m_pattern.m_body);
}
void setStackBase(uint8_t *stackBase) {
m_stackBase = stackBase;
}
bool isOverRecursed() {
/*
* Bug 616491: attempt detection of over-recursion.
* "256KB should be enough stack for anyone."
*/
uint8_t stackDummy_;
JS_ASSERT(m_stackBase != nullptr);
if (m_stackBase - &stackDummy_ > (1 << 18))
return true;
return false;
}
void assertionBOL()
{
if (!m_alternative->m_terms.size() & !m_invertParentheticalAssertion) {
@ -590,7 +574,12 @@ public:
ErrorCode setupAlternativeOffsets(PatternAlternative* alternative, unsigned currentCallFrameSize, unsigned initialInputPosition,
unsigned *callFrameSizeOut)
{
if (isOverRecursed())
/*
* Attempt detection of over-recursion:
* "1MB should be enough stack for anyone."
*/
uint8_t stackDummy_;
if (m_stackBase - &stackDummy_ > 1024*1024)
return PatternTooLarge;
alternative->m_hasFixedSize = true;
@ -685,9 +674,6 @@ public:
ErrorCode setupDisjunctionOffsets(PatternDisjunction* disjunction, unsigned initialCallFrameSize, unsigned initialInputPosition, unsigned *maximumCallFrameSizeOut)
{
if (isOverRecursed())
return PatternTooLarge;
if ((disjunction != m_pattern.m_body) && (disjunction->m_alternatives.size() > 1))
initialCallFrameSize += YarrStackSpaceForBackTrackInfoAlternative;
@ -864,6 +850,10 @@ public:
}
}
void setStackBase(uint8_t *stackBase) {
m_stackBase = stackBase;
}
private:
YarrPattern& m_pattern;
uint8_t * m_stackBase;
@ -877,9 +867,6 @@ ErrorCode YarrPattern::compile(const String& patternString)
{
YarrPatternConstructor constructor(*this);
uint8_t stackDummy_;
constructor.setStackBase(&stackDummy_);
if (ErrorCode error = parse(constructor, patternString))
return error;
@ -900,6 +887,9 @@ ErrorCode YarrPattern::compile(const String& patternString)
ASSERT(numSubpatterns == m_numSubpatterns);
}
uint8_t stackDummy_;
constructor.setStackBase(&stackDummy_);
constructor.checkForTerminalParentheses();
constructor.optimizeDotStarWrappedExpressions();
constructor.optimizeBOL();