[vlc-commits] demux:mkv: do not preload the first Cluster

[Steve Lhomme]

vlc | branch: master | Steve Lhomme <robux4 at ycbcr.xyz> | Tue Jan  2 16:38:18 2018 +0100| [6b10c2e60624aee8b9ca689cef8e0688b8ae35ef] | committer: Jean-Baptiste Kempf

demux:mkv: do not preload the first Cluster

We stop preload the Matroska head when encountering the first Cluster ID+Length
and we keep it for when we start reading (unless we seek to a different position
on startup).

Fix #19177

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=6b10c2e60624aee8b9ca689cef8e0688b8ae35ef
---

 modules/demux/mkv/matroska_segment.cpp | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/modules/demux/mkv/matroska_segment.cpp b/modules/demux/mkv/matroska_segment.cpp
index dad3d70d63..e6361a1953 100644
--- a/modules/demux/mkv/matroska_segment.cpp
+++ b/modules/demux/mkv/matroska_segment.cpp
@@ -628,21 +628,8 @@ bool matroska_segment_c::Preload( )
             }
             msg_Dbg( &sys.demuxer, "|   + Cluster" );
 
-            if( !ParseCluster( kc_ptr ) )
-                break;
 
             cluster = kc_ptr;
-            IndexAppendCluster( cluster );
-
-            // add first cluster as trusted seekpoint for all tracks
-            for( tracks_map_t::const_iterator it = tracks.begin();
-                 it != tracks.end(); ++it )
-            {
-                _seeker.add_seekpoint( it->first,
-                    SegmentSeeker::Seekpoint( cluster->GetElementPosition(), 0 ) );
-            }
-
-            ep->Down();
             /* stop pre-parsing the stream */
             break;
         }