Kaydet (Commit) 9f2d4510 authored tarafından Eike Rathke's avatar Eike Rathke

assert(mnRangeOpPosInSymbol < MAXSTRLEN)

We should survive that now..

Change-Id: I4cadfa788bda470ee08958347671d843522669e8
Reviewed-on: https://gerrit.libreoffice.org/45820Reviewed-by: 's avatarEike Rathke <erack@redhat.com>
Tested-by: 's avatarJenkins <ci@libreoffice.org>
üst aa0b0898
...@@ -3268,8 +3268,9 @@ bool ScCompiler::IsReference( const OUString& rName, const OUString* pErrRef ) ...@@ -3268,8 +3268,9 @@ bool ScCompiler::IsReference( const OUString& rName, const OUString* pErrRef )
return true; return true;
// Now try with a symbol up to the range operator, rewind source // Now try with a symbol up to the range operator, rewind source
// position. // position.
if (mnRangeOpPosInSymbol == MAXSTRLEN-1) assert(mnRangeOpPosInSymbol < MAXSTRLEN); // We should have caught the maldoers.
return false; if (mnRangeOpPosInSymbol >= MAXSTRLEN) // TODO: this check and return
return false; // can be removed when sure.
sal_Int32 nLen = mnRangeOpPosInSymbol; sal_Int32 nLen = mnRangeOpPosInSymbol;
while (cSymbol[++nLen]) while (cSymbol[++nLen])
; ;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment