sw/source/core/docnode/nodes.cxx | 1 +
1 file changed, 1 insertion(+)
New commits:
commit 2f0e8176defbac0f19c06e3ec75fb57567e8ef52
Author: Caolán McNamara <[email protected]>
AuthorDate: Sun Oct 24 16:29:49 2021 +0100
Commit: Caolán McNamara <[email protected]>
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 <[email protected]>
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 <[email protected]>
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)];
