Summary: | Marble crashes on start [BookmarkManager::updateBookmarkFile, GeoWriter::writeElement] | ||
---|---|---|---|
Product: | [Applications] marble | Reporter: | Jonathan Marten <jjm> |
Component: | general | Assignee: | Dennis Nienhüser <nienhueser> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | adaptee, nienhueser, utkuaydin34 |
Priority: | HI | Keywords: | drkonqi |
Version: | 1.7 (KDE 4.12) | ||
Target Milestone: | 1.7 (KDE 4.12) | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/marble/7603ee9e386ed7a6e638bdf5cf7a450b04a752b4 | Version Fixed In: | |
Sentry Crash Report: |
Description
Jonathan Marten
2013-11-11 11:52:25 UTC
Git commit 1c0fa695cd6ec7884dcfef1f0d27e58b1cb9c7cf by Dennis Nienhüser. Committed on 13/11/2013 at 17:49. Pushed by nienhueser into branch 'master'. Respect general sync activation status M +4 -4 src/lib/marble/cloudsync/BookmarkSyncManager.cpp M +1 -1 src/lib/marble/cloudsync/RouteSyncManager.cpp http://commits.kde.org/marble/1c0fa695cd6ec7884dcfef1f0d27e58b1cb9c7cf Note that the commit above just fixes the trigger of the crash, not the cause. Git commit ef4e3fd9d667f82471cd8555408fc492619c5626 by Dennis Nienhüser. Committed on 13/11/2013 at 17:49. Pushed by nienhueser into branch 'KDE/4.12'. Respect general sync activation status (cherry picked from commit 1c0fa695cd6ec7884dcfef1f0d27e58b1cb9c7cf) M +4 -4 src/lib/marble/cloudsync/BookmarkSyncManager.cpp M +1 -1 src/lib/marble/cloudsync/RouteSyncManager.cpp http://commits.kde.org/marble/ef4e3fd9d667f82471cd8555408fc492619c5626 Git commit 7603ee9e386ed7a6e638bdf5cf7a450b04a752b4 by Dennis Nienhüser. Committed on 13/11/2013 at 18:19. Pushed by nienhueser into branch 'KDE/4.12'. Fix regression introduced in 9e630ca Parsing a broken/non-existing KML document returns a null GeoDocument, so we need to recreate a bookmark document in order to prevent crashes later on. M +4 -2 src/lib/marble/BookmarkManager.cpp http://commits.kde.org/marble/7603ee9e386ed7a6e638bdf5cf7a450b04a752b4 Git commit eafac1612e17b9935b1d6dfc533ad155e3a06069 by Dennis Nienhüser. Committed on 13/11/2013 at 18:19. Pushed by nienhueser into branch 'master'. Fix regression introduced in 9e630ca Parsing a broken/non-existing KML document returns a null GeoDocument, so we need to recreate a bookmark document in order to prevent crashes later on. (cherry picked from commit 7603ee9e386ed7a6e638bdf5cf7a450b04a752b4) M +4 -2 src/lib/marble/BookmarkManager.cpp http://commits.kde.org/marble/eafac1612e17b9935b1d6dfc533ad155e3a06069 Confirmed fixed with current trunk master branch, with the bookmarks file not existing at startup. No crash, Marble works as expected. Many thanks for the quick fix! Git commit e9e94dfd95cdf96a49b351cf60707d042da56d7e by Dennis Nienhüser. Committed on 13/11/2013 at 18:19. Pushed by nienhueser into branch 'KDE/4.11'. Fix regression introduced in 9e630ca Parsing a broken/non-existing KML document returns a null GeoDocument, so we need to recreate a bookmark document in order to prevent crashes later on. Related: bug 328221 FIXED-IN: 4.11.5 (cherry picked from commit 7603ee9e386ed7a6e638bdf5cf7a450b04a752b4) M +4 -2 src/lib/BookmarkManager.cpp http://commits.kde.org/marble/e9e94dfd95cdf96a49b351cf60707d042da56d7e |