Hi all,

https://bugs.freedesktop.org/show_bug.cgi?id=36545

This one is a trivial one, please see the attachment.  All in all, just
check for validity of a pointer we are going to dereference...

Please check, sign-off, and push to libreoffice-3-4

Regards,
Kendy
>From 02c0c09f0a8cb22b876b25ff670127cc85df1ddd Mon Sep 17 00:00:00 2001
From: Jan Holesovsky <ke...@suse.cz>
Date: Fri, 1 Jul 2011 11:06:15 +0200
Subject: [PATCH] Don't crash when selecting target to db document, fdo#36545.

---
 dbaccess/source/ui/app/AppController.cxx |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/dbaccess/source/ui/app/AppController.cxx b/dbaccess/source/ui/app/AppController.cxx
index 7545e73..ba65284 100644
--- a/dbaccess/source/ui/app/AppController.cxx
+++ b/dbaccess/source/ui/app/AppController.cxx
@@ -1936,7 +1936,8 @@ Reference< XComponent > OApplicationController::openElementWithArguments( const
 IMPL_LINK( OApplicationController, OnSelectContainer, void*, _pType )
 {
     ElementType eType = (ElementType)reinterpret_cast< sal_IntPtr >( _pType );
-    getContainer()->selectContainer(eType);
+    if (getContainer())
+        getContainer()->selectContainer(eType);
     return 0L;
 }
 // -----------------------------------------------------------------------------
-- 
1.7.5.4

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to