commit 157c74e7ace77ae026cb59f48e400fb4ffe3c529
author Andreas Huber <andih@google.com> Tue Jun 21 12:13:41 2011 -0700
committer Android (Google) Code Review <android-gerrit@google.com> Tue Jun 21 12:13:41 2011 -0700
tree 88c53126f431f503958332ec4c8b813505056422
parent bbf7c449e11258b676343a09463329c7cb6375c4
parent 727863d2de17ea121ad0bcccdf2227d13a3ec9ca

Merge "Don't assert on malformed .mp4 content that contains fewer samples than"

media/libstagefright/SampleTable.cpp

diff --git a/media/libstagefright/SampleTable.cpp b/media/libstagefright/SampleTable.cpp
index eb135ab..a8a094e 100644
--- a/media/libstagefright/SampleTable.cpp
+++ b/media/libstagefright/SampleTable.cpp
@@ -416,12 +416,16 @@
         uint32_t delta = mTimeToSample[2 * i + 1];
 
         for (uint32_t j = 0; j < n; ++j) {
-            CHECK(sampleIndex < mNumSampleSizes);
+            if (sampleIndex < mNumSampleSizes) {
+                // Technically this should always be the case if the file
+                // is well-formed, but you know... there's (gasp) malformed
+                // content out there.
 
-            mSampleTimeEntries[sampleIndex].mSampleIndex = sampleIndex;
+                mSampleTimeEntries[sampleIndex].mSampleIndex = sampleIndex;
 
-            mSampleTimeEntries[sampleIndex].mCompositionTime =
-                sampleTime + getCompositionTimeOffset(sampleIndex);
+                mSampleTimeEntries[sampleIndex].mCompositionTime =
+                    sampleTime + getCompositionTimeOffset(sampleIndex);
+            }
 
             ++sampleIndex;
             sampleTime += delta;