diff mbox series

[FFmpeg-devel,v1] avformat/concat: priv_data should be freed internally

Message ID 20200403130306.17814-1-lance.lmwang@gmail.com
State Accepted
Commit 6da653e525d57484bb1e5988a08255e9bbb6cf41
Headers show
Series [FFmpeg-devel,v1] avformat/concat: priv_data should be freed internally | expand

Checks

Context Check Description
andriy/ffmpeg-patchwork success Make fate finished

Commit Message

Lance Wang April 3, 2020, 1:03 p.m. UTC 
From: Limin Wang <lance.lmwang@gmail.com>

Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
---
 libavformat/concat.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Lance Wang May 1, 2020, 10:10 a.m. UTC | #1 
On Fri, Apr 03, 2020 at 09:03:06PM +0800, lance.lmwang@gmail.com wrote:
> From: Limin Wang <lance.lmwang@gmail.com>
> 
> Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
> ---
>  libavformat/concat.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/libavformat/concat.c b/libavformat/concat.c
> index ea3bc1dfde..cfe14760eb 100644
> --- a/libavformat/concat.c
> +++ b/libavformat/concat.c
> @@ -75,7 +75,6 @@ static av_cold int concat_open(URLContext *h, const char *uri, int flags)
>          if (uri[i] == *AV_CAT_SEPARATOR) {
>              /* integer overflow */
>              if (++len == UINT_MAX / sizeof(*nodes)) {
> -                av_freep(&h->priv_data);
>                  return AVERROR(ENAMETOOLONG);
>              }
>          }
> -- 
> 2.21.0
> 

will apply it.
diff mbox series

Patch

diff --git a/libavformat/concat.c b/libavformat/concat.c
index ea3bc1dfde..cfe14760eb 100644
--- a/libavformat/concat.c
+++ b/libavformat/concat.c
@@ -75,7 +75,6 @@  static av_cold int concat_open(URLContext *h, const char *uri, int flags)
         if (uri[i] == *AV_CAT_SEPARATOR) {
             /* integer overflow */
             if (++len == UINT_MAX / sizeof(*nodes)) {
-                av_freep(&h->priv_data);
                 return AVERROR(ENAMETOOLONG);
             }
         }