summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Jia <wjia@google.com>2015-08-23 19:27:03 +0000
committerAndroid Git Automerger <android-git-automerger@android.com>2015-08-23 19:27:03 +0000
commitcf69487d1af3eeba724b65716a63ae13238e6e4a (patch)
tree2820d3d9bfdc72a048f8fe7949a4bbc844ee9d5d
parent719cd33e4ebc1c600772f81db28f81ccfe99ec4e (diff)
parent6f80b09f61da018d2f05e5303c0a798f9011d667 (diff)
downloadframeworks_av-cf69487d1af3eeba724b65716a63ae13238e6e4a.zip
frameworks_av-cf69487d1af3eeba724b65716a63ae13238e6e4a.tar.gz
frameworks_av-cf69487d1af3eeba724b65716a63ae13238e6e4a.tar.bz2
am 6f80b09f: am 252a8c09: am 0f3ab16c: am af7e33f6: libstagefright: check overflow before memory allocation in OMXCodec.cpp
* commit '6f80b09f61da018d2f05e5303c0a798f9011d667': libstagefright: check overflow before memory allocation in OMXCodec.cpp
-rw-r--r--media/libstagefright/OMXCodec.cpp3
1 files changed, 3 insertions, 0 deletions
diff --git a/media/libstagefright/OMXCodec.cpp b/media/libstagefright/OMXCodec.cpp
index 6da00e6..6cb0775 100644
--- a/media/libstagefright/OMXCodec.cpp
+++ b/media/libstagefright/OMXCodec.cpp
@@ -1650,6 +1650,9 @@ status_t OMXCodec::allocateBuffersOnPort(OMX_U32 portIndex) {
def.nBufferCountActual, def.nBufferSize,
portIndex == kPortIndexInput ? "input" : "output");
+ if (def.nBufferSize != 0 && def.nBufferCountActual > SIZE_MAX / def.nBufferSize) {
+ return BAD_VALUE;
+ }
size_t totalSize = def.nBufferCountActual * def.nBufferSize;
mDealer[portIndex] = new MemoryDealer(totalSize, "OMXCodec");