From b0e4f32011ff39efc6d6a45de4aa091c7aa8d170 Mon Sep 17 00:00:00 2001
From: James Zern <jzern@google.com>
Date: Fri, 21 Aug 2015 19:48:17 -0700
Subject: [PATCH] mkvparser/Cues: convert asserts to failure returns

Change-Id: I88b5dfecfd03fec7f5808959ec6f640bff838d7a
---
 mkvparser.cpp | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/mkvparser.cpp b/mkvparser.cpp
index ddce106..b32419c 100644
--- a/mkvparser.cpp
+++ b/mkvparser.cpp
@@ -1903,7 +1903,8 @@ bool Cues::LoadCuePoint() const {
 
     if (id != 0x3B) {  // CuePoint ID
       m_pos += size;  // consume payload
-      assert(m_pos <= stop);
+      if (m_pos > stop)
+        return false;
 
       continue;
     }
@@ -1924,7 +1925,8 @@ bool Cues::LoadCuePoint() const {
     --m_preload_count;
 
     m_pos += size;  // consume payload
-    assert(m_pos <= stop);
+    if (m_pos > stop)
+      return false;
 
     return true;  // yes, we loaded a cue point
   }
@@ -2303,7 +2305,8 @@ bool CuePoint::Load(IMkvReader* pReader) {
     }
 
     pos += size;  // consume payload
-    assert(pos <= stop);
+    if (pos > stop)
+      return false;
   }
 
   assert(size_t(p - m_track_positions) == m_track_positions_count);