diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-10-23 16:39:45 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-10-24 09:24:53 +0100 |
commit | bec28033628473bdfde0c5a9e1af522613968051 (patch) | |
tree | 159e16975a1f1751aafd9c6c1eca8206e4dffaee /basic/source | |
parent | coverity#704231 Logically dead code (diff) | |
download | core-bec28033628473bdfde0c5a9e1af522613968051.tar.gz core-bec28033628473bdfde0c5a9e1af522613968051.zip |
coverity#704558 Dereference after null check
Change-Id: I6fe44d8926acd185bb6bc671fb7df8ae935998c1
Diffstat (limited to 'basic/source')
-rw-r--r-- | basic/source/comp/exprnode.cxx | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/basic/source/comp/exprnode.cxx b/basic/source/comp/exprnode.cxx index e245fef19382..4340848bf66e 100644 --- a/basic/source/comp/exprnode.cxx +++ b/basic/source/comp/exprnode.cxx @@ -240,13 +240,12 @@ void SbiExprNode::CollectBits() // If a twig can be converted, True will be returned. In this case // the result is in the left twig. - void SbiExprNode::FoldConstants() { if( IsOperand() || eTok == LIKE ) return; if( pLeft ) pLeft->FoldConstants(); - if( pRight ) + if (pLeft && pRight) { pRight->FoldConstants(); if( pLeft->IsConstant() && pRight->IsConstant() @@ -419,7 +418,7 @@ void SbiExprNode::FoldConstants() } } } - else if( pLeft && pLeft->IsNumber() ) + else if (pLeft && pLeft->IsNumber()) { nVal = pLeft->nVal; delete pLeft; |