sc/source/core/tool/compiler.cxx | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
New commits:
commit 547230e672e179539b2477ff36acb00076149f55
Author: Caolán McNamara <caol...@redhat.com>
Date: Wed Nov 29 17:05:07 2017 +0000
ofz#4484 don't set mnRangeOpPosInSymbol on StringOverflow case
Change-Id: I35d86c97e4fb082a676a5eeadf8375738cecd7bd
Reviewed-on: https://gerrit.libreoffice.org/45565
Reviewed-by: Eike Rathke <er...@redhat.com>
Reviewed-by: Caolán McNamara <caol...@redhat.com>
Tested-by: Caolán McNamara <caol...@redhat.com>
diff --git a/sc/source/core/tool/compiler.cxx b/sc/source/core/tool/compiler.cxx
index 65780e5ad1e2..dce7841b80cd 100644
--- a/sc/source/core/tool/compiler.cxx
+++ b/sc/source/core/tool/compiler.cxx
@@ -2214,14 +2214,16 @@ Label_MaskStateMachine:
{
// One range operator may form Sheet1.A:A, which we need to
// pass as one entity to IsReference().
- mnRangeOpPosInSymbol = pSym - &cSymbol[0];
if( pSym == &cSymbol[ MAXSTRLEN ] )
{
SetError(FormulaError::StringOverflow);
eState = ssStop;
}
else
+ {
+ mnRangeOpPosInSymbol = pSym - &cSymbol[0];
*pSym++ = c;
+ }
}
else if ( 128 <= c || '\'' == c )
{ // High values need reparsing with i18n,
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
