[FFmpeg-devel] [PATCH 4/5] ogg: allow streams to update metadata
nfxjfg at googlemail.com
Sat Oct 19 22:52:06 CEST 2013
On Sat, 19 Oct 2013 22:43:27 +0200
Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:
> On 19.10.2013, at 15:40, Ben Boeckel <mathstuf at gmail.com> wrote:
> > + av_free(os->new_metadata);
> > + os->new_metadata = NULL;
> These (appearing in two places) can be merged into a single av_freep.
> > @@ -639,6 +646,7 @@ static int ogg_read_close(AVFormatContext *s)
> > ogg->streams[i].codec->cleanup(s, i);
> > }
> > av_free(ogg->streams[i].private);
> > + av_free(ogg->streams[i].new_metadata);
> > }
> > av_free(ogg->streams);
> Not really related to your patch, but I wonder why none of these use av_freep.
Why would they? The whole streams array is freed right after this.
More information about the ffmpeg-devel