| Summary: | Kwin crashes on login. | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Sean <sean+kdebug> |
| Component: | general | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | hrvoje.senjan, ijont, jack.moody, martin, mcfloundinho, simonandric5, tek56rus |
| Priority: | NOR | Flags: | thomas.luebking:
ReviewRequest+
|
| Version: | 4.10.3 | ||
| Target Milestone: | --- | ||
| Platform: | Fedora RPMs | ||
| OS: | Linux | ||
| URL: | https://git.reviewboard.kde.org/r/110872/ | ||
| Latest Commit: | http://commits.kde.org/kde-workspace/7a1bab2410317195793c660d78933b858040a3f3 | Version Fixed In: | 4.11 |
| Sentry Crash Report: | |||
| Attachments: |
New crash information added by DrKonqi
Rebased patch |
||
|
Description
Sean
2013-06-01 03:17:55 UTC
is this reproducible and can you apply this patch? http://bugsfiles.kde.org/attachment.cgi?id=79795 Created attachment 80311 [details]
New crash information added by DrKonqi
kwin (4.10.60 "release 8") on KDE Platform 4.10.60 "release 8" using Qt 4.8.5
- What I was doing when the application crashed:
Happens only on 2nd and following logins, never had it after a "clean" reboot (though i don't reboot that often ;-)). Will try the patch - but not 100% reproducible here. Could be a Mesa bug(?)
-- Backtrace (Reduced):
#6 parent (this=0x111d1a0) at /usr/include/QtCore/qobject.h:273
#7 KWin::EffectWindow::addLayerRepaint (this=0x111d1a0, r=...) at /usr/src/debug/kde-workspace-git/kwin/libkwineffects/kwineffects.cpp:808
#8 0x00007f1e415c6405 in KWin::AnimationEffect::postPaintScreen (this=<optimized out>) at /usr/src/debug/kde-workspace-git/kwin/libkwineffects/kwinanimationeffect.cpp:588
#9 0x00007f1e41e8c0b2 in KWin::EffectsHandlerImpl::postPaintScreen (this=0x10f7890) at /usr/src/debug/kde-workspace-git/kwin/effects.cpp:394
#10 0x00007f1e41e695db in KWin::Scene::paintScreen (this=0xf46bb0, mask=<optimized out>, region=0x7fff9dd06fc0) at /usr/src/debug/kde-workspace-git/kwin/scene.cpp:145
Thanks If you can, please report back before Do 13. Jun 01:59:00 CEST 2013 : KDE SC 4.11 Beta 1 Tagging and Release Created attachment 80314 [details]
Rebased patch
@Thomas, will do ;-)
Attaching rebased patch for master to get a confirmation is it valid?
- Q_D(const AnimationEffect);
+ Q_D(AnimationEffect);
if (d->m_isInitialized)
return; // not more than once, please
+ d->m_isInitialized = true;
It's not correct in the original patch either (still it would bypass the original bug, just add maaaaany connections ;-)
(In reply to comment #5) > It's not correct in the original patch either (still it would bypass the > original bug, just add maaaaany connections ;-) OK, adjusted the patch, tried logging out/in a few times, crash did not yet happen. Will use it for a few days to confirm 100% *** Bug 319201 has been marked as a duplicate of this bug. *** *** Bug 315235 has been marked as a duplicate of this bug. *** Git commit 7a1bab2410317195793c660d78933b858040a3f3 by Thomas Lübking. Committed on 09/05/2013 at 17:39. Pushed by luebking into branch 'master'. ensure to init animationeffect bindings it seems an animation can be triggered and the resp. window deleted in the first event cycle (before the deleted signal is bound) - so we add an initialization flag to ensure the binding happens before the first animation is added FIXED-IN: 4.11 REVIEW: 110872 M +8 -2 kwin/libkwineffects/kwinanimationeffect.cpp http://commits.kde.org/kde-workspace/7a1bab2410317195793c660d78933b858040a3f3 *** Bug 321125 has been marked as a duplicate of this bug. *** *** Bug 323217 has been marked as a duplicate of this bug. *** *** Bug 326154 has been marked as a duplicate of this bug. *** *** Bug 326574 has been marked as a duplicate of this bug. *** *** Bug 327067 has been marked as a duplicate of this bug. *** |