summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Leon Scroggins III <scroggo@google.com> 2015-05-27 17:52:51 +0000
committer Android Git Automerger <android-git-automerger@android.com> 2015-05-27 17:52:51 +0000
commit557ee34d97c435e19034235aff258aea093efc3f (patch)
tree276bed53acce189d6985f15448f6f07c1cd63b11
parent0f7712407d96a847845b10eef34d9a259c687533 (diff)
parent2adefefa3576142767c45b84d97e05078ead3539 (diff)
am 2adefefa: am b6e3f930: am 2bbcc253: am b36ace74: Merge "Check that the parcel contained the expected amount of region data. DO NOT MERGE" into lmp-dev
* commit '2adefefa3576142767c45b84d97e05078ead3539': Check that the parcel contained the expected amount of region data. DO NOT MERGE
-rw-r--r--core/jni/android/graphics/Region.cpp8
1 files changed, 6 insertions, 2 deletions
diff --git a/core/jni/android/graphics/Region.cpp b/core/jni/android/graphics/Region.cpp
index 912968a9da3f..6b99de85a248 100644
--- a/core/jni/android/graphics/Region.cpp
+++ b/core/jni/android/graphics/Region.cpp
@@ -212,9 +212,13 @@ static jlong Region_createFromParcel(JNIEnv* env, jobject clazz, jobject parcel)
android::Parcel* p = android::parcelForJavaObject(env, parcel);
+ const size_t size = p->readInt32();
+ const void* regionData = p->readInplace(size);
+ if (regionData == NULL) {
+ return NULL;
+ }
SkRegion* region = new SkRegion;
- size_t size = p->readInt32();
- region->readFromMemory(p->readInplace(size), size);
+ region->readFromMemory(regionData, size);
return reinterpret_cast<jlong>(region);
}