[FFmpeg-devel] [PATCH] avformat/vobsub: compare correct packet stream IDs

wm4 nfxjfg at googlemail.com
Mon Sep 21 12:09:08 CEST 2015

The stream ID is essentially an arbitrary number defined by the .idx
file headers. They have to match the IDs in the .sub stream. The vobsub
demuxer assumed the IDs would just start from 0, increassing by 1 for
each stream. This is not correct. In the sample I had, the IDs were
starting from 1, leading to no subtitles being displayed at all.

Fix this by using the correct stream ID.
 libavformat/mpeg.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c
index ab1a9ca..55f2b3f 100644
--- a/libavformat/mpeg.c
+++ b/libavformat/mpeg.c
@@ -940,7 +940,7 @@ static int vobsub_read_packet(AVFormatContext *s, AVPacket *pkt)
         total_read += pkt_size;
         /* the current chunk doesn't match the stream index (unlikely) */
-        if ((startcode & 0x1f) != idx_pkt.stream_index)
+        if ((startcode & 0x1f) != s->streams[idx_pkt.stream_index]->id)
         ret = av_grow_packet(pkt, to_read);

More information about the ffmpeg-devel mailing list