summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLeon Scroggins III <scroggo@google.com>2015-06-10 16:21:26 +0000
committerAndroid Git Automerger <android-git-automerger@android.com>2015-06-10 16:21:26 +0000
commit3ff978a5206a257b404fafc8fe090de81960aabf (patch)
tree3e02900236fe45a833ba0cbdc4506181e7608334
parent34c7bd7c5ca5e41e151c93cd417efd7a28157e9d (diff)
parent77dde3cbd049e75ea7447d3413bf52dbfe8c6342 (diff)
downloadframeworks_base-3ff978a5206a257b404fafc8fe090de81960aabf.zip
frameworks_base-3ff978a5206a257b404fafc8fe090de81960aabf.tar.gz
frameworks_base-3ff978a5206a257b404fafc8fe090de81960aabf.tar.bz2
am 77dde3cb: am 6b16398d: am 604ae32b: am 4dae8e41: Merge "DO NOT MERGE: Ensure that unparcelling Region only reads the expected number of bytes" into lmp-dev
* commit '77dde3cbd049e75ea7447d3413bf52dbfe8c6342': DO NOT MERGE: Ensure that unparcelling Region only reads the expected number of bytes
-rw-r--r--core/jni/android/graphics/Region.cpp7
1 files changed, 6 insertions, 1 deletions
diff --git a/core/jni/android/graphics/Region.cpp b/core/jni/android/graphics/Region.cpp
index 6b99de8..ec4d8bf 100644
--- a/core/jni/android/graphics/Region.cpp
+++ b/core/jni/android/graphics/Region.cpp
@@ -218,7 +218,12 @@ static jlong Region_createFromParcel(JNIEnv* env, jobject clazz, jobject parcel)
return NULL;
}
SkRegion* region = new SkRegion;
- region->readFromMemory(regionData, size);
+ size_t actualSize = region->readFromMemory(regionData, size);
+
+ if (size != actualSize) {
+ delete region;
+ return NULL;
+ }
return reinterpret_cast<jlong>(region);
}