Summary: | Crash during icons change | ||
---|---|---|---|
Product: | [Plasma] Oxygen | Reporter: | retired <pepko94> |
Component: | style | Assignee: | Hugo Pereira Da Costa <hugo.pereira.da.costa> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | apcomptec, hugo.pereira.da.costa |
Priority: | NOR | ||
Version First Reported In: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
retired
2012-08-05 17:58:25 UTC
*** Bug 306389 has been marked as a duplicate of this bug. *** Git commit e6597b73e9840f6398021c1970d1c0409f87dbe5 by Hugo Pereira Da Costa. Committed on 07/09/2012 at 15:58. Pushed by hpereiradacosta into branch 'KDE/4.9'. Check transitionWidget validity before setting duration. M +4 -1 kstyles/oxygen/transitions/oxygentransitiondata.h http://commits.kde.org/kde-workspace/e6597b73e9840f6398021c1970d1c0409f87dbe5 Git commit 1789b5f16c3398e7b2eb99a0aad9c5b7347b7a1c by Hugo Pereira Da Costa. Committed on 07/09/2012 at 15:58. Pushed by hpereiradacosta into branch 'master'. Check transitionWidget validity before setting duration. M +4 -1 kstyles/oxygen/transitions/oxygentransitiondata.h http://commits.kde.org/kde-workspace/1789b5f16c3398e7b2eb99a0aad9c5b7347b7a1c Patch above (pushed both to master and KDE/4.9) should fix the crash. I could not reproduce it (I guess its the result of some race condition), but the patch makes sense, and the check on transitionWidget should be done there, as done elsewhere in the same code (especially since a QWeakPointer is used). I guess one should leave the bug open for a while, though, to give a chance to people reporting to test. Since nobody closed, I will ;) |