avoid crash in 130+ odt export tests

e.g. fdo44181-1.ott to odt

Change-Id: I23403415378ddf50ff007fb4b5d88b4009668bf6
This commit is contained in:
Caolán McNamara
2016-12-22 15:42:10 +00:00
parent 89d067775a
commit 2085d8203b

View File

@@ -1196,14 +1196,14 @@ void SwTextPaintInfo::DrawBackBrush( const SwLinePortion &rPor ) const
nLen = pPos->GetLen();
for ( int i = nIdx; i < (nIdx + nLen); ++i )
{
if ( GetText()[i] == CH_TXTATR_NEWLINE )
if (i < GetText().getLength() && GetText()[i] == CH_TXTATR_NEWLINE)
{
if ( i >= (GetIdx() + rPor.GetLen()) )
{
goto drawcontinue;
}
}
if ( GetText()[i] != CH_BLANK )
if (i >= GetText().getLength() || GetText()[i] != CH_BLANK)
{
draw = true;
if ( i >= (GetIdx() + rPor.GetLen()) )
@@ -1230,11 +1230,11 @@ void SwTextPaintInfo::DrawBackBrush( const SwLinePortion &rPor ) const
nLen = pPos->GetLen();
for ( int i = (nIdx + nLen - 1); i >= nIdx; --i )
{
if ( GetText()[i] == CH_TXTATR_NEWLINE )
if (i < GetText().getLength() && GetText()[i] == CH_TXTATR_NEWLINE)
{
continue;
}
if ( GetText()[i] != CH_BLANK )
if (i >= GetText().getLength() || GetText()[i] != CH_BLANK)
{
sal_uInt16 nOldWidth = rPor.Width();
sal_uInt16 nNewWidth = GetTextSize( m_pOut, nullptr, GetText(), nIdx, (i + 1 - nIdx) ).Width();