Summary: | xembedsniproxy crashes when clicking on empty fcitx icon | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Wolfgang Bauer <wbauer1> |
Component: | XembedSNIProxy | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kde, pulfer, rdieter, sudhir |
Priority: | NOR | ||
Version: | 5.5.3 | ||
Target Milestone: | 1.0 | ||
Platform: | openSUSE | ||
OS: | Linux | ||
URL: | https://bugzilla.opensuse.org/show_bug.cgi?id=954623 | ||
Latest Commit: | Version Fixed In: | ||
Bug Depends on: | 358227 | ||
Bug Blocks: |
Description
Wolfgang Bauer
2016-01-21 16:17:23 UTC
It seems the original fix was not enough. (In reply to Wolfgang Bauer from comment #0) > #0 0x000000000040eaf8 in SNIProxy::sendClick(unsigned char, int, int) > (this=0x62a6e0, mouseButton=mouseButton@entry=1 '\001', x=1008, y=998) > at > /usr/src/debug/plasma-workspace-5.5.3/xembed-sni-proxy/sniproxy.cpp:438 This line still uses clientGeom without checking for null. If clientGeom was null in SNIProxy::SNIProxy() (which caused the original crash, that was fixed by adding a check in line 163), it will also be null here. Urgh, thanks. Fixed with 988a7bef7e84a4a2fc6a2f6ec6145e093dfb84f8 *** Bug 358546 has been marked as a duplicate of this bug. *** (In reply to David Edmundson from comment #3) > Fixed with 988a7bef7e84a4a2fc6a2f6ec6145e093dfb84f8 Perhaps this commit should be pushed to master branch as well? It's still only in Plasma/5.5. |