diff options
| author | Keith Packard <keithp@neko.keithp.com> | 2007-02-19 15:28:37 -0800 |
|---|---|---|
| committer | Keith Packard <keithp@neko.keithp.com> | 2007-02-19 15:28:37 -0800 |
| commit | 8a42af6a935b1cf0e15102e986bb527f4fab31a8 (patch) | |
| tree | 47e7b514da212dbe2f4571f9d2797829fbd3e038 /xfixes/cursor.c | |
| parent | 4660eaaffb36f526f71d5847ae1309c10ee133c6 (diff) | |
Check for clientGone before sending events from XFixes (bug #1753).
Freeing resources during client closedown can cause cursor changes which
attempt to send cursor events through the XFixes extension; a client in the
process of closing down has no file to send events to, causing a crash when
this path is hit.
Diffstat (limited to 'xfixes/cursor.c')
| -rwxr-xr-x | xfixes/cursor.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xfixes/cursor.c b/xfixes/cursor.c index 86a512c25..3cdacc0e4 100755 --- a/xfixes/cursor.c +++ b/xfixes/cursor.c @@ -143,7 +143,8 @@ CursorDisplayCursor (ScreenPtr pScreen, CursorCurrent = pCursor; for (e = cursorEvents; e; e = e->next) { - if (e->eventMask & XFixesDisplayCursorNotifyMask) + if ((e->eventMask & XFixesDisplayCursorNotifyMask) && + !e->pClient->clientGone) { xXFixesCursorNotifyEvent ev; ev.type = XFixesEventBase + XFixesCursorNotify; |