[Libreoffice-commits] core.git: sc/source

[Julien Nabet]

 sc/source/ui/dbgui/consdlg.cxx    |    3 +--
 sc/source/ui/pagedlg/areasdlg.cxx |    3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

New commits:
commit 0bcc66f209bf04d3f8fde09635524d4bab468f6d
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Sat Sep 7 12:22:26 2013 +0200

    CID#704695/704708: Dereference after null check
    
    Change-Id: Ia0d9f61b20713295fb40629b7fb57d91c2b98619
    Reviewed-on: https://gerrit.libreoffice.org/5855
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    Tested-by: Markus Mohrhard <markus.mohrhard at googlemail.com>

diff --git a/sc/source/ui/dbgui/consdlg.cxx b/sc/source/ui/dbgui/consdlg.cxx
index d6c2985..23e2b23 100644
--- a/sc/source/ui/dbgui/consdlg.cxx
+++ b/sc/source/ui/dbgui/consdlg.cxx
@@ -280,9 +280,8 @@ void ScConsolidateDlg::SetReference( const ScRange& rRef, ScDocument* pDocP )
             aStr = rRef.aStart.Format(nFmt, pDocP, eConv);
 
         pRefInputEdit->SetRefString( aStr );
+        ModifyHdl( pRefInputEdit );
     }
-
-    ModifyHdl( pRefInputEdit );
 }
 
 
diff --git a/sc/source/ui/pagedlg/areasdlg.cxx b/sc/source/ui/pagedlg/areasdlg.cxx
index dd83ee1..268e03b 100644
--- a/sc/source/ui/pagedlg/areasdlg.cxx
+++ b/sc/source/ui/pagedlg/areasdlg.cxx
@@ -218,9 +218,8 @@ void ScPrintAreasDlg::SetReference( const ScRange& rRef, ScDocument* /* pDoc */
             lcl_GetRepeatRangeString(&rRef, pDoc, bRow, aStr);
             pRefInputEdit->SetRefString( aStr );
         }
+        Impl_ModifyHdl( pRefInputEdit );
     }
-
-    Impl_ModifyHdl( pRefInputEdit );
 }