From c178920505b530f411d0bc93f9e55278af557fc7 Mon Sep 17 00:00:00 2001
From: wm4 <wm4@nowhere>
Date: Tue, 31 Jan 2017 12:44:14 +0100
Subject: demux_mkv: any reference makes a frame not a keyframe

Fixes seeking with:

https://bugs.chromium.org/p/chromium/issues/detail?id=497889

Haali also ignores the element's contents, and interprets its presence
as the block not being a keyframe. FFmpeg is going to have an equivalent
change.

I don't know yet whether the affected sample is valid - a reference
timestamp of 0 doesn't make too much sense to me.
---
 demux/demux_mkv.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

(limited to 'demux')

diff --git a/demux/demux_mkv.c b/demux/demux_mkv.c
index ddda2ecb61..5db975926e 100644
--- a/demux/demux_mkv.c
+++ b/demux/demux_mkv.c
@@ -2604,8 +2604,7 @@ static int read_block_group(demuxer_t *demuxer, int64_t end,
             int64_t num = ebml_read_int(s);
             if (num == EBML_INT_INVALID)
                 goto error;
-            if (num)
-                block->keyframe = false;
+            block->keyframe = false;
             break;
 
         case MATROSKA_ID_CLUSTER:
-- 
cgit v1.2.3