commit 635bc8f90429b2fdcaf7f8d43f7f59bcd0fe951c
author Dan Albert <danalbert@google.com> Fri May 08 10:43:54 2015 -0700
committer Dan Albert <danalbert@google.com> Fri May 08 10:43:54 2015 -0700
tree 5f597e27cd9494cc74779d5f7c8c68e814e89686
parent 9ba21b9418d10ddcc39f08901e24fbf43d82b2bc

Fix comparison sign warnings.

Change-Id: I6f2e2b03b968a569b122004b4803c5d17fccfb12

media/libstagefright/MPEG4Extractor.cpp

diff --git a/media/libstagefright/MPEG4Extractor.cpp b/media/libstagefright/MPEG4Extractor.cpp
index fd1722b..5c0afa9 100644
--- a/media/libstagefright/MPEG4Extractor.cpp
+++ b/media/libstagefright/MPEG4Extractor.cpp
@@ -2018,10 +2018,10 @@
             *offset += chunk_size;
 
             if (mFileMetaData != NULL) {
-                ALOGV("chunk_data_size = %lld and data_offset = %lld",
-                        (long long)chunk_data_size, (long long)data_offset);
+                ALOGV("chunk_data_size = %" PRId64 " and data_offset = %" PRId64,
+                      chunk_data_size, data_offset);
 
-                if (chunk_data_size >= SIZE_MAX - 1) {
+                if (chunk_data_size < 0 || static_cast<uint64_t>(chunk_data_size) >= SIZE_MAX - 1) {
                     return ERROR_MALFORMED;
                 }
                 sp<ABuffer> buffer = new ABuffer(chunk_data_size + 1);