On Wed, Jan 09, 2013 at 01:14:54PM +1000, Dave Airlie wrote: > From: Dave Airlie <[email protected]> > > The previous fix for the previous fix, didn't fully work, > > If we don't set compat_output we end up doing derferences > of arrays with -1, leading to valgrind warnings. > > Signed-off-by: Dave Airlie <[email protected]>
Reviewed-by: Peter Hutterer <[email protected]> Cheers, Peter > --- > hw/xfree86/modes/xf86Crtc.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/hw/xfree86/modes/xf86Crtc.c b/hw/xfree86/modes/xf86Crtc.c > index 77a0218..45764fc 100644 > --- a/hw/xfree86/modes/xf86Crtc.c > +++ b/hw/xfree86/modes/xf86Crtc.c > @@ -1848,8 +1848,10 @@ SetCompatOutput(xf86CrtcConfigPtr config) > } > > /* All outputs are disconnected, select one to fake */ > - if (!output && config->num_output) > - output = config->output[0]; > + if (!output && config->num_output) { > + config->compat_output = 0; > + output = config->output[config->compat_output]; > + } > > return output; > } > -- > 1.8.1 > > _______________________________________________ > [email protected]: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: http://lists.x.org/mailman/listinfo/xorg-devel > _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
