Summary: | Carsh at Startup | ||
---|---|---|---|
Product: | Network Management | Reporter: | Chris <crglasoe> |
Component: | KDED Module | Assignee: | Lamarque V. Souza <lamarque> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | cfeck |
Priority: | NOR | ||
Version: | 0.9 | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/networkmanagement/75942e42032ca1d505ba4c7f77b3ff05a2865eac | Version Fixed In: | 0.9.0.6 |
Description
Chris
2012-10-12 13:27:33 UTC
Lamarque, the problem is probably with the qDeleteAll() change in Qt 4.8. Within the destructor, you must not remove yourself from the list. See also https://git.reviewboard.kde.org/r/102981/ Git commit 989ced86c8e14c69036fca5ce254d65990087c0a by Lamarque V. Souza. Committed on 12/10/2012 at 17:40. Pushed by lvsouza into branch 'nm09'. Fix another potential case of qDeleteAll() crash. M +5 -2 kded/nm08connections.cpp M +1 -1 plasma_nm_version.h http://commits.kde.org/networkmanagement/989ced86c8e14c69036fca5ce254d65990087c0a Git commit 3d95007a00e39d4a9ba814bc0f69428fba32a090 by Lamarque V. Souza. Committed on 12/10/2012 at 17:29. Pushed by lvsouza into branch 'nm09'. Fix crash when deleting connection. FIXED-IN: 0.9.0.6 M +12 -5 libs/internals/connection.cpp M +1 -0 libs/internals/connection.h http://commits.kde.org/networkmanagement/3d95007a00e39d4a9ba814bc0f69428fba32a090 Git commit 75942e42032ca1d505ba4c7f77b3ff05a2865eac by Lamarque V. Souza. Committed on 12/10/2012 at 17:41. Pushed by lvsouza into branch 'master'. Fix crash when deleting connection. FIXED-IN: 0.9.0.6 M +12 -5 libs/internals/connection.cpp M +1 -0 libs/internals/connection.h http://commits.kde.org/networkmanagement/75942e42032ca1d505ba4c7f77b3ff05a2865eac Git commit ae9e7b8ad72fbf1ce4e0093e1ff43ac0e7cd7478 by Lamarque V. Souza. Committed on 12/10/2012 at 17:42. Pushed by lvsouza into branch 'master'. Fix another potential case of qDeleteAll() crash. M +5 -2 kded/nm08connections.cpp M +1 -1 plasma_nm_version.h http://commits.kde.org/networkmanagement/ae9e7b8ad72fbf1ce4e0093e1ff43ac0e7cd7478 (In reply to comment #1) > Lamarque, the problem is probably with the qDeleteAll() change in Qt 4.8. > Within the destructor, you must not remove yourself from the list. See also > https://git.reviewboard.kde.org/r/102981/ Ok, thanks for the tip. Git commit 98819f62ef5a1a47d643cc76b506ea2b0e1310ed by David Faure. Committed on 13/10/2012 at 19:19. Pushed by dfaure into branch 'master'. Fix crash on kded4 startup Previous commit removed the clearning of the list, and didn't remove items from the list in the loop (-> double delete, and infinite loop) M +2 -1 libs/internals/connection.cpp http://commits.kde.org/networkmanagement/98819f62ef5a1a47d643cc76b506ea2b0e1310ed Git commit c06adc6a5f4e34d267f699a4cb4a04c9fdfb28d5 by Lamarque V. Souza. Committed on 13/10/2012 at 19:45. Pushed by lvsouza into branch 'master'. Fix crash on kded4 startup. M +1 -1 kded/nm08connections.cpp M +1 -1 plasma_nm_version.h http://commits.kde.org/networkmanagement/c06adc6a5f4e34d267f699a4cb4a04c9fdfb28d5 Git commit f35856aae1dd5b8b1a44038fc5427c1ed2bb7549 by Lamarque V. Souza. Committed on 13/10/2012 at 19:45. Pushed by lvsouza into branch 'nm09'. Fix crash on kded4 startup. M +1 -1 kded/nm08connections.cpp M +1 -1 plasma_nm_version.h http://commits.kde.org/networkmanagement/f35856aae1dd5b8b1a44038fc5427c1ed2bb7549 Git commit a9d5297996a977d8f2b78e78e63c9ecab9896adc by Lamarque V. Souza, on behalf of David Faure. Committed on 13/10/2012 at 19:19. Pushed by lvsouza into branch 'nm09'. Fix crash on kded4 startup Previous commit removed the clearning of the list, and didn't remove items from the list in the loop (-> double delete, and infinite loop) (cherry picked from commit 98819f62ef5a1a47d643cc76b506ea2b0e1310ed) M +2 -1 libs/internals/connection.cpp http://commits.kde.org/networkmanagement/a9d5297996a977d8f2b78e78e63c9ecab9896adc |