From ec76f39ab52bbfa8d2ce18e28856a443c4fd1d6e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= <caolanm@redhat.com>
Date: Thu, 13 Mar 2014 10:54:33 +0000
Subject: [PATCH] coverity#705808 Dereference before null check

Change-Id: Ibeaf8989e54017f7cf30a4ffc3f639cd43983d03
---
 dbaccess/source/ui/querydesign/QueryTableView.cxx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dbaccess/source/ui/querydesign/QueryTableView.cxx b/dbaccess/source/ui/querydesign/QueryTableView.cxx
index 1843379ff6c9..2cd7e56116bd 100644
--- a/dbaccess/source/ui/querydesign/QueryTableView.cxx
+++ b/dbaccess/source/ui/querydesign/QueryTableView.cxx
@@ -969,11 +969,11 @@ sal_Bool OQueryTableView::ExistsAVisitedConn(const OQueryTableWindow* pFrom) con
             OQueryTableConnection* pTemp = static_cast<OQueryTableConnection*>(*aIter);
             if (pTemp->IsVisited() &&
                 (pFrom == static_cast< OQueryTableWindow*>(pTemp->GetSourceWin()) || pFrom == static_cast< OQueryTableWindow*>(pTemp->GetDestWin())))
-                return pTemp != NULL;
+                return true;
         }
     }
 
-    return sal_False;
+    return false;
 }
 
 void OQueryTableView::onNoColumns_throw()