[vlc-commits] shoutcast: Fix leak in case of missing end element.

Hugo Beauzée-Luyssen git at videolan.org
Thu Apr 6 17:49:08 CEST 2017


vlc | branch: master | Hugo Beauzée-Luyssen <hugo at beauzee.fr> | Thu Apr  6 17:42:48 2017 +0200| [7c5f9a8ca9ad1c41e82f7e21be240592040ba7bc] | committer: Hugo Beauzée-Luyssen

shoutcast: Fix leak in case of missing end element.

CID #1253103

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

 modules/demux/playlist/shoutcast.c | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/modules/demux/playlist/shoutcast.c b/modules/demux/playlist/shoutcast.c
index c3e1c54..168be31 100644
--- a/modules/demux/playlist/shoutcast.c
+++ b/modules/demux/playlist/shoutcast.c
@@ -359,6 +359,16 @@ static int DemuxStation( demux_t *p_demux, xml_reader_t *p_xml_reader,
                 break;
         }
     }
-    /* FIXME: leaks on missing ENDELEMENT? */
+    /* Free all strings anyway, in case of missing end element */
+    FREENULL( psz_base );
+    FREENULL( psz_name );
+    FREENULL( psz_mt );
+    FREENULL( psz_id );
+    FREENULL( psz_br );
+    FREENULL( psz_genre );
+    FREENULL( psz_ct );
+    FREENULL( psz_lc );
+    FREENULL( psz_rt );
+    FREENULL( psz_load );
     return 0;
 }



More information about the vlc-commits mailing list