[Libreoffice-commits] core.git: sw/source

Caolán McNamara caolanm at redhat.com
Thu Jun 4 13:01:09 PDT 2015


 sw/source/core/fields/expfld.cxx |    7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

New commits:
commit 4e96b278965875a95f02993590f30ab5859c1fed
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Thu Jun 4 21:00:13 2015 +0100

    Resolves: tdf#91637 avoid -1 string index
    
    Change-Id: I006040f4589065141310fe2a89b8d2ba3e54ad2c

diff --git a/sw/source/core/fields/expfld.cxx b/sw/source/core/fields/expfld.cxx
index b9007ab..e7da510 100644
--- a/sw/source/core/fields/expfld.cxx
+++ b/sw/source/core/fields/expfld.cxx
@@ -746,8 +746,11 @@ bool SwSeqFieldList::SeekEntry( const _SeqFieldLstElem& rNew, size_t* pP ) const
                 sal_Int32 nNum1 = sNum1.toInt32();
                 nCmp = nNum2 - nNum1;
                 if( 0 == nCmp )
-                    nCmp = rCaseColl.compareString( rTmp2.copy( nFndPos2 ),
-                                                    rTmp1.copy( nFndPos1 ));
+                {
+                    OUString aTmp1 = nFndPos1 != -1 ? rTmp1.copy(nFndPos1) : OUString();
+                    OUString aTmp2 = nFndPos2 != -1 ? rTmp2.copy(nFndPos2) : OUString();
+                    nCmp = rCaseColl.compareString(aTmp2, aTmp1);
+                }
             }
             else
                 nCmp = rColl.compareString( rTmp2, rTmp1 );


More information about the Libreoffice-commits mailing list