[vlc-commits] demux/playlist/itml: remove unnecessary branch

Filip Roséen git at videolan.org
Wed Mar 15 19:33:27 CET 2017


vlc | branch: master | Filip Roséen <filip at atch.se> | Wed Mar 15 10:10:13 2017 +0100| [d8a1ff9974aa3762ab524d127a9b8c635313d057] | committer: Jean-Baptiste Kempf

demux/playlist/itml: remove unnecessary branch

An xml-document containing a tag without a name is ill-formed, so this
case will never happen in practice (as the xml-reader will error out
before).

However, if it for some reason would happen, it will be handled due to
the fact that there will not be any handler matching "".

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

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

 modules/demux/playlist/itml.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/modules/demux/playlist/itml.c b/modules/demux/playlist/itml.c
index c77cbaa..2c9a0b7 100644
--- a/modules/demux/playlist/itml.c
+++ b/modules/demux/playlist/itml.c
@@ -179,11 +179,6 @@ static bool parse_dict( demux_t *p_demux, input_item_node_t *p_input_node,
         {
         /*  element start tag  */
         case XML_READER_STARTELEM:
-            if( !*node )
-            {
-                msg_Err( p_demux, "invalid XML stream" );
-                goto end;
-            }
             /* choose handler */
             for( p_handler = p_handlers;
                      p_handler->name && strcmp( node, p_handler->name );



More information about the vlc-commits mailing list