[MPlayer-dev-eng] Crash in update_subtitles when audio only stream but with subtitles

[Reimar Döffinger]

On Sun, Feb 14, 2010 at 03:15:36AM +0100, Gianluigi Tiesi wrote:
> On Sat, Feb 13, 2010 at 12:28:39PM +0100, Reimar D?ffinger wrote:
> > On Sat, Feb 13, 2010 at 12:08:27PM +0100, Gianluigi Tiesi wrote:
> > > On Sat, Feb 13, 2010 at 12:05:48PM +0100, Gianluigi Tiesi wrote:
> > > > When an audio only clip (or the vo/vc failed to init)
> > > > 
> > > > mplayer tries to update anyway subtitles:
> > > > 
> > > > at line 3776:
> > > > update_subtitles(NULL, a_pos, mpctx->d_sub, 0);
> > > > 
> > > > in "/*========================== PLAY AUDIO ============================*/"
> > > > 
> > > > in mpcommon.c:
> > > > void update_subtitles(sh_video_t *sh_video, double refpts, demux_stream_t *d_dvdsub, int reset)
> > > > ...
> > > > 
> > > >         spudec_heartbeat(vo_spudec, 90000*sh_video->timer);
> > > > 
> > > > but sh_video is NULL
> > > > 
> > > 
> > > I think there is no point to:
> > >   update_subtitles(NULL, a_pos, mpctx->d_sub, 0);
> > >   update_osd_msg();
> > > 
> > > when mpctx->sh_video is NULL
> > > I don't known about gui (if osd msg can be updated anyway)
> > 
> > OSD and subtitles are displayed on the command line, so this is certainly not
> > pointless.
> > And when sh_video is NULL, vo_spudec is _supposed_ to be NULL as well and thus this
> > whole code should be skipped.
> > So a proper bugreport would be useful, while I can reproduce a crash, it already
> > crashes at mplayer.c:1168 for me.
> 
> 3159:
> 
>       vo_vobsub=vobsub_open(buf,spudec_ifo,0,&vo_spudec);
> 
> this inits vo_spudec

Please, a proper bug report with command line and output is something we
don't just ask of users because they can't defend themselves but because it is
useful.
I now guess that you are using -vobsub which I admit I hadn't tested yet.