Bug 1275853: [mp4] Don't reject boxes that contains padding but can be easily skipped. r=kentuckyfriedtakahe

My guess is that the application generating those files, does a very rough estimate on what the sample table size is going to be, which allows to perform the conversion in a single pass. This allows to place the moov box at the beginning.

MozReview-Commit-ID: IGzxTho4Akk

--HG--
extra : rebase_source : b6ca711a00f8b2d7d8708acdd17f52f59b469685

media/libstagefright/frameworks/av/media/libstagefright/MPEG4Extractor.cpp

@@ -1171,9 +1171,8 @@ status_t MPEG4Extractor::parseChunk(off64_t *offset, int depth) {
                 }
             }
 
-            if (*offset != stop_offset) {
-                return ERROR_MALFORMED;
-            }
+            // Some muxers add some padding after the stsd content. Skip it.
+            *offset = stop_offset;
             break;
         }
 

media/libstagefright/frameworks/av/media/libstagefright/SampleTable.cpp

@@ -395,7 +395,7 @@ status_t SampleTable::setCompositionTimeToSampleParams(
         return ERROR_MALFORMED;
     }
 
-    if (data_size != ((uint64_t)numEntries + 1) * 8) {
+    if (data_size < ((uint64_t)numEntries + 1) * 8) {
         return ERROR_MALFORMED;
     }