diff options
| author | Miklos Vajna <vmiklos@collabora.co.uk> | 2013-12-17 10:07:10 +0100 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2013-12-17 10:21:38 +0100 |
| commit | 4d2be4e7fa811db504a0fade72514b514ac84c96 (patch) | |
| tree | 85d68893384f6cb0230a793d1864c4443670abed /starmath/source | |
| parent | 4a66c737a137ec6a4c4578901edb6db8a724f4bb (diff) | |
fdo#61638 SmParser::NextToken: don't crash on ending dot
Change-Id: Ia9c168e52a99286910c9e63e0e052671c5259ba8
(cherry picked from commit cdd59ee13b1dca7e3d269bf1a3c555903eee96ee)
Diffstat (limited to 'starmath/source')
| -rw-r--r-- | starmath/source/parse.cxx | 38 |
1 files changed, 22 insertions, 16 deletions
diff --git a/starmath/source/parse.cxx b/starmath/source/parse.cxx index 5a79bbb7cd11..98b579a5f88f 100644 --- a/starmath/source/parse.cxx +++ b/starmath/source/parse.cxx @@ -872,24 +872,30 @@ void SmParser::NextToken() break; case '.': { - // for compatibility with SO5.2 - // texts like .34 ...56 ... h ...78..90 - // will be treated as numbers - m_aCurToken.eType = TNUMBER; - m_aCurToken.cMathChar = '\0'; - m_aCurToken.nGroup = 0; - m_aCurToken.nLevel = 5; - - sal_Int32 nTxtStart = m_nBufferIndex; - sal_Unicode cChar; - do + // Only one character? Then it can't be a number. + if (m_nBufferIndex < m_aBufferString.getLength() - 1) { - cChar = m_aBufferString[ ++m_nBufferIndex ]; + // for compatibility with SO5.2 + // texts like .34 ...56 ... h ...78..90 + // will be treated as numbers + m_aCurToken.eType = TNUMBER; + m_aCurToken.cMathChar = '\0'; + m_aCurToken.nGroup = 0; + m_aCurToken.nLevel = 5; + + sal_Int32 nTxtStart = m_nBufferIndex; + sal_Unicode cChar; + do + { + cChar = m_aBufferString[ ++m_nBufferIndex ]; + } + while ( cChar == '.' || rtl::isAsciiDigit( cChar ) ); + + m_aCurToken.aText = m_aBufferString.copy( nTxtStart, m_nBufferIndex - nTxtStart ); + aRes.EndPos = m_nBufferIndex; } - while ( cChar == '.' || rtl::isAsciiDigit( cChar ) ); - - m_aCurToken.aText = m_aBufferString.copy( nTxtStart, m_nBufferIndex - nTxtStart ); - aRes.EndPos = m_nBufferIndex; + else + bHandled = false; } break; case '/': |