summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlanker <i@flanker017.me>2015-10-22 02:15:13 +0000
committerandroid-build-merger <android-build-merger@google.com>2015-10-22 02:15:13 +0000
commit9830f11dc6b6aa4001c79147966fdb9eaa71c2f2 (patch)
tree4d1415bf1ae0610016bc2b0db65324b7838e2eda
parent7b4863f606bdb48b188b0235447dfaba6c8c3cf0 (diff)
parent90dc6dc825ebd1f0f4c9474fad887cd7eac8e9bd (diff)
downloadframeworks_native-9830f11dc6b6aa4001c79147966fdb9eaa71c2f2.zip
frameworks_native-9830f11dc6b6aa4001c79147966fdb9eaa71c2f2.tar.gz
frameworks_native-9830f11dc6b6aa4001c79147966fdb9eaa71c2f2.tar.bz2
add number constraint for samples per MotionEvent am: 5d17838ade am: 72c8ca4a01 am: 9964d83869
am: 90dc6dc825 * commit '90dc6dc825ebd1f0f4c9474fad887cd7eac8e9bd': add number constraint for samples per MotionEvent
-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 c360f63..b067a6b 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 3a7afe9..cd55ee5 100644
--- a/libs/input/Input.cpp
+++ b/libs/input/Input.cpp
@@ -421,7 +421,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;
}