On 9/19/2017 7:26 PM, Mark Thompson wrote:
> On 17/09/17 04:20, James Almer wrote:
>> This also prevents the use of an uninitialized variable.
>>
>> Signed-off-by: James Almer <jamr...@gmail.com>
>> ---
>> libavutil/hwcontext_dxva2.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavutil/hwcontext_dxva2.c b/libavutil/hwcontext_dxva2.c
>> index 6c41788e2e..2ddd4be7b1 100644
>> --- a/libavutil/hwcontext_dxva2.c
>> +++ b/libavutil/hwcontext_dxva2.c
>> @@ -307,8 +307,10 @@ static int dxva2_map_frame(AVHWFramesContext *ctx,
>> AVFrame *dst, const AVFrame *
>> }
>>
>> map = av_mallocz(sizeof(*map));
>> - if (!map)
>> + if (!map) {
>> + err = AVERROR(ENOMEM);
>> goto fail;
>> + }
>>
>> err = ff_hwframe_map_create(src->hw_frames_ctx, dst, src,
>> dxva2_unmap_frame, map);
>>
>
> LGTM.
>
> Thanks,
>
> - Mark
Pushed.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
