On Fri, Jan 19, 2018 at 11:33 PM, Timo Rothenpieler <t...@rothenpieler.org> wrote: > Am 19.01.2018 um 19:47 schrieb Michael Niedermayer: >> >> On Fri, Jan 19, 2018 at 11:09:51AM +0100, Timo Rothenpieler wrote: >>> >>> Am 18.01.2018 um 07:52 schrieb Yogender Gupta: >>>> >>>> Improved the patch by dynamic allocation. >>>> >>>> Thanks, >>>> Yogender >>>> >>> >>>> @@ -1072,6 +1083,8 @@ static void cuvid_flush(AVCodecContext *avctx) >>>> if (ret < 0) >>>> goto error; >>>> >> >>>> + av_free(ctx->key_frame); >> >> >> this should probably be av_freep() >> unless the place where it resides is before some code >> unconditionally overwriting it afterwards > > > Yeah, it should. Changed and applied. > >
Shouldn't we generally recommend people use the new NVDEC implementation instead? It already has all these features and more. :) - Hendrik _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
