[vlc-commits] commit: Fixed a potential double free on error in qtl/shoutcast demuxers. ( Laurent Aimar )

git at videolan.org git at videolan.org
Sun Oct 31 13:05:27 CET 2010


vlc | branch: master | Laurent Aimar <fenrir at videolan.org> | Sun Oct 31 12:31:32 2010 +0100| [d9f4825a3fa8b2193a14d41d6f01db1eaa6f3f7a] | committer: Laurent Aimar 

Fixed a potential double free on error in qtl/shoutcast demuxers.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=d9f4825a3fa8b2193a14d41d6f01db1eaa6f3f7a
---

 modules/demux/playlist/qtl.c       |    1 +
 modules/demux/playlist/shoutcast.c |    1 -
 2 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/modules/demux/playlist/qtl.c b/modules/demux/playlist/qtl.c
index 73aaff5..5b2965c 100644
--- a/modules/demux/playlist/qtl.c
+++ b/modules/demux/playlist/qtl.c
@@ -143,6 +143,7 @@ static int Demux( demux_t *p_demux )
         msg_Err( p_demux, "invalid root node %i, %s",
                  xml_ReaderNodeType( p_xml_reader ), psz_eltname );
         free( psz_eltname );
+        psz_eltname = NULL;
 
         /* second line has <?quicktime tag ... so we try to skip it */
         msg_Dbg( p_demux, "trying to read one more node" );
diff --git a/modules/demux/playlist/shoutcast.c b/modules/demux/playlist/shoutcast.c
index a8c2e5d..4f90b03 100644
--- a/modules/demux/playlist/shoutcast.c
+++ b/modules/demux/playlist/shoutcast.c
@@ -181,7 +181,6 @@ static int DemuxGenre( demux_t *p_demux, xml_reader_t *p_xml_reader,
                         {
                             free( psz_attrname );
                             free( psz_attrvalue );
-                            free( psz_eltname );
                             break;
                         }
 



More information about the vlc-commits mailing list