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

[Reimar Döffinger]

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.