summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Huddleston <jeremyhu@freedesktop.org>2009-01-11 16:10:45 -0800
committerJeremy Huddleston <jeremyhu@freedesktop.org>2009-01-11 16:10:45 -0800
commit3c596c061e75848cfa76dd9259c23a3f3a67444c (patch)
tree4946877aff9ed4c2b9250ab8a852333833c209d8
parent6461729647ff4441d80811e73f0c0d2f108f2700 (diff)
mieq: Avoid possible race condition whereby one thread might call mieqEnqueue before InitAndStartDevices finishes
This is more of a hack around the problem. This is something that will need to be addressed in a more structured manner with the multi threaded input efforts.
-rw-r--r--mi/mieq.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/mi/mieq.c b/mi/mieq.c
index 2905c7782..a64cb8740 100644
--- a/mi/mieq.c
+++ b/mi/mieq.c
@@ -248,7 +248,14 @@ mieqEnqueue(DeviceIntPtr pDev, xEvent *e)
evt->event->u.keyButtonPointer.time = miEventQueue.lastEventTime;
miEventQueue.lastEventTime = evt->event->u.keyButtonPointer.time;
- miEventQueue.events[oldtail].pScreen = EnqueueScreen(pDev);
+
+ /* Avoid possible crash when multithreaded and mieqEnqueue is called before
+ * InitAndStartDevices finishes.
+ */
+ if(pDev && pDev->spriteInfo && pDev->spriteInfo->sprite)
+ miEventQueue.events[oldtail].pScreen = EnqueueScreen(pDev);
+ else
+ miEventQueue.events[oldtail].pScreen = NULL;
miEventQueue.events[oldtail].pDev = pDev;
miEventQueue.lastMotion = isMotion;
@@ -399,7 +406,7 @@ mieqProcessInputEvents(void)
/* Custom event handler */
handler = miEventQueue.handlers[type];
- if (screen != DequeueScreen(dev) && !handler) {
+ if (screen && screen != DequeueScreen(dev) && !handler) {
/* Assumption - screen switching can only occur on motion events. */
DequeueScreen(dev) = screen;
x = event->u.keyButtonPointer.rootX;