Bug 1342808: Ignore user data track. r?gerald draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Mon, 27 Feb 2017 18:41:55 +0100
changeset 490118 794895be510db6f7f3c5b713772a51554b93bdbf
parent 489767 cff601859a99bffa1443bc8df6d8eae9fc4e62c1
child 547173 0a278f937e8c1953cd24571352635103629f9a07
push id47002
push userbmo:jyavenard@mozilla.com
push dateMon, 27 Feb 2017 17:43:13 +0000
reviewersgerald
bugs1342808
milestone54.0a1
Bug 1342808: Ignore user data track. r?gerald That's the 'udta' box. They serve no purpose as far as we're concerned and stagefright handling of it is buggy. MozReview-Commit-ID: Af6wCpq2vZr
media/libstagefright/frameworks/av/media/libstagefright/MPEG4Extractor.cpp
--- a/media/libstagefright/frameworks/av/media/libstagefright/MPEG4Extractor.cpp
+++ b/media/libstagefright/frameworks/av/media/libstagefright/MPEG4Extractor.cpp
@@ -748,17 +748,16 @@ status_t MPEG4Extractor::parseChunk(off6
         case FOURCC('m', 'd', 'i', 'a'):
         case FOURCC('m', 'i', 'n', 'f'):
         case FOURCC('d', 'i', 'n', 'f'):
         case FOURCC('s', 't', 'b', 'l'):
         case FOURCC('m', 'v', 'e', 'x'):
         case FOURCC('m', 'o', 'o', 'f'):
         case FOURCC('t', 'r', 'a', 'f'):
         case FOURCC('m', 'f', 'r', 'a'):
-        case FOURCC('u', 'd', 't', 'a'):
         case FOURCC('i', 'l', 's', 't'):
         case FOURCC('s', 'i', 'n', 'f'):
         case FOURCC('s', 'c', 'h', 'i'):
         case FOURCC('e', 'd', 't', 's'):
         {
             if (chunk_type == FOURCC('s', 't', 'b', 'l')) {
                 ALOGV("sampleTable chunk is %d bytes long.", (size_t)chunk_size);