lib: avoid NULL deref on error
authorRémi Denis-Courmont <remi@remlab.net>
Wed, 23 Apr 2014 16:41:54 +0000 (19:41 +0300)
committerRémi Denis-Courmont <remi@remlab.net>
Wed, 23 Apr 2014 16:41:54 +0000 (19:41 +0300)
lib/media.c

index af70b4a..bac11da 100644 (file)
@@ -113,15 +113,14 @@ static void input_item_subitem_added( const vlc_event_t *p_event,
                 p_event->u.input_item_subitem_added.p_new_child );
 
     /* Add this to our media list */
                 p_event->u.input_item_subitem_added.p_new_child );
 
     /* Add this to our media list */
-    if( !p_md->p_subitems )
+    if( p_md->p_subitems == NULL )
     {
         p_md->p_subitems = libvlc_media_list_new( p_md->p_libvlc_instance );
     {
         p_md->p_subitems = libvlc_media_list_new( p_md->p_libvlc_instance );
+        if( unlikely(p_md->p_subitems == NULL) )
+            abort();
         libvlc_media_list_set_media( p_md->p_subitems, p_md );
     }
         libvlc_media_list_set_media( p_md->p_subitems, p_md );
     }
-    if( p_md->p_subitems )
-    {
-        libvlc_media_list_add_media( p_md->p_subitems, p_md_child );
-    }
+    libvlc_media_list_add_media( p_md->p_subitems, p_md_child );
 
     /* Construct the event */
     event.type = libvlc_MediaSubItemAdded;
 
     /* Construct the event */
     event.type = libvlc_MediaSubItemAdded;