[PATCH] VidMode: prevent crash with no modes

Chris Wilson chris at chris-wilson.co.uk
Tue Jan 25 04:14:21 PST 2011


On Tue, 25 Jan 2011 13:31:55 +0200, Pauli Nieminen <ext-pauli.nieminen at nokia.com> wrote:
> >      pScrn = xf86Screens[scrnIndex];
> > +    if (pScrn->modes == NULL)
> > +	    return FALSE;
> > +
> 
> Should we check for missing pScrn->modes in VidModeAvailable?

I interpreted that as 'is the VidMode extension enabled for this screen?'
Which is obviously a separate question from whether there are any modes
available.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the xorg-devel mailing list