Summary: | Crash in QV4::QObjectWrapper::getProperty | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Unknown <null> |
Component: | generic-crash | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED UPSTREAM | ||
Severity: | crash | CC: | adamvnz, ajgrif, alanm, aramgrigoryan, aron.heinecke, awilfox, boruccia5, bugseforuns, bwat47, crazylegoguy, dabbs.drew, danilo.luvizotto, dariusz.debicki, david.decos, edwardb_kde, elman, erz96cxw, florianb3522, frederick888, gareth, grosser.meister.morti, hmcilvain1, i, ivan, jodr666, junglee.rakshas, kde, kde, klaussemmler, laszlo.stark, luizfreitasdepaula, mathias.ciliberto, michael59802, mo78, nate, notuxius, nviorres, ondreyn, oysstu, paulkd, piotr.mierzwinski, plasma-bugs, reisenweber, sad.dgs, sebastiansack, simonandric5, stanczakdominik, thecursedfly, toralf.foerster, travneff, weltqgel, yanp.bugz, yasin.unal.83 |
Priority: | NOR | Keywords: | drkonqi |
Version: | 5.10.4 | ||
Target Milestone: | 1.0 | ||
Platform: | Manjaro | ||
OS: | Linux | ||
URL: | https://codereview.qt-project.org/#/c/222250/ | ||
Latest Commit: | Version Fixed In: | Qt 5.11 | |
Attachments: | Qt patch that fixes the system tray example |
Description
Unknown
2017-08-22 06:43:56 UTC
*** Bug 383993 has been marked as a duplicate of this bug. *** *** Bug 384042 has been marked as a duplicate of this bug. *** *** Bug 384626 has been marked as a duplicate of this bug. *** *** Bug 384824 has been marked as a duplicate of this bug. *** *** Bug 385632 has been marked as a duplicate of this bug. *** [13:40] <kbroulik> to reproduce: 1.) open sys tray settings, go to "Entries", 2.) set a shortcut for a plasmoid from there, 3.) close systray settings, 4.) open systray settings again, 5.) now invoke the shortcut. boom. Will be the same cause as the QQmlBinding::getProperty crash. (https://bugreports.qt.io/browse/QTBUG-61681) We have an object, we put it between two objects. First engine closes, deletes the cache; just hitting v4 lookup not bindings *** Bug 386286 has been marked as a duplicate of this bug. *** *** Bug 382290 has been marked as a duplicate of this bug. *** *** Bug 386538 has been marked as a duplicate of this bug. *** *** Bug 385788 has been marked as a duplicate of this bug. *** *** Bug 386627 has been marked as a duplicate of this bug. *** *** Bug 386650 has been marked as a duplicate of this bug. *** *** Bug 387074 has been marked as a duplicate of this bug. *** *** Bug 372881 has been marked as a duplicate of this bug. *** *** Bug 387218 has been marked as a duplicate of this bug. *** *** Bug 387771 has been marked as a duplicate of this bug. *** *** Bug 387869 has been marked as a duplicate of this bug. *** *** Bug 388520 has been marked as a duplicate of this bug. *** *** Bug 388765 has been marked as a duplicate of this bug. *** *** Bug 389718 has been marked as a duplicate of this bug. *** *** Bug 389752 has been marked as a duplicate of this bug. *** *** Bug 389777 has been marked as a duplicate of this bug. *** *** Bug 389769 has been marked as a duplicate of this bug. *** *** Bug 389027 has been marked as a duplicate of this bug. *** *** Bug 389521 has been marked as a duplicate of this bug. *** Ok, this is getting a bit ridiculous now. Bug is effectively QTBUG-61681, continued fallout from: 749a7212e903d8e8c6f256edb1836b9449cc7fe1 but for functions instead of bindings. This happens when one QObject is wrapped between multiple engines, and then one engine is destroyed. The former is left with a dangling entry in it's property cache. I wrote a patch for bindings (ab5d4c78224c9ec79165e8890e5f8b8e838e0709), but we still see it (to a lesser extent) in functions. I can patch one more place in Qt, but I fear it'll just keep coming up all over the place. The other alternative approach is we use the shared the engine in the ConfigView. The only reason we don't is for this feature where one could use controls from the QQC1 import and have them automatically styled in Plasma whilst still having env var say to use the desktop style, so we see that in the config windows. But literally no-one does that and all import Plasma components explicitly; for QQC2 we're doing that on a per-context basis (I think?) Would be a oneliner in p-f (plus a tonne of testing it doesn't break any assumptions in old releases). @Marco, opinion? *** Bug 390125 has been marked as a duplicate of this bug. *** *** Bug 388681 has been marked as a duplicate of this bug. *** *** Bug 390008 has been marked as a duplicate of this bug. *** *** Bug 390885 has been marked as a duplicate of this bug. *** Created attachment 110942 [details]
Qt patch that fixes the system tray example
Qt patch that fixes this specific issue. Tested with the systray case, and verified with qDebug in my if statement that we went in here.
Personally I'd rather avoid the second engine, but maybe it's still worth me pushing this upstream. Can someone else test for some of the other causes?
*** Bug 392089 has been marked as a duplicate of this bug. *** *** Bug 392548 has been marked as a duplicate of this bug. *** *** Bug 392937 has been marked as a duplicate of this bug. *** *** Bug 392904 has been marked as a duplicate of this bug. *** *** Bug 392981 has been marked as a duplicate of this bug. *** *** Bug 393092 has been marked as a duplicate of this bug. *** *** Bug 386441 has been marked as a duplicate of this bug. *** *** Bug 393289 has been marked as a duplicate of this bug. *** *** Bug 393166 has been marked as a duplicate of this bug. *** *** Bug 391863 has been marked as a duplicate of this bug. *** *** Bug 393689 has been marked as a duplicate of this bug. *** *** Bug 393782 has been marked as a duplicate of this bug. *** *** Bug 394174 has been marked as a duplicate of this bug. *** *** Bug 394020 has been marked as a duplicate of this bug. *** *** Bug 394630 has been marked as a duplicate of this bug. *** *** Bug 394729 has been marked as a duplicate of this bug. *** *** Bug 395908 has been marked as a duplicate of this bug. *** *** Bug 396610 has been marked as a duplicate of this bug. *** *** Bug 397033 has been marked as a duplicate of this bug. *** *** Bug 395581 has been marked as a duplicate of this bug. *** *** Bug 397546 has been marked as a duplicate of this bug. *** *** Bug 398677 has been marked as a duplicate of this bug. *** *** Bug 405547 has been marked as a duplicate of this bug. *** *** Bug 409920 has been marked as a duplicate of this bug. *** *** Bug 412449 has been marked as a duplicate of this bug. *** *** Bug 404584 has been marked as a duplicate of this bug. *** |