Yep, looks right to me. Reviewed-by: Jeremy Huddleston Sequoia: <jerem...@apple.com>
On Feb 10, 2014, at 07:57, Ian Romanick <i...@freedesktop.org> wrote: > Perhaps Jeremy can review this patch... > > On 02/10/2014 01:58 AM, Siavash Eliasi wrote: >> --- >> src/glx/apple/apple_glx_context.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/src/glx/apple/apple_glx_context.c >> b/src/glx/apple/apple_glx_context.c >> index 0bb25b4..8a6ded2 100644 >> --- a/src/glx/apple/apple_glx_context.c >> +++ b/src/glx/apple/apple_glx_context.c >> @@ -142,6 +142,7 @@ apple_glx_create_context(void **ptr, Display * dpy, int >> screen, >> if (sharedac && !is_context_valid(sharedac)) { >> *errorptr = GLXBadContext; >> *x11errorptr = false; >> + free(ac); >> return true; >> } >> >> > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev