Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit da887965 authored by Avichal Rakesh's avatar Avichal Rakesh Committed by Automerger Merge Worker
Browse files

Merge "camera: Fix exception handling from ImageReader#detach" into tm-d1-dev... 

Merge "camera: Fix exception handling from ImageReader#detach" into tm-d1-dev am: 00d8fb50 am: f7be5a5d am: 6bb32af3

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/19139699



Change-Id: I58d6043bc3fb6a912736797abc51ac4556490c53
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 838cd583 6bb32af3

core/java/android/hardware/camera2/impl/CameraExtensionSessionImpl.java

+31 −3
Original line number Diff line number Diff line
@@ -1737,6 +1737,20 @@ public final class CameraExtensionSessionImpl extends CameraExtensionSession { 
                } catch (IllegalStateException e) {

                    // This is possible in case the client disconnects from the output surface

                    // abruptly.

                    Log.w(TAG, "Output surface likely abandoned, dropping buffer!");

                    img.close();

                } catch (RuntimeException e) {

                    // NOTE: This is intended to catch RuntimeException from ImageReader.detachImage

                    // ImageReader.detachImage is not supposed to throw RuntimeExceptions but the

                    // bug went unchecked for a few years and now its behavior cannot be changed

                    // without breaking backwards compatibility.



                    if (!e.getClass().equals(RuntimeException.class)) {

                        // re-throw any exceptions that aren't base RuntimeException since they are

                        // coming from elsewhere, and we shouldn't silently drop those.

                        throw e;

                    }



                    Log.w(TAG, "Output surface likely abandoned, dropping buffer!");

                    img.close();

                }
@@ -1773,9 +1787,23 @@ public final class CameraExtensionSessionImpl extends CameraExtensionSession { 
                }

                try {

                    reader.detachImage(img);

                } catch (Exception e) {

                    Log.e(TAG,

                            "Failed to detach image!");

                } catch (IllegalStateException e) {

                    Log.e(TAG, "Failed to detach image!");

                    img.close();

                    return;

                } catch (RuntimeException e) {

                    // NOTE: This is intended to catch RuntimeException from ImageReader.detachImage

                    // ImageReader.detachImage is not supposed to throw RuntimeExceptions but the

                    // bug went unchecked for a few years and now its behavior cannot be changed

                    // without breaking backwards compatibility.



                    if (!e.getClass().equals(RuntimeException.class)) {

                        // re-throw any exceptions that aren't base RuntimeException since they are

                        // coming from elsewhere, and we shouldn't silently drop those.

                        throw e;

                    }



                    Log.e(TAG, "Failed to detach image!");

                    img.close();

                    return;

                }