| Summary: | Marble crashes on closing | ||
|---|---|---|---|
| Product: | [Applications] marble | Reporter: | Andreas Krohn <Hamburger1984> |
| Component: | general | Assignee: | marble-bugs |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | bugs.m1, ht990332 |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | 1.3 (KDE 4.8) | ||
| Platform: | Fedora RPMs | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | KDE-4.8.0 | |
| Sentry Crash Report: | |||
| Attachments: |
New crash information added by DrKonqi
New crash information added by DrKonqi |
||
|
Description
Andreas Krohn
2011-12-09 12:20:22 UTC
Git commit 4a1af05a822894d77bacac0932df6d38e74bcc10 by Dennis Nienhüser. Committed on 31/12/2011 at 10:55. Pushed by nienhueser into branch 'master'. Do not unload plugins in PluginManager dtor. According to qt documentation this is not needed. Not unloading plugins avoids timing issues at destruction time: Plugin related instances can be deleted after the deletion of MarbleModel now. Previously MarbleModel deleted the PluginManager, which unloaded the plugins and then created dangling pointers for plugin related instances not cleaned up at that time. Should be backported to the KDE/4.8 branch after testing. Related: bug 274558 M +20 -31 src/lib/PluginManager.cpp http://commits.kde.org/marble/4a1af05a822894d77bacac0932df6d38e74bcc10 Created attachment 67301 [details]
New crash information added by DrKonqi
marble (1.2.2 (stable release)) on KDE Platform 4.7.3 (4.7.3) using Qt 4.7.4
- What I was doing when the application crashed:
used marble, then closed marble and it crashed
-- Backtrace (Reduced):
#6 0x00007fb0ad3c8421 in notify_helper (e=0x5408600, receiver=0x45a2fb0, this=0x11079b0) at kernel/qapplication.cpp:4486
#7 QApplicationPrivate::notify_helper (this=0x11079b0, receiver=0x45a2fb0, e=0x5408600) at kernel/qapplication.cpp:4458
#8 0x00007fb0ad3cd291 in QApplication::notify (this=0x7ffff6384340, receiver=0x45a2fb0, e=0x5408600) at kernel/qapplication.cpp:4365
#9 0x00007fb0aea4d126 in KApplication::notify (this=0x7ffff6384340, receiver=0x45a2fb0, event=0x5408600) at ../../kdeui/kernel/kapplication.cpp:311
#10 0x00007fb0ae017afc in QCoreApplication::notifyInternal (this=0x7ffff6384340, receiver=0x45a2fb0, event=0x5408600) at kernel/qcoreapplication.cpp:787
Created attachment 67879 [details]
New crash information added by DrKonqi
marble (1.2.95 (1.3 Release Candidate 1)) on KDE Platform 4.7.98 (4.8 RC2+ (4.7.98) using Qt 4.8.0
- What I was doing when the application crashed:
marble 4.8rc2 crashes on closing the application under Qt 4.8
-- Backtrace (Reduced):
#7 0xb679f78d in QObjectPrivate::deleteChildren (this=0x8896f50) at kernel/qobject.cpp:1908
#8 0xb67a5f04 in QObject::~QObject (this=0x8896d40, __in_chrg=<optimized out>) at kernel/qobject.cpp:927
#9 0xb75afe57 in Marble::MarbleRunnerManager::~MarbleRunnerManager (this=0x8896d40, __in_chrg=<optimized out>) at /home/hussam/packages/archlinux/packages/kdeedu-marble/trunk/src/marble-4.7.97/src/lib/MarbleRunnerManager.cpp:136
#10 0xb75afe92 in Marble::MarbleRunnerManager::~MarbleRunnerManager (this=0x8896d40, __in_chrg=<optimized out>) at /home/hussam/packages/archlinux/packages/kdeedu-marble/trunk/src/marble-4.7.97/src/lib/MarbleRunnerManager.cpp:139
#11 0xb679f790 in QObjectPrivate::deleteChildren (this=0x8896db0) at kernel/qobject.cpp:1908
Git commit 460e57a0ab45eaf606635a52bf0e3f2df5183c8b by Dennis Nienhüser. Committed on 18/01/2012 at 20:37. Pushed by nienhueser into branch 'KDE/4.8'. Do not delete plugin instances. Fixes segfaults with multiple widgets. (cherry picked from commit 8f5d08a5d4dd8a78536327bd73ac170303655790) Related: bug 274558 FIXED-IN: KDE-4.8.0 M +5 -21 src/lib/PluginManager.cpp M +2 -0 src/plugins/runner/monav/MonavPlugin.cpp M +2 -0 src/plugins/runner/monav/MonavPlugin.h http://commits.kde.org/marble/460e57a0ab45eaf606635a52bf0e3f2df5183c8b Git commit 8c126cf90013bb231a96270d7574ef011b1c322a by Dennis Nienhüser. Committed on 31/12/2011 at 10:55. Pushed by nienhueser into branch 'KDE/4.8'. Do not unload plugins in PluginManager dtor. According to qt documentation this is not needed. Not unloading plugins avoids timing issues at destruction time: Plugin related instances can be deleted after the deletion of MarbleModel now. Previously MarbleModel deleted the PluginManager, which unloaded the plugins and then created dangling pointers for plugin related instances not cleaned up at that time. Should be backported to the KDE/4.8 branch after testing. Related: bug 274558 (cherry picked from commit 4a1af05a822894d77bacac0932df6d38e74bcc10) M +20 -31 src/lib/PluginManager.cpp http://commits.kde.org/marble/8c126cf90013bb231a96270d7574ef011b1c322a works for me - thanks! Git commit 1246d4e04dbdba4b4a3b6c00f2c105d79794c4b7 by Dennis Nienhüser. Committed on 31/12/2011 at 10:55. Pushed by nienhueser into branch 'Touch/1.3'. Do not unload plugins in PluginManager dtor. According to qt documentation this is not needed. Not unloading plugins avoids timing issues at destruction time: Plugin related instances can be deleted after the deletion of MarbleModel now. Previously MarbleModel deleted the PluginManager, which unloaded the plugins and then created dangling pointers for plugin related instances not cleaned up at that time. Should be backported to the KDE/4.8 branch after testing. Related: bug 274558 (cherry picked from commit 4a1af05a822894d77bacac0932df6d38e74bcc10) M +20 -31 src/lib/PluginManager.cpp http://commits.kde.org/marble/1246d4e04dbdba4b4a3b6c00f2c105d79794c4b7 Git commit b6d75a4e70bea2bf0313453f5d83b30a5f52deb9 by Dennis Nienhüser. Committed on 18/01/2012 at 20:37. Pushed by nienhueser into branch 'Touch/1.3'. Do not delete plugin instances. Fixes segfaults with multiple widgets. (cherry picked from commit 8f5d08a5d4dd8a78536327bd73ac170303655790) Related: bug 274558 FIXED-IN: KDE-4.8.0 M +5 -21 src/lib/PluginManager.cpp M +2 -0 src/plugins/runner/monav/MonavPlugin.cpp M +2 -0 src/plugins/runner/monav/MonavPlugin.h http://commits.kde.org/marble/b6d75a4e70bea2bf0313453f5d83b30a5f52deb9 |