Bug 5151

Summary: Segfault in xf86Cursor.c due to uninitialized reference screen
Product: xorg Reporter: Matthias Hopf <mat>
Component: Server/GeneralAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: high    
Version: git   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
Fix for segfault none

Description Matthias Hopf 2005-11-24 23:32:16 UTC 
I found this with the i810 driver, but AFAICS it could occure with any driver.
If a xinerama configuration references a screen that is not correctly configured
(e.g. because the hardware coudn't be found) the xserver segfaults in xf86Cursor
in Line 618ff due to access of xf86Screens[ref]->pScreen with xf86Screens[ref]
== NULL.

This should create an error message instead.
Comment 1 Matthias Hopf 2006-08-16 09:16:02 UTC 
Created attachment 6584 [details] [review]
Fix for segfault

This patch hopefully fixes the segfault.

Will apply soon if noone objects.
Comment 2 Daniel Stone 2007-02-27 01:28:51 UTC 
Sorry about the phenomenal bug spam, guys.  Adding xorg-team@ to the QA contact so bugs don't get lost in future.
Comment 3 Matthias Hopf 2007-03-15 09:49:38 UTC 
This had already been committed as 73e58adda96c1d1b5176d819107faa7697c3eb94 on Wed Aug 16 2006.

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.