[FFmpeg-devel] [PATCH] cuvid : add support to force intra frames as in input source

Hendrik Leppkes h.leppkes at gmail.com
Sun Jan 21 01:03:57 EET 2018


On Fri, Jan 19, 2018 at 11:33 PM, Timo Rothenpieler
<timo at 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


More information about the ffmpeg-devel mailing list