summaryrefslogtreecommitdiffstats
path: root/media/libstagefright/MPEG4Extractor.cpp
diff options
context:
space:
mode:
authorWei Jia <wjia@google.com>2015-10-06 20:11:33 +0000
committerAndroid Git Automerger <android-git-automerger@android.com>2015-10-06 20:11:33 +0000
commitb8f5d50cd652b4ff6121b9107b5dd7d91ca5929e (patch)
tree1b142ca7d634f14181908a04269091102ecc149f /media/libstagefright/MPEG4Extractor.cpp
parent8b3aa04a625dfd3adf68118e64da1eafec9db594 (diff)
parent3674abc050997e664c3906b8507d9d4b63d52e9d (diff)
downloadframeworks_av-b8f5d50cd652b4ff6121b9107b5dd7d91ca5929e.zip
frameworks_av-b8f5d50cd652b4ff6121b9107b5dd7d91ca5929e.tar.gz
frameworks_av-b8f5d50cd652b4ff6121b9107b5dd7d91ca5929e.tar.bz2
am 3674abc0: am fb0d63df: am 62d89908: am df68d9d5: am 60b25aa0: am 9ab9c85e: Merge "MPEG4Extractor: ensure buffer size is not less than 8 for LastCommentData." into klp-dev
* commit '3674abc050997e664c3906b8507d9d4b63d52e9d': MPEG4Extractor: ensure buffer size is not less than 8 for LastCommentData.
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 08e989b..96423e6 100644
--- a/media/libstagefright/MPEG4Extractor.cpp
+++ b/media/libstagefright/MPEG4Extractor.cpp
@@ -2355,6 +2355,12 @@ status_t MPEG4Extractor::parseITunesMetaData(off64_t offset, size_t size) {
mLastCommentName.setTo((const char *)buffer + 4);
break;
case FOURCC('d', 'a', 't', 'a'):
+ if (size < 8) {
+ delete[] buffer;
+ buffer = NULL;
+ ALOGE("b/24346430");
+ return ERROR_MALFORMED;
+ }
mLastCommentData.setTo((const char *)buffer + 8);
break;
}