summaryrefslogtreecommitdiffstats
path: root/media/libstagefright/OMXCodec.cpp
diff options
context:
space:
mode:
authorChad Brubaker <cbrubaker@google.com>2015-08-11 15:21:02 -0700
committerSteve Kondik <steve@cyngn.com>2015-11-05 21:16:18 -0800
commit41cd3f51d445ea4e5f9984910e6eb95cdec1438a (patch)
tree51f882b29c028f9accb890e63447251fbb88b3c7 /media/libstagefright/OMXCodec.cpp
parent86ea14b97299e54addac4e5227b05c67a129be57 (diff)
downloadframeworks_av-41cd3f51d445ea4e5f9984910e6eb95cdec1438a.zip
frameworks_av-41cd3f51d445ea4e5f9984910e6eb95cdec1438a.tar.gz
frameworks_av-41cd3f51d445ea4e5f9984910e6eb95cdec1438a.tar.bz2
Fix benign unsigned integer overflow
Bug:23110888 Change-Id: Ibd6f1c7a9204cf584536c1ee63f6a8e5c6e4aa59 (cherry picked from commit 69c1e038717d3462ea5991425ba009f0aed148d2)
Diffstat (limited to 'media/libstagefright/OMXCodec.cpp')
-rw-r--r--media/libstagefright/OMXCodec.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/media/libstagefright/OMXCodec.cpp b/media/libstagefright/OMXCodec.cpp
index 8d0bccd..194edd5 100644
--- a/media/libstagefright/OMXCodec.cpp
+++ b/media/libstagefright/OMXCodec.cpp
@@ -2689,7 +2689,8 @@ status_t OMXCodec::freeBuffersOnPort(
status_t stickyErr = OK;
- for (size_t i = buffers->size(); i-- > 0;) {
+ for (size_t i = buffers->size(); i > 0;) {
+ i--;
BufferInfo *info = &buffers->editItemAt(i);
if (onlyThoseWeOwn && info->mStatus == OWNED_BY_COMPONENT) {