Re: [libav-devel] [PATCH v2] cpu: split flag checks per arch in av_cpu_max_align()

2017-10-07 Thread James Almer 
On 9/15/2017 4:33 AM, Luca Barbato wrote:
> On 14/09/2017 21:51, James Almer wrote:
>> Signed-off-by: James Almer 
>> ---
>>   libavutil/aarch64/cpu.c  |  9 +
>>   libavutil/arm/cpu.c  |  9 +
>>   libavutil/cpu.c  | 15 ---
>>   libavutil/cpu_internal.h |  5 +
>>   libavutil/ppc/cpu.c  |  9 +
>>   libavutil/x86/cpu.c  | 11 +++
>>   6 files changed, 51 insertions(+), 7 deletions(-)
>>
> 
> I like it.

Ping?
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] avcodec/extract_extradata: return an error when buffer allocation fails

2017-10-07 Thread James Almer 
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 
>> ---
>>   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(_buf);
>> +    ret = AVERROR(ENOMEM);
>>   goto fail;
>>   }
>>  
> 
> Seems fine.

Ping?
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel