On Wed, 20 Mar 2013 14:36:25 +0200
Imre Deak <[email protected]> wrote:

> > +           pipe = -1;
> >  
> >             if (encoder->get_hw_state(encoder, &pipe)) {
> > -                   encoder->base.crtc =
> > -                           dev_priv->pipe_to_crtc_mapping[pipe];
> > -           } else {
> > +                   crtc = 
> > to_intel_crtc(dev_priv->pipe_to_crtc_mapping[pipe]);  
> 
> get_hw_state() can return true and still leave pipe at -1, causing an
> invalid access here.

Fixed, thanks.

-- 
Jesse Barnes, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to