libavformat: Free AVFormatContext->streams

Message ID 1303291935-74806-1-git-send-email-martin@martin.st
State Committed
Commit 7bbb67d58005f3c0384f657570fabcdc0bc8cc21
Headers show

Commit Message

Martin Storsjö April 20, 2011, 9:32 a.m.
After switching this from a statically allocated array to a
dynamically allocated one in the major bump, this needs explicit
freeing.
---
Tested that it passes FATE, too.

 libavformat/utils.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Comments

Ronald Bultje April 20, 2011, 12:34 p.m. | #1
Hi,

On Wed, Apr 20, 2011 at 5:32 AM, Martin Storsjö <martin@martin.st> wrote:
> After switching this from a statically allocated array to a
> dynamically allocated one in the major bump, this needs explicit
> freeing.
> ---
> Tested that it passes FATE, too.
>
>  libavformat/utils.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/libavformat/utils.c b/libavformat/utils.c
> index 0c80ff6..19498b9 100644
> --- a/libavformat/utils.c
> +++ b/libavformat/utils.c
> @@ -2564,6 +2564,7 @@ void avformat_free_context(AVFormatContext *s)
>     av_freep(&s->chapters);
>     av_metadata_free(&s->metadata);
>     av_freep(&s->key);
> +    av_freep(&s->streams);
>     av_free(s);
>  }

OK.

Ronald

Patch

diff --git a/libavformat/utils.c b/libavformat/utils.c
index 0c80ff6..19498b9 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -2564,6 +2564,7 @@  void avformat_free_context(AVFormatContext *s)
     av_freep(&s->chapters);
     av_metadata_free(&s->metadata);
     av_freep(&s->key);
+    av_freep(&s->streams);
     av_free(s);
 }