[FFmpeg-cvslog] r11562 - trunk/libavformat/mxf.c

[Baptiste Coudurier]

Michael Niedermayer wrote:
> On Sat, Jan 19, 2008 at 04:21:30PM +0100, bcoudurier wrote:
>> Author: bcoudurier
>> Date: Sat Jan 19 16:21:30 2008
>> New Revision: 11562
>>
>> Log:
>> check malloc return
>>
>> Modified:
>>    trunk/libavformat/mxf.c
>>
>> Modified: trunk/libavformat/mxf.c
>> ==============================================================================
>> --- trunk/libavformat/mxf.c	(original)
>> +++ trunk/libavformat/mxf.c	Sat Jan 19 16:21:30 2008
>> @@ -281,6 +281,8 @@ static int mxf_decrypt_triplet(AVFormatC
>>  
>>      if (!mxf->aesc && s->key && s->keylen == 16) {
>>          mxf->aesc = av_malloc(av_aes_size);
>> +        if (!mxf->aesc)
>> +            return -1;
> 
> shouldnt all these be AVERROR(ENOMEM) ?
> 

Well, yes, probably, and I need to propagate error ...

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312