On Friday 20 November 2015 02:59:34 am James Almer wrote: > On 11/19/2015 10:05 PM, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes a memleak with the sample from bug 909.
> > /* find out the channel configuration */ > > ret = ff_opus_parse_extradata(avctx, c); > > - if (ret < 0) > > + if (ret < 0) { > > + av_freep(&c->fdsp); > > return ret; > > ff_opus_parse_extradata() can allocate s->channel_maps, so i think it > should also be freed here. > > LGTM aside from that. I have applied the patch with this change. Thank you, Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel