summaryrefslogtreecommitdiffstats
path: root/media/libstagefright/foundation
diff options
context:
space:
mode:
authorVishwath Mohan <vishwath@google.com>2015-08-11 17:23:50 -0700
committerSteve Kondik <steve@cyngn.com>2015-11-05 21:16:18 -0800
commitfefd7d1974ad75ae99c2b4bbbbdee6146d133ab6 (patch)
tree92f4b4a352aed2d7f40c807df51c49cfe1d319c8 /media/libstagefright/foundation
parent41cd3f51d445ea4e5f9984910e6eb95cdec1438a (diff)
downloadframeworks_av-fefd7d1974ad75ae99c2b4bbbbdee6146d133ab6.zip
frameworks_av-fefd7d1974ad75ae99c2b4bbbbdee6146d133ab6.tar.gz
frameworks_av-fefd7d1974ad75ae99c2b4bbbbdee6146d133ab6.tar.bz2
Fix benign unsigned integer overflow
Fixes a benign overflow error in ANetworkSession.cpp Bug: 23110888 Change-Id: Ic64f5100ab1e3f2ffe247ca4f7b40d38c1ff0c2a (cherry picked from cfe302317c873001b0b35352d9f4f52c39f6f4b7)
Diffstat (limited to 'media/libstagefright/foundation')
-rw-r--r--media/libstagefright/foundation/ANetworkSession.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/media/libstagefright/foundation/ANetworkSession.cpp b/media/libstagefright/foundation/ANetworkSession.cpp
index b230400..4bcb1f6 100644
--- a/media/libstagefright/foundation/ANetworkSession.cpp
+++ b/media/libstagefright/foundation/ANetworkSession.cpp
@@ -1318,7 +1318,8 @@ void ANetworkSession::threadLoop() {
List<sp<Session> > sessionsToAdd;
- for (size_t i = mSessions.size(); res > 0 && i-- > 0;) {
+ for (size_t i = mSessions.size(); res > 0 && i > 0;) {
+ i--;
const sp<Session> &session = mSessions.valueAt(i);
int s = session->socket();
@@ -1409,4 +1410,3 @@ void ANetworkSession::threadLoop() {
}
} // namespace android
-