[FFmpeg-cvslog] r25049 - trunk/libavformat/matroskadec.c

[reimar]

Author: reimar
Date: Mon Sep  6 19:51:44 2010
New Revision: 25049

Log:
Add a special function to mkv demxuer to parse length values that includes
special-case code to handle all possible encodings of "unknown length".

Modified:
   trunk/libavformat/matroskadec.c

Modified: trunk/libavformat/matroskadec.c
==============================================================================
--- trunk/libavformat/matroskadec.c	Mon Sep  6 18:57:34 2010	(r25048)
+++ trunk/libavformat/matroskadec.c	Mon Sep  6 19:51:44 2010	(r25049)
@@ -575,6 +575,20 @@ static int ebml_read_num(MatroskaDemuxCo
     return read;
 }
 
+/**
+ * Read a EBML length value.
+ * This needs special handling for the "unknown length" case which has multiple
+ * encodings.
+ */
+static int ebml_read_length(MatroskaDemuxContext *matroska, ByteIOContext *pb,
+                            uint64_t *number)
+{
+    int res = ebml_read_num(matroska, pb, 8, number);
+    if (res > 0 && *number + 1 == 1ULL << (7 * res))
+        *number = 0xffffffffffffffULL;
+    return res;
+}
+
 /*
  * Read the next element as an unsigned int.
  * 0 is success, < 0 is failure.
@@ -782,7 +796,7 @@ static int ebml_parse_elem(MatroskaDemux
 
     if (syntax->type != EBML_PASS && syntax->type != EBML_STOP) {
         matroska->current_id = 0;
-        if ((res = ebml_read_num(matroska, pb, 8, &length)) < 0)
+        if ((res = ebml_read_length(matroska, pb, &length)) < 0)
             return res;
     }