Re: [Piglit] [PATCH v1] piglit/vbo: memory leak

2018-10-12 Thread Sergii Romantsov 
Thanks, Ian.
But just for case: don't have push-rights...

On Fri, Oct 12, 2018 at 12:26 AM Ian Romanick  wrote:

> Reviewed-by: Ian Romanick 
>
> On 10/11/2018 05:59 AM, Sergii Romantsov wrote:
> > Trivial fix of memory leaking for test 'initialized-vbo'.
> >
> > Fixes: 6a467e00cfae (security/initialized-vbo: test contents of
> uninitialized VBOs)
> > CC: Brian Paul 
> > Signed-off-by: Sergii Romantsov 
> > ---
> >  tests/security/initialized-vbo.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/security/initialized-vbo.c
> b/tests/security/initialized-vbo.c
> > index 20c8155..16656bc 100644
> > --- a/tests/security/initialized-vbo.c
> > +++ b/tests/security/initialized-vbo.c
> > @@ -70,6 +70,7 @@ piglit_display(void)
> >
> >   glDeleteBuffers(1, &buf);
> >
> > + free(tmp);
> >   return pass ? PIGLIT_PASS : PIGLIT_WARN;
> >  }
> >
> >
>
>
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Re: [Piglit] [PATCH v1] piglit/vbo: memory leak

2018-10-11 Thread Ian Romanick 
Reviewed-by: Ian Romanick 

On 10/11/2018 05:59 AM, Sergii Romantsov wrote:
> Trivial fix of memory leaking for test 'initialized-vbo'.
> 
> Fixes: 6a467e00cfae (security/initialized-vbo: test contents of uninitialized 
> VBOs)
> CC: Brian Paul 
> Signed-off-by: Sergii Romantsov 
> ---
>  tests/security/initialized-vbo.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/security/initialized-vbo.c 
> b/tests/security/initialized-vbo.c
> index 20c8155..16656bc 100644
> --- a/tests/security/initialized-vbo.c
> +++ b/tests/security/initialized-vbo.c
> @@ -70,6 +70,7 @@ piglit_display(void)
>  
>   glDeleteBuffers(1, &buf);
>  
> + free(tmp);
>   return pass ? PIGLIT_PASS : PIGLIT_WARN;
>  }
>  
> 

___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit