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