sw/source/core/doc/docbm.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits: commit 74bb4f6f4ccc6877de12d22d1d8badd955ed1144 Author: Mike Kaganski <mike.kagan...@collabora.com> AuthorDate: Sun Apr 28 13:20:33 2019 +0300 Commit: Mike Kaganski <mike.kagan...@collabora.com> CommitDate: Sun Apr 28 13:17:52 2019 +0200 tdf#120703 PVS: Avoid dereferencing nullptr V595 The 'pEndIdx' pointer was utilized before it was verified against nullptr. Check lines: 1328, 1347. Change-Id: If9aff472e6ec7429ef2055b44dd11e62d4489ba4 Reviewed-on: https://gerrit.libreoffice.org/71458 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com> diff --git a/sw/source/core/doc/docbm.cxx b/sw/source/core/doc/docbm.cxx index 840462916484..ac20908aff32 100644 --- a/sw/source/core/doc/docbm.cxx +++ b/sw/source/core/doc/docbm.cxx @@ -1458,7 +1458,7 @@ void DelBookmarks( { // illegal range ?? if(rStt.GetIndex() > rEnd.GetIndex() - || (rStt == rEnd && (!pSttIdx || pSttIdx->GetIndex() >= pEndIdx->GetIndex()))) + || (rStt == rEnd && (!pSttIdx || !pEndIdx || pSttIdx->GetIndex() >= pEndIdx->GetIndex()))) return; SwDoc* const pDoc = rStt.GetNode().GetDoc(); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits