| Summary: | KWin crashed after a Konqueror crash | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Aurelien Gateau <agateau> |
| Component: | general | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | 1lukas1, amantia, arvidjaar, bartek, bigras.bruno, bugzilla, jbrsubscribe, julroy67, kde, kde, kourtzis, m.sledziona, marcus, markm999, markotahal, mitunin.dmitry, null, out.there, pascal, RunebladeOmega, t.kijas, vo.zaeb, wittemar, xtekhne |
| Priority: | NOR | ||
| Version First Reported In: | git master | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: | New crash information added by DrKonqi | ||
SVN commit 1056931 by graesslin: Crash less BUG: 216908 M +1 -1 scene.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1056931 Shouldn't this also fix the crashes that happen in buildQuads (scene.cpp 485) from bug 216609 and duplicates? (In reply to comment #2) > Shouldn't this also fix the crashes that happen in buildQuads (scene.cpp 485) > from bug 216609 and duplicates? Yes indeed they are the same. Thanks. (I think I have to reshuffle some duplicates) *** Bug 217831 has been marked as a duplicate of this bug. *** *** Bug 217761 has been marked as a duplicate of this bug. *** *** Bug 217671 has been marked as a duplicate of this bug. *** *** Bug 217552 has been marked as a duplicate of this bug. *** *** Bug 217461 has been marked as a duplicate of this bug. *** *** Bug 216813 has been marked as a duplicate of this bug. *** *** Bug 216609 has been marked as a duplicate of this bug. *** *** Bug 217817 has been marked as a duplicate of this bug. *** *** Bug 217945 has been marked as a duplicate of this bug. *** *** Bug 217979 has been marked as a duplicate of this bug. *** *** Bug 218150 has been marked as a duplicate of this bug. *** *** Bug 218370 has been marked as a duplicate of this bug. *** *** Bug 218405 has been marked as a duplicate of this bug. *** *** Bug 218621 has been marked as a duplicate of this bug. *** Created attachment 39042 [details]
New crash information added by DrKonqi
*** Bug 218761 has been marked as a duplicate of this bug. *** *** Bug 218850 has been marked as a duplicate of this bug. *** *** Bug 218851 has been marked as a duplicate of this bug. *** *** Bug 218936 has been marked as a duplicate of this bug. *** *** Bug 218942 has been marked as a duplicate of this bug. *** *** Bug 219071 has been marked as a duplicate of this bug. *** *** Bug 219255 has been marked as a duplicate of this bug. *** *** Bug 219364 has been marked as a duplicate of this bug. *** *** Bug 219565 has been marked as a duplicate of this bug. *** *** Bug 219607 has been marked as a duplicate of this bug. *** *** Bug 220038 has been marked as a duplicate of this bug. *** *** Bug 220369 has been marked as a duplicate of this bug. *** *** Bug 220633 has been marked as a duplicate of this bug. *** *** Bug 220810 has been marked as a duplicate of this bug. *** *** Bug 222040 has been marked as a duplicate of this bug. *** *** Bug 223061 has been marked as a duplicate of this bug. *** *** Bug 223178 has been marked as a duplicate of this bug. *** |
KWin crashed right after a Konqueror crash. For some reason the backtrace got lost by the report tool, but I was able to read it and it pointed to the Scene::Window::buildQuads() method from kdebase/workspace/kwin/scene.cpp, line 487. Looking at the code I can see a dynamic cast which is not tested: ---- Client *client = dynamic_cast<Client*>( toplevel ); QRegion contents = clientShape(); QRegion center = client->transparentRect(); // <- Crash QRegion decoration = (client && Workspace::self()->decorationHasAlpha() ? QRegion(client->decorationRect()) : shape()) - center; ---- If the dynamic_cast fails, client->transparentRect() crashes. Further code does test for the validity of client, so it can probably be fixed. I do not know the code enough to do it myself, though.