On Wed, 20 Mar 2013 14:36:25 +0200
Imre Deak <imre.deak at intel.com> 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
