diff options
| author | 2023-02-14 18:43:57 +0000 | |
|---|---|---|
| committer | 2023-02-14 18:43:57 +0000 | |
| commit | aec1b4622e77084ad064446d46d85f357fa82e19 (patch) | |
| tree | 25dc3c423f943446bfe5c706a52deb323aba1bad | |
| parent | a56f46adce9b1931d9620c258fb2c4ebe162e725 (diff) | |
| parent | 146fe44f902ede302d42ab796bdb7025b46df03f (diff) | |
Merge "Camera: Avoid possible NPE after extension session configuration" am: 8c53fa447b am: 146fe44f90
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2429179
Change-Id: I127a87ec81162450232abee354081389947ee806
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | core/java/android/hardware/camera2/impl/CameraAdvancedExtensionSessionImpl.java | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/core/java/android/hardware/camera2/impl/CameraAdvancedExtensionSessionImpl.java b/core/java/android/hardware/camera2/impl/CameraAdvancedExtensionSessionImpl.java index c8dc2d0b0b91..b52daab284b5 100644 --- a/core/java/android/hardware/camera2/impl/CameraAdvancedExtensionSessionImpl.java +++ b/core/java/android/hardware/camera2/impl/CameraAdvancedExtensionSessionImpl.java @@ -528,8 +528,15 @@ public final class CameraAdvancedExtensionSessionImpl extends CameraExtensionSes boolean status = true; synchronized (mInterfaceLock) { try { - mSessionProcessor.onCaptureSessionStart(mRequestProcessor); - mInitialized = true; + if (mSessionProcessor != null) { + mSessionProcessor.onCaptureSessionStart(mRequestProcessor); + mInitialized = true; + } else { + Log.v(TAG, "Failed to start capture session, session released before " + + "extension start!"); + status = false; + mCaptureSession.close(); + } } catch (RemoteException e) { Log.e(TAG, "Failed to start capture session," + " extension service does not respond!"); |