Copy WindowState Bounds before modifying them

It is not safe to modify bounds returned by
getBounds, so it is important to copy them.

Change-Id: Ic19ed5e0326dc427fc6e38695fc5760980315599
Bug: 273045619
Test: build passes
Merged-In: I625afa17711fcd52bbb7fe9a9ba51500eb1bef5c

1 files changed, 1 insertions, 1 deletions

diff --git a/services/core/java/com/android/server/wm/AccessibilityWindowsPopulator.java b/services/core/java/com/android/server/wm/AccessibilityWindowsPopulator.java
index 4b8a5b7fcf56..26f781f62078 100644
--- a/services/core/java/com/android/server/wm/AccessibilityWindowsPopulator.java
+++ b/services/core/java/com/android/server/wm/AccessibilityWindowsPopulator.java
@@ -861,7 +861,7 @@ public final class AccessibilityWindowsPopulator extends WindowInfosListener {
 
         private static void getLetterBoxBounds(WindowState windowState, Region outRegion) {
             final Rect letterboxInsets = windowState.mActivityRecord.getLetterboxInsets();
-            final Rect nonLetterboxRect = windowState.getBounds();
+            final Rect nonLetterboxRect = new Rect(windowState.getBounds());
 
             nonLetterboxRect.inset(letterboxInsets);
             outRegion.set(windowState.getBounds());