summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAdam Lesinski <adamlesinski@google.com>2015-10-12 22:15:50 +0000
committerAndroid (Google) Code Review <android-gerrit@google.com>2015-10-12 22:15:50 +0000
commit16b0ae105e4e94ff09d1517b68f7c66f2022a19e (patch)
tree0c9fe584a133edeaad880bfb158f52e1450e92b5
parenta2f78b44ee30839e2c047d07525dd6bc4fe50f1d (diff)
parent552a8a5d8df32f659b8d11311a244cdc6d3b7733 (diff)
downloadframeworks_native-16b0ae105e4e94ff09d1517b68f7c66f2022a19e.zip
frameworks_native-16b0ae105e4e94ff09d1517b68f7c66f2022a19e.tar.gz
frameworks_native-16b0ae105e4e94ff09d1517b68f7c66f2022a19e.tar.bz2
Merge "add number constraint for samples per MotionEvent" into mnc-dr-dev
-rw-r--r--include/input/Input.h5
-rw-r--r--libs/input/Input.cpp3
2 files changed, 7 insertions, 1 deletions
diff --git a/include/input/Input.h b/include/input/Input.h
index 4a67f47..92b3d36 100644
--- a/include/input/Input.h
+++ b/include/input/Input.h
@@ -111,6 +111,11 @@ enum {
#define MAX_POINTERS 16
/*
+ * Maximum number of samples supported per motion event.
+ */
+#define MAX_SAMPLES UINT16_MAX
+
+/*
* Maximum pointer id value supported in a motion event.
* Smallest pointer id is 0.
* (This is limited by our use of BitSet32 to track pointer assignments.)
diff --git a/libs/input/Input.cpp b/libs/input/Input.cpp
index 2c1418e..b64cb2c 100644
--- a/libs/input/Input.cpp
+++ b/libs/input/Input.cpp
@@ -424,7 +424,8 @@ void MotionEvent::transform(const float matrix[9]) {
status_t MotionEvent::readFromParcel(Parcel* parcel) {
size_t pointerCount = parcel->readInt32();
size_t sampleCount = parcel->readInt32();
- if (pointerCount == 0 || pointerCount > MAX_POINTERS || sampleCount == 0) {
+ if (pointerCount == 0 || pointerCount > MAX_POINTERS ||
+ sampleCount == 0 || sampleCount > MAX_SAMPLES) {
return BAD_VALUE;
}