dbaccess/source/ui/browser/unodatbr.cxx |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit 4c602fd2554952fa3d2d2144fed3a19bdbfe4481
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Fri May 10 08:48:38 2024 +0100
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Fri May 10 11:44:11 2024 +0200

    cid#1597273 Unchecked return value
    
    Change-Id: Ia48abe54185c23f740b1385ca2bfd45950209f77
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167434
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/dbaccess/source/ui/browser/unodatbr.cxx 
b/dbaccess/source/ui/browser/unodatbr.cxx
index e8c6f007719e..201033332880 100644
--- a/dbaccess/source/ui/browser/unodatbr.cxx
+++ b/dbaccess/source/ui/browser/unodatbr.cxx
@@ -2159,9 +2159,7 @@ IMPL_LINK(SbaTableQueryBrowser, OnExpandEntry, const 
weld::TreeIter&, rParent, b
 
         // it could be that we already have a connection
         SharedConnection xConnection;
-        ensureConnection(xFirstParent.get(), xConnection);
-
-        if ( xConnection.is() )
+        if (ensureConnection(xFirstParent.get(), xConnection) && 
xConnection.is())
         {
             SQLExceptionInfo aInfo;
             try

Reply via email to