[FFmpeg-devel] [PATCH] flacdec: fix memleaks

Paul B Mahol onemda at gmail.com
Sun Jul 29 17:36:28 CEST 2012


On 7/29/12, Reimar Doeffinger <Reimar.Doeffinger at gmx.de> wrote:
> On Sun, Jul 29, 2012 at 02:14:12PM +0000, Paul B Mahol wrote:
>> Signed-off-by: Paul B Mahol <onemda at gmail.com>
>
> Thanks, but could you please use goto instead?
> This is quite a lot of code duplication, and if someone e.g. adds a second
> buffer that needs to be freed there's a good chance they'll miss to add
> it at some places.

Use goto only for this specific case or for all cases?


More information about the ffmpeg-devel mailing list