Reviewed-by: Marek Olšák <marek.ol...@amd.com> Marek
On Sun, Jan 12, 2014 at 11:52 PM, Emil Velikov <emil.l.veli...@gmail.com> wrote: > _ColorDrawBufferIndexes is defined as GLint* and using a GLuint* > will result in the first part of the conditional to be evaluated to > true always. > > Unintentionally introduced by the following commit, this will result > in a driver segfault if one is using an old version of the piglit test > > bin/clearbuffer-mixed-format -auto -fbo > > commit 03d848ea1003abefd8fe51a5b4a780527cd852af > Author: Marek Olšák <marek.ol...@amd.com> > Date: Wed Dec 4 00:27:20 2013 +0100 > > mesa: fix interpretation of glClearBuffer(drawbuffer) > > This corresponding piglit tests supported this incorrect behavior instead > of > pointing at it. > > Cc: Marek Olšák <marek.ol...@amd.com> > Cc: 10.0 9.2 9.1 <mesa-sta...@lists.freedesktop.org> > Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> > --- > src/mesa/main/clear.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/mesa/main/clear.c b/src/mesa/main/clear.c > index f0b525f..d568ed8 100644 > --- a/src/mesa/main/clear.c > +++ b/src/mesa/main/clear.c > @@ -274,7 +274,7 @@ make_color_buffer_mask(struct gl_context *ctx, GLint > drawbuffer) > break; > default: > { > - GLuint buf = ctx->DrawBuffer->_ColorDrawBufferIndexes[drawbuffer]; > + GLint buf = ctx->DrawBuffer->_ColorDrawBufferIndexes[drawbuffer]; > > if (buf >= 0 && att[buf].Renderbuffer) { > mask |= 1 << buf; > -- > 1.8.5.2 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev