summaryrefslogtreecommitdiff
path: root/drawinglayer/source/primitive3d
diff options
context:
space:
mode:
authorMike Kaganski <mike.kaganski@collabora.com>2019-04-01 00:25:16 +0300
committerMike Kaganski <mike.kaganski@collabora.com>2019-04-01 00:49:12 +0200
commit85456fae54029edd26df2277a9eec5e2fe3d9739 (patch)
treefd55ad48b34c66f0b839edcbc952a8a5a0752c72 /drawinglayer/source/primitive3d
parent1d35b89e7581e12024769d19bfb6d6eb2c3274b8 (diff)
tdf#120703 PVS: Silence V522 warnings
V522 There might be dereferencing of a potential null pointer. Change-Id: Ie617b41a8f8d334022cf5313b242a236baedba48 Reviewed-on: https://gerrit.libreoffice.org/70017 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'drawinglayer/source/primitive3d')
-rw-r--r--drawinglayer/source/primitive3d/baseprimitive3d.cxx8
1 files changed, 1 insertions, 7 deletions
diff --git a/drawinglayer/source/primitive3d/baseprimitive3d.cxx b/drawinglayer/source/primitive3d/baseprimitive3d.cxx
index 828e4fdcf14c..3e184400f612 100644
--- a/drawinglayer/source/primitive3d/baseprimitive3d.cxx
+++ b/drawinglayer/source/primitive3d/baseprimitive3d.cxx
@@ -166,14 +166,8 @@ namespace drawinglayer
const BasePrimitive3D* pA(dynamic_cast< const BasePrimitive3D* >(rxA.get()));
const BasePrimitive3D* pB(dynamic_cast< const BasePrimitive3D* >(rxB.get()));
- const bool bAEqualZero(pA == nullptr);
- if(bAEqualZero != (pB == nullptr))
- {
- return false;
- }
-
- if(bAEqualZero)
+ if(!pA || !pB)
{
return false;
}