[FFmpeg-cvslog] r16378 - in trunk/libavformat: rdt.c rmdec.c

[Ronald S. Bultje]

Hi,

On Sun, Dec 28, 2008 at 6:45 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Mon, Dec 29, 2008 at 12:24:00AM +0100, rbultje wrote:
>> Modified: trunk/libavformat/rdt.c
>> ==============================================================================
>> --- trunk/libavformat/rdt.c   Mon Dec 29 00:12:46 2008        (r16377)
>> +++ trunk/libavformat/rdt.c   Mon Dec 29 00:23:59 2008        (r16378)
>> @@ -438,6 +438,7 @@ static void
>>  rdt_free_extradata (PayloadContext *rdt)
>>  {
>>      ff_rm_free_rmstream(rdt->rmst[0]);
>> +    av_free(rdt->rmst[0]);
>>      if (rdt->rmctx)
>>          av_close_input_stream(rdt->rmctx);
>>      av_freep(&rdt->mlti_data);
>
> this really wasnt part of reimars patch
> besides it should be av_freep(), as its safer ...

The structure in which the pointer resides (rdt) disappears in the
same function call, so is av_freep() really needed?

Ronald