Bug 5151 - Segfault in xf86Cursor.c due to uninitialized reference screen
Summary: Segfault in xf86Cursor.c due to uninitialized reference screen
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: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-11-24 23:32 UTC by Matthias Hopf
Modified: 2007-03-15 09:49 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
Fix for segfault (3.47 KB, patch)
2006-08-16 09:16 UTC, Matthias Hopf 		no flags Details | Splinter Review
View All

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.