Bug 339210 - digiKam crashes when trying to stop find duplicates
Summary: digiKam crashes when trying to stop find duplicates
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Searches-Similarity (show other bugs)
Version: 4.3.0
Platform: Ubuntu Linux
: NOR crash
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-19 12:44 UTC by Mark Fraser
Modified: 2014-09-19 22:48 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.4.0


Attachments
digiKam backtrace (10.78 KB, text/plain)
2014-09-19 12:45 UTC, Mark Fraser
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Fraser 2014-09-19 12:44:41 UTC
I ran find duplicates and then noticed that the process had finished but was still showing 100%. When I tried to stop the process, digiKam crashed.
I restarted digiKam and tried again, this time I tried to quit digiKam instead, but it still crashed.

Reproducible: Always

Steps to Reproduce:
1.Run "find duplicates"
2.When complete, try to stop still running process
3.digiKam crashes.
Comment 1 Mark Fraser 2014-09-19 12:45:46 UTC
Created attachment 88751 [details]
digiKam backtrace
Comment 2 caulier.gilles 2014-09-19 16:11:45 UTC
Already fixed with this commit :

https://projects.kde.org/projects/extragear/graphics/digikam/repository/revisions/30e65a812f5aff808173dad52100c93dcc24b988

Gilles Caulier
Comment 3 nucleo 2014-09-19 18:56:45 UTC
(In reply to Gilles Caulier from comment #2)
> Already fixed with this commit :
> 
> https://projects.kde.org/projects/extragear/graphics/digikam/repository/
> revisions/30e65a812f5aff808173dad52100c93dcc24b988
> 
> Gilles Caulier

patching file utilities/maintenance/imageinfojob.cpp
Reversed (or previously applied) patch detected!  Assume -R? [n]

Looks like this commit alreafy applied in digiKam 4.3.0 sources, so if crash fixed then maybe in other commit?
Comment 4 caulier.gilles 2014-09-19 21:15:33 UTC
Nucleao,

Look the date of commit :

ID: 30e65a812f5aff808173dad52100c93dcc24b988
Added by Teemu Rytilahti 4 days ago
Don't crash if there's an error during the job

4.3.0 tarball have been created well before...

Gilles Caulier
Comment 5 nucleo 2014-09-19 21:22:10 UTC
Here commit:

--- a/utilities/maintenance/imageinfojob.cpp
+++ b/utilities/maintenance/imageinfojob.cpp
@@ -110,8 +110,6 @@ bool ImageInfoJob::isRunning() const
 
 void ImageInfoJob::slotResult(KJob* job)
 {
-    d->job = 0;
-
     if (job->error())
     {
         kWarning() << "Failed to list url: " << job->errorString();
@@ -121,6 +119,8 @@ void ImageInfoJob::slotResult(KJob* job)
                              DigikamApp::instance(), DigikamApp::instance()->windowTitle());
     }
 
+    d->job = 0;
+
     emit signalCompleted();
 }
 


And here actual code from 4.3.0 tarball:

void ImageInfoJob::slotResult(KJob* job)
{
    if (job->error())
    {
        kWarning() << "Failed to list url: " << job->errorString();

        // Pop-up a message about the error.
        DNotificationWrapper(QString(), d->job->errorString(),
                             DigikamApp::instance(), DigikamApp::instance()->windowTitle());
        return;
    }

    d->job = 0;

    emit signalCompleted();
}

So patch already applied in 4.3.0 sources.
Comment 6 caulier.gilles 2014-09-19 21:26:00 UTC
Ahh...

This one is not include to 4.3.0 i think :

https://projects.kde.org/projects/extragear/graphics/digikam/repository/revisions/d523a9768ab8276860c76e02b9d9b598a3d607f9

Gilles Caulier
Comment 7 nucleo 2014-09-19 22:48:18 UTC
(In reply to Gilles Caulier from comment #6)
> Ahh...
> 
> This one is not include to 4.3.0 i think :
> 
> https://projects.kde.org/projects/extragear/graphics/digikam/repository/
> revisions/d523a9768ab8276860c76e02b9d9b598a3d607f9

This commit fixes crash in 4.3.0.
Thanks.