summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Leon Scroggins <scroggo@google.com> 2022-01-13 14:40:58 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2022-01-13 14:40:58 +0000
commitd4c6a656d265504207e79fde2fcd988548cf4c37 (patch)
treed286e172f9e3aa9802ceb7b1734b22dc51f34941
parentf261da004a94303bc904971c2242d1294f6dab8a (diff)
parentae16b80b779db790e100ebe6e9d292b96dc04c9c (diff)
Merge "RegionSamplingListener: Treat empty Rect as BAD_VALUE"
-rw-r--r--services/surfaceflinger/SurfaceFlinger.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 3d6bcb712b..64a18c6d5c 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -1605,7 +1605,7 @@ status_t SurfaceFlinger::getCompositionPreference(
status_t SurfaceFlinger::addRegionSamplingListener(const Rect& samplingArea,
const sp<IBinder>& stopLayerHandle,
const sp<IRegionSamplingListener>& listener) {
- if (!listener || samplingArea == Rect::INVALID_RECT) {
+ if (!listener || samplingArea == Rect::INVALID_RECT || samplingArea.isEmpty()) {
return BAD_VALUE;
}