diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-04-10 20:21:48 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-04-10 21:07:56 +0100 |
commit | 97865e0a72272d5be8c65d636eafdba04736a58d (patch) | |
tree | 59c0de4f8c817396c07b0be8d2a839ddfe4e1155 /dbaccess/source/ui/querydesign | |
parent | af110680a5dafffbb4ab13c23cb4c977afad16ca (diff) |
coverity#983369 Dereference before null check
Change-Id: Ifadcbe32aa64a9653f4e8dd8742dd3c5da22facd
Diffstat (limited to 'dbaccess/source/ui/querydesign')
-rw-r--r-- | dbaccess/source/ui/querydesign/QueryDesignView.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/dbaccess/source/ui/querydesign/QueryDesignView.cxx b/dbaccess/source/ui/querydesign/QueryDesignView.cxx index 2f105d74603f..c31239990d9c 100644 --- a/dbaccess/source/ui/querydesign/QueryDesignView.cxx +++ b/dbaccess/source/ui/querydesign/QueryDesignView.cxx @@ -1212,11 +1212,11 @@ namespace const ::connectivity::OSQLParseNode* pNode, sal_uInt16& rLevel ) { - if (!SQL_ISRULE(pNode, select_statement)) + if (!pNode || !SQL_ISRULE(pNode, select_statement)) return eNoSelectStatement; // nyi: more checking for the correct structure! - pNode = pNode ? pNode->getChild(3)->getChild(1) : NULL; + pNode = pNode->getChild(3)->getChild(1); // no where clause found if (!pNode || pNode->isLeaf()) return eOk; |