Bug 221569 - Crash when closing showfoto while a raw image is loading
Summary: Crash when closing showfoto while a raw image is loading
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Showfoto-RawProcessing (show other bugs)
Version: 1.1.0
Platform: Arch Linux Unspecified
: NOR crash
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-06 20:43 UTC by Johannes Wienke
Modified: 2022-02-05 04:55 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 1.1.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Johannes Wienke 2010-01-06 20:43:38 UTC
Version:           trunk 1070690 (using KDE 4.3.4)
Installed from:    Archlinux Packages

Showfoto crashes when closing it while a raw file is opened (demosaiced etc.)

#0  0x00007ffff5360eb9 in QMutex::lock() () from /usr/lib/libQtCore.so.4
#1  0x00007ffff79784f4 in CacheLock (this=0x7fffe44eaa40, cache=0x84c7b0) at /home/languitar/workspace/digiKam/libs/threadimageio/loadingcache.cpp:456
#2  0x00007ffff797f499 in Digikam::SharedLoadingTask::execute (this=0x95d850) at /home/languitar/workspace/digiKam/libs/threadimageio/loadsavetask.cpp:198
#3  0x00007ffff796f456 in Digikam::LoadSaveThread::run (this=0x84cc20) at /home/languitar/workspace/digiKam/libs/threadimageio/loadsavethread.cpp:134
#4  0x00007ffff5365ba5 in ?? () from /usr/lib/libQtCore.so.4
#5  0x00007ffff50f681a in start_thread () from /lib/libpthread.so.0
#6  0x00007ffff35cbc7d in clone () from /lib/libc.so.6
#7  0x0000000000000000 in ?? ()

I suspect, that in line 198 of loadsavetask the static cache instance was already deleted (cleanup method) from a destructor call or something like that but I don't have an idea how to prevent this from happening.
Comment 1 Marcel Wiesweg 2010-01-07 18:54:22 UTC
All loading must be stopped before LoadingCacheInterface::cleanUp() is called in the ShowFoto destructor. Normally from EditorWindow's destructor, but ShowFoto is an EditorWindow...what about delete m_canvas;m_canvas=0 before cleaning up the cache?
Comment 2 Johannes Wienke 2010-01-07 20:54:15 UTC
I don't know, I've never seen this part of digikam before. ;)
Comment 3 Marcel Wiesweg 2010-01-21 17:46:19 UTC
SVN commit 1078146 by mwiesweg:

Delete canvas (normally done in EditorWindow destructor) before shutting down loading cache.
Dont get a crash here now.

BUG: 221569

 M  +3 -0      showfoto.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1078146