commit 315bc0971539f97aaf424c6b21b789a7c8ca9e04
author Eino-Ville Talvala <etalvala@google.com> Fri Aug 21 16:30:27 2015 -0700
committer Eino-Ville Talvala <etalvala@google.com> Fri Aug 21 16:31:58 2015 -0700
tree 75f63d02c31f58871df88899459eaaee67c1b1c5
parent cb59747de7222bc7310162c4174f7b460c34adef

Camera2: Legacy: Lock down surface sizes before creating EGL surfaces

This ensures that sizes don't change after being initially inspected,
and/or stale size information isn't reused incorrectly.

Bug: 22092273
Change-Id: I5b26ea53977da4c1412e476d06d94aa791846393

core/java/android/hardware/camera2/legacy/LegacyCameraDevice.java

diff --git a/core/java/android/hardware/camera2/legacy/LegacyCameraDevice.java b/core/java/android/hardware/camera2/legacy/LegacyCameraDevice.java
index e786707..fddfbde 100644
--- a/core/java/android/hardware/camera2/legacy/LegacyCameraDevice.java
+++ b/core/java/android/hardware/camera2/legacy/LegacyCameraDevice.java
@@ -338,6 +338,8 @@
                     } else {
                         sizedSurfaces.add(new Pair<>(output, s));
                     }
+                    // Lock down the size before configuration
+                    setSurfaceDimens(output, s.getWidth(), s.getHeight());
                 } catch (BufferQueueAbandonedException e) {
                     Log.e(TAG, "Surface bufferqueue is abandoned, cannot configure as output: ", e);
                     return BAD_VALUE;