[vlc-devel] [PATCH] playlist: m3u: fix memory leak (#25138)

Romain Vimont rom1v at videolabs.io
Wed Sep 23 10:38:00 CEST 2020


LGTM

On Wed, Sep 23, 2020 at 10:33:42AM +0200, Alexandre Janniaux wrote:
> The array of options was not released when the meta object was cleaned.
> 
> Fix #25138
> ---
>  modules/demux/playlist/m3u.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/modules/demux/playlist/m3u.c b/modules/demux/playlist/m3u.c
> index 2cf71fd8d8..56ee64ac87 100644
> --- a/modules/demux/playlist/m3u.c
> +++ b/modules/demux/playlist/m3u.c
> @@ -239,6 +239,7 @@ static void entry_meta_Clean( struct entry_meta_s *e )
>      free( e->psz_tvgid );
>      free( e->psz_grouptitle );
>      while( e->i_options-- ) free( (char*)e->ppsz_options[e->i_options] );
> +    TAB_CLEAN( e->i_options, e->ppsz_options );
>  }
>  
>  static void parseEXTINF( char *, char *(*)(const char *), struct entry_meta_s * );
> -- 
> 2.28.0
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list