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

[Mark Hung]

 sw/source/core/doc/DocumentRedlineManager.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 77cb94613be6315b618783b2aa76226ad3107163
Author: Mark Hung <marklh9 at gmail.com>
Date:   Wed Mar 25 23:17:11 2015 +0800

    Fix DocumentRedlineManager code that crashes Writer.
    
    One of my document crashes Writer when I delete one table row, with
    change tracking enabled. I observed that in DeleteRedline(),
    either Start or End of the SwRangeRedline object has been changed
    so that it breaks the order of SwRedlineTable.
    
    Change-Id: I6e477488a894996dea4efb433eb3e03edf5fd26d
    Reviewed-on: https://gerrit.libreoffice.org/15002
    Tested-by: Caolán McNamara <caolanm at redhat.com>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/core/doc/DocumentRedlineManager.cxx b/sw/source/core/doc/DocumentRedlineManager.cxx
index bc31678..4fb65aa 100644
--- a/sw/source/core/doc/DocumentRedlineManager.cxx
+++ b/sw/source/core/doc/DocumentRedlineManager.cxx
@@ -1973,7 +1973,7 @@ bool DocumentRedlineManager::DeleteRedline( const SwPaM& rRange, bool bSaveInUnd
                     if( !pRedl->HasValidRange() )
                     {
                         // re-insert
-                        mpRedlineTbl->Remove( pRedl );
+                        mpRedlineTbl->Remove( n );
                         mpRedlineTbl->Insert( pRedl );
                         --n;
                     }