Hi,
On Thu, Apr 8, 2010 at 9:10 AM, zhentan feng <[email protected]> wrote:
> On Thu, Apr 8, 2010 at 4:43 AM, Aurelien Jacobs <[email protected]> wrote:
>> On Wed, Apr 07, 2010 at 06:34:05PM +0200, spyfeng wrote:
>> > Author: spyfeng
>> > Date: Wed Apr 7 18:34:05 2010
>> > New Revision: 5736
>> >
>> > Log:
>> > check av_realloc() failure.
>> >
>> > Modified:
>> > mms/mmst.c
>> >
>> > Modified: mms/mmst.c
>> >
>> ==============================================================================
>> > --- mms/mmst.c Wed Apr 7 18:27:52 2010 (r5735)
>> > +++ mms/mmst.c Wed Apr 7 18:34:05 2010 (r5736)
>> > @@ -321,6 +321,8 @@ static MMSSCPacketType get_tcp_server_re
>> > mms->asf_header =
>> av_realloc(mms->asf_header,
>> > mms->asf_header_size
>> > + mms->pkt_buf_len);
>> > + if (!mms->asf_header)
>> > + return -1;
>>
>> memory leak...
>
> i fixed it, thanks.
Nope, see man realloc:
For realloc(), the input pointer is still valid if reallocation failed.
In other words, you should free() mms->asf_header() if realloc
returned NULL but mms->asf_header (the argument) was non-NULL.
Ronald
_______________________________________________
FFmpeg-soc mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
