Re: [Mesa-dev] [PATCH] prog_parameter.c ASAN Patch

2013-07-01 Thread Myles C. Maxfield
t; + } > + for (; j < 4; j++) { > + paramList->ParameterValues[**oldNum + i][j] = 0.0f; > > + } > + } > values += 4; > p->Initialized = GL_TRUE; > } > >

Re: [Mesa-dev] [PATCH] prog_parameter.c ASAN Patch

2013-06-28 Thread Myles C. Maxfield
Friendly ping regarding this patch :-) --Myles On Wed, Jun 19, 2013 at 12:47 AM, Myles C. Maxfield < myles.maxfi...@gmail.com> wrote: > Any word on this? > > Thanks, > Myles > > > On Mon, Jun 17, 2013 at 12:09 PM, Myles C. Maxfield < > myles.maxfi...@gmail.c

Re: [Mesa-dev] [PATCH] prog_parameter.c ASAN Patch

2013-06-19 Thread Myles C. Maxfield
Any word on this? Thanks, Myles On Mon, Jun 17, 2013 at 12:09 PM, Myles C. Maxfield < myles.maxfi...@gmail.com> wrote: > Sure. I was under the impression that |size| couldn't be both greater than > 4 and a non-multiple of 4, but I've reworked the patch to incorporate thi

Re: [Mesa-dev] [PATCH] prog_parameter.c ASAN Patch

2013-06-17 Thread Myles C. Maxfield
e subject of this email thread? Anyway, here's a similar but modified patch: From: Myles C. Maxfield Date: Mon, 17 Jun 2013 11:50:05 -0700 Subject: [PATCH] Appeasing Address Sanitizer --- src/mesa/program/prog_parameter.c | 13 - 1 file changed, 12 insertions(+), 1 deletio

[Mesa-dev] [PATCH] prog_parameter.c ASAN Patch

2013-06-14 Thread Myles C. Maxfield
efore-initialize because the last 3 of the 4 elements haven't been written to yet. I was hoping to contribute a patch that will silence this problem that ASAN reports. I'm happy to incorporate any feedback anyone has into this patch. Thanks, Myles C. Maxfield [1] https://code.google.com/p/ad