summaryrefslogtreecommitdiffstats
path: root/media/libstagefright/MPEG4Extractor.cpp
diff options
context:
space:
mode:
authorMarco Nelissen <marcone@google.com>2013-11-25 08:39:56 -0800
committerMarco Nelissen <marcone@google.com>2013-11-25 08:39:56 -0800
commit465127fa3dfba81d8ad2a164ba6036478f71c8a6 (patch)
tree5c66eaa8374e5153ff7c8a5677d44a2a821002e1 /media/libstagefright/MPEG4Extractor.cpp
parent11e715589e0f3f96389d0eedab677369da680fd4 (diff)
downloadframeworks_av-465127fa3dfba81d8ad2a164ba6036478f71c8a6.zip
frameworks_av-465127fa3dfba81d8ad2a164ba6036478f71c8a6.tar.gz
frameworks_av-465127fa3dfba81d8ad2a164ba6036478f71c8a6.tar.bz2
Fix SIGABRT when playing mp4 file
If the track duration was shorter than the segment duration, the calculated encoder padding would be negative, resulting in a crash. b/11823061 https://code.google.com/p/android/issues/detail?id=62610 Change-Id: I3989ad88caea38d212b61355c15aec13382c6116
Diffstat (limited to 'media/libstagefright/MPEG4Extractor.cpp')
-rw-r--r--media/libstagefright/MPEG4Extractor.cpp6
1 files changed, 6 insertions, 0 deletions
diff --git a/media/libstagefright/MPEG4Extractor.cpp b/media/libstagefright/MPEG4Extractor.cpp
index d0b6185..13e22d6 100644
--- a/media/libstagefright/MPEG4Extractor.cpp
+++ b/media/libstagefright/MPEG4Extractor.cpp
@@ -963,6 +963,12 @@ status_t MPEG4Extractor::parseChunk(off64_t *offset, int depth) {
mLastTrack->meta->setInt32(kKeyEncoderDelay, delay);
int64_t paddingus = duration - (segment_duration + media_time);
+ if (paddingus < 0) {
+ // track duration from media header (which is what kKeyDuration is) might
+ // be slightly shorter than the segment duration, which would make the
+ // padding negative. Clamp to zero.
+ paddingus = 0;
+ }
int64_t paddingsamples = (paddingus * samplerate + 500000) / 1000000;
mLastTrack->meta->setInt32(kKeyEncoderPadding, paddingsamples);
}