commit d4c6a656d265504207e79fde2fcd988548cf4c37
author Leon Scroggins <scroggo@google.com> Thu Jan 13 14:40:58 2022 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Thu Jan 13 14:40:58 2022 +0000
tree d286e172f9e3aa9802ceb7b1734b22dc51f34941
parent f261da004a94303bc904971c2242d1294f6dab8a
parent ae16b80b779db790e100ebe6e9d292b96dc04c9c

Merge "RegionSamplingListener: Treat empty Rect as BAD_VALUE"

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 3d6bcb7..64a18c6 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -1605,7 +1605,7 @@
 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;
     }