Avoid leaking input items

diff --git a/src/playlist/engine.c b/src/playlist/engine.c
index 444932d89ee9451b9ecdde081cb4fbf306dc4a0e..5dc8257afe33cb0891a2b4f66c3d697158e35574 100644 (file)
--- a/src/playlist/engine.c
+++ b/src/playlist/engine.c
@@ -478,11 +478,9 @@ void playlist_LastLoop( playlist_t *p_playlist )
     playlist_MLDump( p_playlist );
 
     PL_LOCK;
-    /* Go through all items, and simply free everything without caring
-     * about the tree structure. Do not decref, it will be done by doing
-     * the same thing on the input items array */
     FOREACH_ARRAY( playlist_item_t *p_del, p_playlist->all_items )
         free( p_del->pp_children );
+        vlc_gc_decref( p_del->p_input );
         free( p_del );
     FOREACH_END();
     ARRAY_RESET( p_playlist->all_items );

diff --git a/src/playlist/loadsave.c b/src/playlist/loadsave.c
index b50c76dcdfedc9ae91d9d94c3c1ad63f0957781c..55b2a2787784db373125f66be03e870e1a7f9c01 100644 (file)
--- a/src/playlist/loadsave.c
+++ b/src/playlist/loadsave.c
@@ -151,6 +151,8 @@ int playlist_MLLoad( playlist_t *p_playlist )
     p_playlist->p_ml_onelevel->p_input =
     p_playlist->p_ml_category->p_input = p_input;
 
+    vlc_gc_incref( p_input );
+
     vlc_event_attach( &p_input->event_manager, vlc_InputItemSubItemAdded,
                         input_item_subitem_added, p_playlist );
 
@@ -194,7 +196,6 @@ int playlist_MLDump( playlist_t *p_playlist )
     stats_TimerStart( p_playlist, "ML Dump", STATS_TIMER_ML_DUMP );
     playlist_Export( p_playlist, psz_dirname, p_playlist->p_ml_category,
                      "export-xspf" );
-    vlc_gc_decref( p_playlist->p_ml_category->p_input );
     stats_TimerStop( p_playlist, STATS_TIMER_ML_DUMP );
 
     return VLC_SUCCESS;