[vlc-devel] [PATCH] demux: mkv: Remove unnecessary throw

Soomin Lee bubu at mikan.io
Tue Nov 27 12:04:15 CET 2018


This case wasn't an error in the file itself and the playback can continue
 to the following files.

Additionally, throwing led to a crash on iOS 12.0.0 and iOS 12.0.1.

(ref: https://rink.hockeyapp.net/manage/apps/194470/app_versions/113/crash_reasons/242850559)
---
 modules/demux/mkv/matroska_segment.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/modules/demux/mkv/matroska_segment.cpp b/modules/demux/mkv/matroska_segment.cpp
index 93647113f8..29e63a2459 100644
--- a/modules/demux/mkv/matroska_segment.cpp
+++ b/modules/demux/mkv/matroska_segment.cpp
@@ -1216,7 +1216,6 @@ int matroska_segment_c::BlockGet( KaxBlock * & pp_block, KaxSimpleBlock * & pp_s
         {
             VLC_UNUSED( kcue );
             msg_Warn( vars.p_demuxer, "find KaxCues FIXME" );
-            throw VLC_EGENERIC;
         }
         E_CASE_DEFAULT(element)
         {
-- 
2.19.1



More information about the vlc-devel mailing list