MPEGPlayer: Fix leakage of file decriptors if file wasn't accepted by playback engine-- code is now getting tested that couldn't be before. Also reset parser in stream init when it doesn't like the file.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@28853 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 4 insertions, 3 deletions

diff --git a/apps/plugins/mpegplayer/stream_mgr.c b/apps/plugins/mpegplayer/stream_mgr.c
index 2eed3d355d..b88c6d70c7 100644
--- a/apps/plugins/mpegplayer/stream_mgr.c
+++ b/apps/plugins/mpegplayer/stream_mgr.c
@@ -592,7 +592,7 @@ static int stream_on_close(void)
 
     stream_mgr_lock();
 
-    /* Any open file? */
+    /* Any open file that was accepted for playback? */
     if (stream_mgr.filename != NULL)
     {
         /* Yes - hide video */
@@ -602,12 +602,13 @@ static int stream_on_close(void)
         stream_on_stop(false);
         /* Tell parser file is finished */
         parser_close_stream();
-        /* Close file */
-        disk_buf_close();
         /* Reinitialize manager */
         stream_mgr_init_state();
     }
 
+    /* Let disk buffer reset itself - file might be open even if no good */
+    disk_buf_close();
+
     stream_mgr_unlock();
 
     return status;