diff options
| author | Peter Hutterer <peter.hutterer@who-t.net> | 2012-08-16 13:54:42 +1000 |
|---|---|---|
| committer | Peter Hutterer <peter.hutterer@who-t.net> | 2012-08-23 14:48:14 +1000 |
| commit | d53e6e02a2595ced1882f5fcd34d08ea039b3b85 (patch) | |
| tree | 8349e070c431f785cf0e76ce16d1e9c4e4fc331b /mi | |
| parent | bafbd99080be49a17be97d2cc758fbe623369945 (diff) | |
mi: don't check for core events in miPointerSetPosition (#53568)
As of 81cfe44b1ed0de84ad1941fe2ca74bebef3fc58d, miPointerSetPosition now
returns the screen pointer of the device. This broke floating slave devices,
as soon as a motion event was submitted, miPointerSetPosition returned NULL,
crashing the server.
dev->coreEvents is only false if the device is a floating slave, in which
case it has a sprite.
X.Org Bug 53568 <http://bugs.freedesktop.org/show_bug.cgi?id=53568>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Keith Packard <keithp@keithp.com>
Diffstat (limited to 'mi')
| -rw-r--r-- | mi/mipointer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mi/mipointer.c b/mi/mipointer.c index a56838ead..4defaf5ec 100644 --- a/mi/mipointer.c +++ b/mi/mipointer.c @@ -575,7 +575,7 @@ miPointerSetPosition(DeviceIntPtr pDev, int mode, double *screenx, miPointerPtr pPointer; - if (!pDev || !pDev->coreEvents) + if (!pDev) return NULL; pPointer = MIPOINTER(pDev); |