On 9/14/2017 4:51 AM, Luca Barbato wrote:
> On 13/09/2017 22:07, James Almer wrote:
>> ret is 0 by default.
>>
>> Signed-off-by: James Almer <[email protected]>
>> ---
>>   libavcodec/extract_extradata_bsf.c | 5 ++++-
>>   1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavcodec/extract_extradata_bsf.c
>> b/libavcodec/extract_extradata_bsf.c
>> index 20840bd6a..100c60d06 100644
>> --- a/libavcodec/extract_extradata_bsf.c
>> +++ b/libavcodec/extract_extradata_bsf.c
>> @@ -101,14 +101,17 @@ static int extract_extradata_h2645(AVBSFContext
>> *ctx, AVPacket *pkt,
>>             if (s->remove) {
>>               filtered_buf = av_buffer_alloc(pkt->size +
>> AV_INPUT_BUFFER_PADDING_SIZE);
>> -            if (!filtered_buf)
>> +            if (!filtered_buf) {
>> +                ret = AVERROR(ENOMEM);
>>                   goto fail;
>> +            }
>>               filtered_data = filtered_buf->data;
>>           }
>>             extradata = av_malloc(extradata_size +
>> AV_INPUT_BUFFER_PADDING_SIZE);
>>           if (!extradata) {
>>               av_buffer_unref(&filtered_buf);
>> +            ret = AVERROR(ENOMEM);
>>               goto fail;
>>           }
>>  
> 
> Seems fine.

Ping?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to