vesa driver reports many modes, but only allows a few

[Daniel Stone]

On Thu, Mar 04, 2010 at 07:52:02AM -0600, Pat Kane wrote:
>   Ross Boylan <ross at biostat.ucsf.edu> wrote:
>    > Program received signal SIGSEGV, Segmentation fault.
>    > 0x0000000000000000 in ?? ()
>    > (gdb) where
>    > #0  0x0000000000000000 in ?? ()
>    > #1  0x000000000051a72a in DoConfigure ()
>    > at ../../../../hw/xfree86/common/xf86Configure.c:832
> 
> The traceback says that someone branched to zero at
> line 832 of hw/xfree86/common/xf86Configure.c, that
> section of code (from git master) looks like this:
> 
> 	ConfiguredMonitor = NULL;
> 
> 	xf86EnableAccess(xf86Screens[dev2screen[j]]);
> 	if ((*xf86Screens[dev2screen[j]]->PreInit)(xf86Screens[dev2screen[j]],
> 832:						 PROBE_DETECT) &&
> 	    ConfiguredMonitor) {
> 	    MonitorPtr = configureDDCMonitorSection(j);
> 	} else {
> 	    MonitorPtr = configureMonitorSection(j);
> 	}
> 
> So it appears that the expression:
>     (*xf86Screens[dev2screen[j]]->PreInit)
> is NULL;  it would be a good idea for that code to check
> if PreInit has been filled in before jumping through it.
> 
> Can you recompile the X server with a test for NULL?

Er, if pScrn->PreInit is NULL, then something's gone quite seriously
wrong and we're screwed anyway.

Cheers,
Daniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg/attachments/20100304/6f3d7ec8/attachment.pgp>