diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2019-04-28 13:20:33 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2019-04-28 13:17:52 +0200 |
| commit | 74bb4f6f4ccc6877de12d22d1d8badd955ed1144 (patch) | |
| tree | 10a8338221086197d9e17084e886ac8e7c2901eb /sw | |
| parent | 65261babf4fbcba08deb7bc8871b8dbcbafcc26f (diff) | |
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.kaganski@collabora.com>
Diffstat (limited to 'sw')
| -rw-r--r-- | sw/source/core/doc/docbm.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
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(); |