Summary: | Xorg server crashes in TryClientEvents() | ||
---|---|---|---|
Product: | xorg | Reporter: | Jay Cotton <jay.cotton> |
Component: | Server/General | Assignee: | Peter Hutterer <peter.hutterer> |
Status: | RESOLVED FIXED | QA Contact: | Xorg Project Team <xorg-team> |
Severity: | normal | ||
Priority: | medium | CC: | peter.hutterer, przanoni |
Version: | unspecified | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Jay Cotton
2009-10-05 16:17:08 UTC
This patch was added to catch the segv. pEvents is the first pointer reference in TryClientEvents and the offset is small. --- events.c.orig Mon Oct 5 12:02:59 2009 +++ events.c Mon Oct 5 12:02:30 2009 @@ -1889,6 +1889,9 @@ { if (grab && !SameClient(grab, client)) return -1; /* don't send, but notify caller */ + + if(!pEvents) /* guard for segv */ + return -1; type = pEvents->u.u.type; if (type == MotionNotify) { please try to find a testcase where this occurs. Just fixing up and "ignoring" the crash isn't enough, we need to know if this has wider repercussions. From the backtrace I can see that it's caused by a key event, that's about as much info as I can tell. Does it happen with the same application each time? any updates on this? Do you still see this issue Jay? Claiming this is fixed since I haven't seen this in a while. Please reopen if it's still here |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.