Summary: | Marble crashes when adding a bookmark and creating a new folder at the same time | ||
---|---|---|---|
Product: | [Applications] marble | Reporter: | flyos |
Component: | general | Assignee: | Dennis Nienhüser <nienhueser> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | alanjprescott, exellent, julienmeme-libre, leenux100, markus, martin.ruessler, nienhueser, rfringuello, v |
Priority: | HI | Keywords: | drkonqi |
Version: | 1.9 (KDE 4.14) | ||
Target Milestone: | --- | ||
Platform: | Kubuntu | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/marble/3d47be400fdb9ae91573700edd5671d58563ddc3 | Version Fixed In: | Applications/14.12.4 |
Sentry Crash Report: | |||
Attachments: |
New crash information added by DrKonqi
New crash information added by DrKonqi New crash information added by DrKonqi New crash information added by DrKonqi |
Description
flyos
2014-02-25 14:51:56 UTC
Also, I forgot to mention that I can't have any bookmark added or bookmark folder created whatsoever (though without any crash when I'm either creating a folder, or adding a bookmark. The crash happens only while doing both at the same time). This is a more minor bug, but I thought the detail could be relevant. Cheers ! Created attachment 85731 [details]
New crash information added by DrKonqi
marble (1.7.0 (stable version)) on KDE Platform 4.12.3 using Qt 4.8.4
- What I was doing when the application crashed:
Attempting to add a bookmark and create a new folder for it
-- Backtrace (Reduced):
#6 0x00007fcb16041cba in Marble::GeoDataTreeModel::index (this=this@entry=0x259b740, object=object@entry=0x37fa2b0) at /usr/src/debug/marble-4.12.3/src/lib/marble/GeoDataTreeModel.cpp:519
#7 0x00007fcb16042772 in addFeature (row=-1, feature=0x39ba080, parent=0x37fa2b0, this=0x259b740) at /usr/src/debug/marble-4.12.3/src/lib/marble/GeoDataTreeModel.cpp:570
#8 Marble::GeoDataTreeModel::addFeature (this=0x259b740, parent=0x37fa2b0, feature=0x39ba080, row=-1) at /usr/src/debug/marble-4.12.3/src/lib/marble/GeoDataTreeModel.cpp:566
#9 0x00007fcb1611d4ca in Marble::BookmarkManager::addBookmark (this=0x259b7c0, container=0x37fa2b0, placemark=...) at /usr/src/debug/marble-4.12.3/src/lib/marble/BookmarkManager.cpp:132
#10 0x00007fcb1603de77 in Marble::MarbleWidgetPopupMenu::addBookmark (this=0x2807be0) at /usr/src/debug/marble-4.12.3/src/lib/marble/MarbleWidgetPopupMenu.cpp:669
Created attachment 86373 [details]
New crash information added by DrKonqi
marble (1.7.97 (1.8 RC 1)) on KDE Platform 4.13.0 using Qt 4.8.5
- What I was doing when the application crashed: I was doing the same things like the other bug. A want add a new bookmark and I had required to create a new folder juste before approve the add.
I observed the impossibilité to create a new folder in the bookmark manager... after validate the folder name, many tags appeared... it is not the good context diplaid...
-- Backtrace (Reduced):
#6 0x00007f16186953d3 in Marble::GeoDataTreeModel::index (this=this@entry=0x1754a50, object=object@entry=0x33dce60) at /usr/src/debug/marble-4.13.0/src/lib/marble/GeoDataTreeModel.cpp:625
#7 0x00007f16186961af in Marble::GeoDataTreeModel::addFeature (this=0x1754a50, parent=parent@entry=0x33dce60, feature=feature@entry=0x33b3ea0, row=row@entry=-1) at /usr/src/debug/marble-4.13.0/src/lib/marble/GeoDataTreeModel.cpp:688
#8 0x00007f16187769ac in Marble::BookmarkManager::addBookmark (this=0x1754ad0, container=container@entry=0x33dce60, placemark=...) at /usr/src/debug/marble-4.13.0/src/lib/marble/BookmarkManager.cpp:137
#9 0x00007f1618692913 in Marble::MarbleWidgetPopupMenu::addBookmark (this=0x1a38300) at /usr/src/debug/marble-4.13.0/src/lib/marble/MarbleWidgetPopupMenu.cpp:424
[...]
#11 0x00007f161670f752 in QAction::triggered (this=this@entry=0x19170f0, _t1=false) at .moc/release-shared/moc_qaction.cpp:276
Created attachment 87671 [details]
New crash information added by DrKonqi
marble (1.7.97 (1.8 RC 1)) on KDE Platform 4.13.1 using Qt 4.8.6
- What I was doing when the application crashed:
Add bookmark and create the category-folder.
Debian package
-- Backtrace (Reduced):
#6 0x00007f8595215483 in Marble::GeoDataTreeModel::index(Marble::GeoDataObject*) () from /usr/lib/libmarblewidget.so.18
#7 0x00007f859521624f in Marble::GeoDataTreeModel::addFeature(Marble::GeoDataContainer*, Marble::GeoDataFeature*, int) () from /usr/lib/libmarblewidget.so.18
#8 0x00007f85952f66ac in Marble::BookmarkManager::addBookmark(Marble::GeoDataContainer*, Marble::GeoDataPlacemark const&) () from /usr/lib/libmarblewidget.so.18
#9 0x00007f85952129e3 in Marble::MarbleWidgetPopupMenu::addBookmark() () from /usr/lib/libmarblewidget.so.18
[...]
#11 0x00007f8593242912 in QAction::triggered (this=this@entry=0x2d1fd50, _t1=false) at .moc/release-shared/moc_qaction.cpp:276
Created attachment 88338 [details]
New crash information added by DrKonqi
marble (1.8.20 (1.9 Alpha)) on KDE Platform 4.13.97 using Qt 4.8.6
- What I was doing when the application crashed:
This crash is still prominent, updating bug information.
-- Backtrace (Reduced):
#6 0x00007f271bbdb4c3 in Marble::GeoDataTreeModel::index (this=this@entry=0xd4f940, object=object@entry=0x4068560) at ../../../../src/lib/marble/GeoDataTreeModel.cpp:733
#7 0x00007f271bbdc29f in Marble::GeoDataTreeModel::addFeature (this=0xd4f940, parent=parent@entry=0x4068560, feature=feature@entry=0x4113c10, row=row@entry=-1) at ../../../../src/lib/marble/GeoDataTreeModel.cpp:796
#8 0x00007f271bcbe1cc in Marble::BookmarkManager::addBookmark (this=0xd4f9c0, container=container@entry=0x4068560, placemark=...) at ../../../../src/lib/marble/BookmarkManager.cpp:137
#9 0x00007f271bbd88e3 in Marble::MarbleWidgetPopupMenu::addBookmark (this=0x1323c00) at ../../../../src/lib/marble/MarbleWidgetPopupMenu.cpp:643
[...]
#11 0x00007f2719c47a62 in QAction::triggered (this=this@entry=0xc911f0, _t1=false) at .moc/release-shared/moc_qaction.cpp:276
*** Bug 338189 has been marked as a duplicate of this bug. *** *** Bug 344880 has been marked as a duplicate of this bug. *** Found the culprit - bookmark sync is active even if it shouldn't, and buggy on top, leading to the crash. *** Bug 333188 has been marked as a duplicate of this bug. *** Git commit 15e1c73f14ea7e8c435789d73b87b7c8caf7deb3 by Dennis Nienhüser. Committed on 06/04/2015 at 13:45. Pushed by nienhueser into branch 'master'. Disable bookmark sync if cloud sync is disabled M +1 -1 src/lib/marble/cloudsync/BookmarkSyncManager.cpp http://commits.kde.org/marble/15e1c73f14ea7e8c435789d73b87b7c8caf7deb3 Git commit b83027de6f80ec41ba5e123306ff905241b8b607 by Dennis Nienhüser. Committed on 06/04/2015 at 13:45. Pushed by nienhueser into branch 'master'. No need to reload bookmarks after copying them to the cache folder As this code section is called after bookmarks are changed in memory, there was a race condition between writing changed bookmarks to disk and reloading them here, often leading to inconsistencies or crashes. M +0 -4 src/lib/marble/cloudsync/BookmarkSyncManager.cpp http://commits.kde.org/marble/b83027de6f80ec41ba5e123306ff905241b8b607 Git commit ba64a37c5c7d26d037d083f54af522456ab042a8 by Dennis Nienhüser. Committed on 06/04/2015 at 13:47. Pushed by nienhueser into branch 'Applications/15.04'. Disable bookmark sync if cloud sync is disabled (cherry picked from commit 15e1c73f14ea7e8c435789d73b87b7c8caf7deb3) M +1 -1 src/lib/marble/cloudsync/BookmarkSyncManager.cpp http://commits.kde.org/marble/ba64a37c5c7d26d037d083f54af522456ab042a8 Git commit 7827de70d20c3b82ec39becf45d789902d7bf606 by Dennis Nienhüser. Committed on 06/04/2015 at 13:47. Pushed by nienhueser into branch 'Applications/15.04'. No need to reload bookmarks after copying them to the cache folder As this code section is called after bookmarks are changed in memory, there was a race condition between writing changed bookmarks to disk and reloading them here, often leading to inconsistencies or crashes. (cherry picked from commit b83027de6f80ec41ba5e123306ff905241b8b607) M +0 -4 src/lib/marble/cloudsync/BookmarkSyncManager.cpp http://commits.kde.org/marble/7827de70d20c3b82ec39becf45d789902d7bf606 Git commit dde065612fb658d05c198f3ab5ad8a76feb02f7c by Dennis Nienhüser. Committed on 06/04/2015 at 13:51. Pushed by nienhueser into branch 'Applications/14.12'. Disable bookmark sync if cloud sync is disabled (cherry picked from commit 15e1c73f14ea7e8c435789d73b87b7c8caf7deb3) (cherry picked from commit ba64a37c5c7d26d037d083f54af522456ab042a8) M +1 -1 src/lib/marble/cloudsync/BookmarkSyncManager.cpp http://commits.kde.org/marble/dde065612fb658d05c198f3ab5ad8a76feb02f7c Git commit 3d47be400fdb9ae91573700edd5671d58563ddc3 by Dennis Nienhüser. Committed on 06/04/2015 at 13:52. Pushed by nienhueser into branch 'Applications/14.12'. No need to reload bookmarks after copying them to the cache folder As this code section is called after bookmarks are changed in memory, there was a race condition between writing changed bookmarks to disk and reloading them here, often leading to inconsistencies or crashes. (cherry picked from commit b83027de6f80ec41ba5e123306ff905241b8b607) (cherry picked from commit 7827de70d20c3b82ec39becf45d789902d7bf606) M +0 -4 src/lib/marble/cloudsync/BookmarkSyncManager.cpp http://commits.kde.org/marble/3d47be400fdb9ae91573700edd5671d58563ddc3 *** Bug 350857 has been marked as a duplicate of this bug. *** |