oggdemux: No need to subtract begin time
authorThiago Santos <thiago.sousa.santos@collabora.co.uk>
Tue, 19 Jan 2010 11:39:14 +0000 (08:39 -0300)
committerThiago Santos <thiago.sousa.santos@collabora.co.uk>
Tue, 19 Jan 2010 11:39:14 +0000 (08:39 -0300)
Last stop is already based on the chain start and there is no need
to subtract the chain start as it may lead to a negative overflow.
This was causing seeking issues when the target chain was not
the first one (that has chain start = 0)

Fixes #606382

ext/ogg/gstoggdemux.c

index 1540af2..cd4711c 100644 (file)
@@ -2138,7 +2138,7 @@ gst_ogg_demux_perform_seek (GstOggDemux * ogg, GstEvent * event)
       stop = MIN (stop, chain->segment_stop);
     }
 
-    last_stop = ogg->segment.last_stop - begin_time;
+    last_stop = ogg->segment.last_stop;
     if (chain->segment_start != GST_CLOCK_TIME_NONE)
       last_stop += chain->segment_start;