Bug 73045

Summary: switching docks when no file is loaded crash kdevelop
Product: [Applications] kdevelop Reporter: tnagy <tnagy256>
Component: generalAssignee: KDevelop Developers <kdevelop-devel>
Status: RESOLVED FIXED    
Severity: crash CC: simon, sylvain.joyeux
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description tnagy 2004-01-20 18:31:45 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources
OS:          Linux

Open kdevelop, and do not open any project or file. Try "ctrl+alt+shift+r" two times, kdevelop will crash. This also works with the letters "b" and "r", or in the menu "view"->"tool dock"->"switch left dock", etc.
Comment 1 Jens Dagerbo 2004-01-20 20:16:30 UTC
Subject: kdelibs/kmdi

CVS commit by dagerbo: 

Quck fix for bug: 73045

CCMAIL: 73045-done@bugs.kde.org


  M +101 -101  kmdidockcontainer.cpp   1.55


--- kdelibs/kmdi/kmdidockcontainer.cpp  #1.54:1.55
@@ -613,5 +613,5 @@ void KMdiDockContainer::toggle() {
                 tabClicked(oldtab);
                 KMdiMainFrm *mainFrm = dynamic_cast<KMdiMainFrm*>(m_mainWin);
-                if (mainFrm)
+            if (mainFrm && mainFrm->activeWindow() )
                     mainFrm->activeWindow()->setFocus();
 


Comment 2 Jens Dagerbo 2004-01-20 20:17:01 UTC
Should be backported to KDE_3_2_BRANCH.
Comment 3 Jens Dagerbo 2004-01-22 12:20:40 UTC
*** Bug 70325 has been marked as a duplicate of this bug. ***
Comment 4 Lukáš Tinkl 2004-01-28 18:34:24 UTC
Subject: KDE_3_2_BRANCH: kdelibs/kmdi

CVS commit by lukas: 

backport for crash in #73045

CCMAIL: 73045@bugs.kde.org


  M +101 -101  kmdidockcontainer.cpp   1.54.2.1


--- kdelibs/kmdi/kmdidockcontainer.cpp  #1.54:1.54.2.1
@@ -613,5 +613,5 @@ void KMdiDockContainer::toggle() {
                 tabClicked(oldtab);
                 KMdiMainFrm *mainFrm = dynamic_cast<KMdiMainFrm*>(m_mainWin);
-                if (mainFrm)
+            if (mainFrm && mainFrm->activeWindow() )
                     mainFrm->activeWindow()->setFocus();
 


Comment 5 Jens Dagerbo 2004-02-20 23:49:32 UTC
*** Bug 75708 has been marked as a duplicate of this bug. ***