Bug 8925 - sprite.screen is NULL unless Xinerama is running
Summary: sprite.screen is NULL unless Xinerama is running
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: git
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Daniel Stone
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-11-07 01:00 UTC by Keith Packard
Modified: 2006-11-30 16:37 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments

Description Keith Packard 2006-11-07 01:00:23 UTC 
You can't use sprite.screen in SyntheticMotion as it is NULL unless Xinerama is
running.
Comment 1 Daniel Stone 2006-11-07 01:09:54 UTC 
#ifdef PANORAMIX
[...]
#define SyntheticMotion(x, y) \
    PostSyntheticMotion(x, y, sprite.screen->myNum, \
                        syncEvents.playingEvents ? \
                          syncEvents.time.milliseconds : \
                          currentTime.milliseconds);
[...]
#endif

i don't use xinerama on any of my sistems, so this would've bit me pretty early on.
Comment 2 Daniel Stone 2006-11-07 01:14:33 UTC 
sorry, i'm being really special today.  fixed in git.
Comment 3 Keith Packard 2006-11-07 01:30:16 UTC 
The problem occurs when PANORAMIX is defined but the DIX code is unused, in this
case sprite.screen is NULL and yet SyntheticMotion dereferences it. I've pushed
a fix to the randr-1.2 branch, including defining this new external function in
dixevents.h
Comment 4 Daniel Stone 2006-11-30 16:37:29 UTC 
already been fixed

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.