[MPlayer-cvslog] r27414 - trunk/command.c

ben subversion at mplayerhq.hu
Mon Aug 4 15:35:15 CEST 2008


Author: ben
Date: Mon Aug  4 15:35:14 2008
New Revision: 27414

Log:
Fixes unsafe 'chapter' command with get_property() call.
Without it, MPlayer segv trying to dereference NULL demuxer.
Patch by Mathieu Schroeter (mathieu dot schroeter at gamesover dot ch)



Modified:
   trunk/command.c

Modified: trunk/command.c
==============================================================================
--- trunk/command.c	(original)
+++ trunk/command.c	Mon Aug  4 15:35:14 2008
@@ -379,12 +379,13 @@ static int mp_property_time_pos(m_option
 static int mp_property_chapter(m_option_t *prop, int action, void *arg,
                                MPContext *mpctx)
 {
-    int chapter;
+    int chapter = -1;
     float next_pts = 0;
     int chapter_num;
     int step_all;
     char *chapter_name = NULL;
 
+    if (mpctx->demuxer)
     chapter = demuxer_get_current_chapter(mpctx->demuxer);
     if (chapter < 0)
         return M_PROPERTY_UNAVAILABLE;



More information about the MPlayer-cvslog mailing list