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

[Caolán McNamara (via logerrit)]

 sw/source/core/docnode/nodes.cxx |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit 2f0e8176defbac0f19c06e3ec75fb57567e8ef52
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Sun Oct 24 16:29:49 2021 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Sun Oct 24 18:23:59 2021 +0200

    cid#1493239 silence bogus Use after free
    
    code was perturbed by
    
    commit 5f9ffc31cd1b5433c354c7d39ce1d80fa0e57fc8
    Author: Noel Grandin <noel.gran...@collabora.co.uk>
    Date:   Tue Oct 19 09:08:35 2021 +0200
    
        introduce SwNodeOffset strong typedef
    
    so coverity has seen in afresh
    
    Change-Id: I200f8f40e193f8725ccc1fa5fbb06e3d182bec67
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124118
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sw/source/core/docnode/nodes.cxx b/sw/source/core/docnode/nodes.cxx
index 92e3692ec6bc..97e229d8e232 100644
--- a/sw/source/core/docnode/nodes.cxx
+++ b/sw/source/core/docnode/nodes.cxx
@@ -2272,6 +2272,7 @@ void SwNodes::RemoveNode( SwNodeOffset nDelPos, 
SwNodeOffset nSz, bool bDel )
         while( nCnt-- )
         {
             delete pDel;
+            // coverity[use_after_free] - pPrev will be reassigned if there 
will be another iteration to the loop
             pDel = pPrev;
             sal_uLong nPrevNdIdx = pPrev->GetPos();
             BigPtrEntry* pTempEntry = &aTempEntries[sal_Int32(nCnt)];