Bug 132957

Summary: Crash using dcop action: album_forward
Product: [Applications] digikam Reporter: Antonio E. <aironmail>
Component: Albums-EngineAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: crash CC: caulier.gilles
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
Latest Commit: Version Fixed In: 0.9.0
Sentry Crash Report:

Description Antonio E. 2006-08-25 10:36:49 UTC 
Version:           0.9-svn (using KDE KDE 3.5.4)
Installed from:    Gentoo Packages
Compiler:          gcc-4.1.1 
OS:                Linux

I described this problem in a comment in #113330, but that issue was a wish. I thought that it would be better to open a new bug for this crash, as it's independient of the description of the wish.

I repeat here the problem:

Playing with dcop I found a bug, that makes digikam to crash (I'm using 0.9svn).

If I have the Image Editor, and I make:

dcop digikam-2830 Digikam activateAction album_forward

Then, digikam crashes. It happens always.

I've tried a couple of other actions from the Image Editor and they work fine. I haven't try so many other dcop actions.
Comment 1 Antonio E. 2006-08-25 10:37:38 UTC 
The crash happens even with the Image Editor hidden.
Comment 2 Marcel Wiesweg 2006-08-25 15:33:08 UTC 
SVN commit 577073 by mwiesweg:

Set this to 0 as in the other similar methods

BUG: 132957



 M  +2 -1      NEWS  
 M  +3 -0      digikam/albumhistory.cpp  


--- trunk/extragear/graphics/digikam/NEWS #577072:577073
@@ -236,6 +236,7 @@
 142 ==> 128817 : configure timestamp naming format for directory names.
 143 ==> 132660 : Shortcuts for ratings do not work as of 0.9.x beta.
 144 ==> 126874 : digikam does not support <shift+del>.
-145 ==> 
+145 ==> 132957 : Crash using dcop action: album_forward
+146 ==>
 
 ----------------------------------------------------------------------------------------------------
--- trunk/extragear/graphics/digikam/digikam/albumhistory.cpp #577072:577073
@@ -268,6 +268,9 @@
 
 void AlbumHistory::forward(Album **album, QWidget **widget, unsigned int steps)
 {
+    *album = 0;
+    *widget = 0;
+
     if(m_forwardStack->isEmpty() || steps > m_forwardStack->count())
         return;
Comment 3 Antonio E. 2006-08-25 20:08:09 UTC 
Thank you very much for the patch