Hi Alex, On 29 April 2016 at 15:30, Alex Deucher <alexdeuc...@gmail.com> wrote: > On Fri, Apr 29, 2016 at 8:56 AM, Emil Velikov <emil.l.veli...@gmail.com> > wrote: >> From: Emil Velikov <emil.veli...@collabora.com> >> >> Earlier commit plugged a memory leak, although it missed a pair of >> brackets. Thus we unconditionally returned even in the case of no error. >> >> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95203 >> Fixes: b87856d25d1 ("st/omx: Fix resource leak on OMX_ErrorNone") >> Cc: Andy Furniss <adf.li...@gmail.com> >> Cc: Robert Foss <robert.f...@collabora.com> >> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> >> --- >> What an embarassing bug - missing brackets. Andy can you confirm that it >> resolves the issue ? > > Reviewed-by: Alex Deucher <alexander.deuc...@amd.com> > Sorry I've missed your review and pushed the fix with Andy and Rob's tags.
-Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev