diff options
| author | Mark Hung <marklh9@gmail.com> | 2015-03-25 23:17:11 +0800 |
|---|---|---|
| committer | Michael Stahl <mstahl@redhat.com> | 2015-05-20 13:14:52 +0000 |
| commit | 5b63043dbc12476ad1c1e69558b53d9493c855f1 (patch) | |
| tree | 2c90554e95e358eba9825c049a8e116773923dc9 /sw | |
| parent | ab6ea2f8e73c4a0785bb2a6b256fc521ee120d98 (diff) | |
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@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
(cherry picked from commit 77cb94613be6315b618783b2aa76226ad3107163)
Reviewed-on: https://gerrit.libreoffice.org/15820
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
Diffstat (limited to 'sw')
| -rw-r--r-- | sw/source/core/doc/DocumentRedlineManager.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sw/source/core/doc/DocumentRedlineManager.cxx b/sw/source/core/doc/DocumentRedlineManager.cxx index 873b4d60d975..3da01ccc3585 100644 --- a/sw/source/core/doc/DocumentRedlineManager.cxx +++ b/sw/source/core/doc/DocumentRedlineManager.cxx @@ -1971,7 +1971,7 @@ bool DocumentRedlineManager::DeleteRedline( const SwPaM& rRange, bool bSaveInUnd if( !pRedl->HasValidRange() ) { // re-insert - mpRedlineTbl->Remove( pRedl ); + mpRedlineTbl->Remove( n ); mpRedlineTbl->Insert( pRedl ); --n; } |