Summary: | crash occured when removing tags | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | Adam Petty <adam> |
Component: | Tags-Engine | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | ajlcary, marcel.wiesweg, michael.hagmann, terri.bugs |
Priority: | NOR | ||
Version: | 1.0.0 | ||
Target Milestone: | --- | ||
Platform: | Unlisted Binaries | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 1.3.0 | |
Sentry Crash Report: |
Description
Adam Petty
2009-11-13 02:51:38 UTC
SVN commit 1048481 by mwiesweg: Ensure removed tags are not kept in map and later accessed when keeping a MetadataHub CCBUG: 214340 M +10 -0 digikam/metadatahub.cpp M +8 -0 digikam/metadatahub.h M +2 -1 libs/imageproperties/imagedescedittab.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1048481 Could you reproduce the problem? If yes, please try as soon rc1 is out if this bug is fixed for you. I dont know if I can reproduce it. ________________________________ From: Marcel Wiesweg <marcel.wiesweg@gmx.de> To: adam@akpetty.com Sent: Fri, November 13, 2009 7:34:46 AM Subject: [Bug 214340] crash occured when removing tags https://bugs.kde.org/show_bug.cgi?id=214340 Marcel Wiesweg <marcel.wiesweg@gmx.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |marcel.wiesweg@gmx.de --- Comment #2 from Marcel Wiesweg <marcel wiesweg gmx de> 2009-11-13 13:34:44 --- Could you reproduce the problem? If yes, please try as soon rc1 is out if this bug is fixed for you. digiKam 1.0.0 is out since few days... http://www.digikam.org/drupal/node/491 Please try with this version coming with more than 400 bug-fixes. Thanks in advance Gilles Caulier I'd assume this is fixed? *** Bug 223219 has been marked as a duplicate of this bug. *** With the new report for 1.0.0, it seems this bug is not fixed. I have no idea why. A metadatahub keeps a map of TAlbum pointers. For a good long-term solution, we need to get away from TAlbums*, they cannot be accessed from a different thread and cause these problems here. We probably need an additional cache of the tags tree, at a low level (AlbumDB, DatabaseAccess), and fully mutex protected. It needs to provide tag id -> tag path, tag path -> tag id, tag name -> all tag ids with that name. digiKam 1.1. release will be done in few days. Please check if this entry still valid. Thanks in advance Gilles Caulier *** Bug 226157 has been marked as a duplicate of this bug. *** *** Bug 227091 has been marked as a duplicate of this bug. *** SVN commit 1091832 by mwiesweg: For the time being (until there is a thread-safe tag cache at a low level) we must look after the MetadataHub and notify it when tags are removed. This code was already added for 1.0, got lost during the model/view merge, but we got crash reports for 1.0-beta5, 1.0.0, 1.1.0, and 1.2.0-svn. So I cannot be sure that this fully fixes 214340. CCBUG: 226504 CCBUG: 214340 M +17 -0 imagedescedittab.cpp M +2 -0 imagedescedittab.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1091832 digiKam and Kipi-plugins 1.2.0 are out. Please check if crash is still valid there. Thanks in advance Gilles Caulier In current SVN, the ImgDescEditTab has been ported to use the new TagsCache. No storing of TAlbum pointers anymore, this crash must be fixed. |