unhandled Camera2 exception triggered inside CameraX primarily on Jio JioPhone Next but also some other devices [231701345]

Fixed

Bug

Status Update

No update yet.

Description

da...@gmail.com

created issue #1

May 7, 2022 11:58AM

We're receiving these crash reports for our app on the Play Store (https://play.google.com/store/apps/details?id=app.grapheneos.camera.play). I'm unsure why it's happening, but perhaps you can get one of these devices and reproduce it. Ideally, CameraX could at least start catching this exception and turning it into an error surfaced to the app.

Jio JioPhone Next (Qualcomm QM215)
Samsung Galaxy A02s (Qualcomm SDM450)
Vivo V2039 (Qualcomm SDM439)

We're mostly seeing these crashes from the Jio JioPhone Next which is a very widely used, very cheap device in India. The crash traceback is nearly identical across the around 5 device models where we've seen it with only minor variations in Camera2 implementation line numbers. All of the impacted devices are Android 11.

java.lang.AssertionError: 
  at android.hardware.camera2.marshal.impl.MarshalQueryableRange$MarshalerRange.unmarshal (MarshalQueryableRange.java:95)
  at android.hardware.camera2.marshal.impl.MarshalQueryableRange$MarshalerRange.unmarshal (MarshalQueryableRange.java:37)
  at android.hardware.camera2.impl.CameraMetadataNative.getBase (CameraMetadataNative.java:584)
  at android.hardware.camera2.impl.CameraMetadataNative.get (CameraMetadataNative.java:444)
  at android.hardware.camera2.impl.CameraMetadataNative.get (CameraMetadataNative.java:413)
  at android.hardware.camera2.CameraCharacteristics.get (CameraCharacteristics.java:258)
  at androidx.camera.camera2.internal.compat.CameraCharacteristicsBaseImpl.get (CameraCharacteristicsBaseImpl.java)
  at androidx.camera.camera2.internal.compat.CameraCharacteristicsCompat.get (CameraCharacteristicsCompat.java)
  at androidx.camera.camera2.internal.ZoomControl.isAndroidRZoomSupported (ZoomControl.java)
  at androidx.camera.camera2.internal.ZoomControl.createZoomImpl (ZoomControl.java)
  at androidx.camera.camera2.internal.ZoomControl.<init> (ZoomControl.java:17)
  at androidx.camera.camera2.internal.Camera2CameraControlImpl.<init> (Camera2CameraControlImpl.java)
  at androidx.camera.camera2.internal.Camera2CameraImpl.<init> (Camera2CameraImpl.java)
  at androidx.camera.camera2.internal.Camera2CameraFactory.getCamera (Camera2CameraFactory.java)
  at androidx.camera.core.impl.CameraRepository.init (CameraRepository.java:49)
  at androidx.camera.core.CameraX.lambda$initAndRetryRecursively$2 (CameraX.java)
  at androidx.camera.camera2.internal.compat.CameraCaptureSessionCompat$CaptureCallbackExecutorWrapper$$InternalSyntheticLambda$8$44f3b3505978b6ed93f62d918bccdd3aaac62bd10e5c81e454c5e6523140bd24$0.run$bridge (CameraCaptureSessionCompat.java)
  at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1167)
  at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:641)
  at java.lang.Thread.run (Thread.java:923)
Caused by: java.lang.reflect.InvocationTargetException: 
  at java.lang.reflect.Constructor.newInstance0 (Native Method)
  at java.lang.reflect.Constructor.newInstance (Constructor.java:343)
  at android.hardware.camera2.marshal.impl.MarshalQueryableRange$MarshalerRange.unmarshal (MarshalQueryableRange.java:87)
  at android.hardware.camera2.marshal.impl.MarshalQueryableRange$MarshalerRange.unmarshal (MarshalQueryableRange.java:37)
  at android.hardware.camera2.impl.CameraMetadataNative.getBase (CameraMetadataNative.java:584)
  at android.hardware.camera2.impl.CameraMetadataNative.get (CameraMetadataNative.java:444)
  at android.hardware.camera2.impl.CameraMetadataNative.get (CameraMetadataNative.java:413)
  at android.hardware.camera2.CameraCharacteristics.get (CameraCharacteristics.java:258)
  at androidx.camera.camera2.internal.compat.CameraCharacteristicsBaseImpl.get (CameraCharacteristicsBaseImpl.java)
  at androidx.camera.camera2.internal.compat.CameraCharacteristicsCompat.get (CameraCharacteristicsCompat.java)
  at androidx.camera.camera2.internal.ZoomControl.isAndroidRZoomSupported (ZoomControl.java)
  at androidx.camera.camera2.internal.ZoomControl.createZoomImpl (ZoomControl.java)
  at androidx.camera.camera2.internal.ZoomControl.<init> (ZoomControl.java:17)
  at androidx.camera.camera2.internal.Camera2CameraControlImpl.<init> (Camera2CameraControlImpl.java)
  at androidx.camera.camera2.internal.Camera2CameraImpl.<init> (Camera2CameraImpl.java)
  at androidx.camera.camera2.internal.Camera2CameraFactory.getCamera (Camera2CameraFactory.java)
  at androidx.camera.core.impl.CameraRepository.init (CameraRepository.java:49)
  at androidx.camera.core.CameraX.lambda$initAndRetryRecursively$2 (CameraX.java)
  at androidx.camera.camera2.internal.compat.CameraCaptureSessionCompat$CaptureCallbackExecutorWrapper$$InternalSyntheticLambda$8$44f3b3505978b6ed93f62d918bccdd3aaac62bd10e5c81e454c5e6523140bd24$0.run$bridge (CameraCaptureSessionCompat.java)
  at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1167)
  at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:641)
  at java.lang.Thread.run (Thread.java:923)
Caused by: java.lang.IllegalArgumentException: 
  at android.util.Range.<init> (Range.java:59)
  at java.lang.reflect.Constructor.newInstance0 (Native Method)
  at java.lang.reflect.Constructor.newInstance (Constructor.java:343)
  at android.hardware.camera2.marshal.impl.MarshalQueryableRange$MarshalerRange.unmarshal (MarshalQueryableRange.java:87)
  at android.hardware.camera2.marshal.impl.MarshalQueryableRange$MarshalerRange.unmarshal (MarshalQueryableRange.java:37)
  at android.hardware.camera2.impl.CameraMetadataNative.getBase (CameraMetadataNative.java:584)
  at android.hardware.camera2.impl.CameraMetadataNative.get (CameraMetadataNative.java:444)
  at android.hardware.camera2.impl.CameraMetadataNative.get (CameraMetadataNative.java:413)
  at android.hardware.camera2.CameraCharacteristics.get (CameraCharacteristics.java:258)
  at androidx.camera.camera2.internal.compat.CameraCharacteristicsBaseImpl.get (CameraCharacteristicsBaseImpl.java)
  at androidx.camera.camera2.internal.compat.CameraCharacteristicsCompat.get (CameraCharacteristicsCompat.java)
  at androidx.camera.camera2.internal.ZoomControl.isAndroidRZoomSupported (ZoomControl.java)
  at androidx.camera.camera2.internal.ZoomControl.createZoomImpl (ZoomControl.java)
  at androidx.camera.camera2.internal.ZoomControl.<init> (ZoomControl.java:17)
  at androidx.camera.camera2.internal.Camera2CameraControlImpl.<init> (Camera2CameraControlImpl.java)
  at androidx.camera.camera2.internal.Camera2CameraImpl.<init> (Camera2CameraImpl.java)
  at androidx.camera.camera2.internal.Camera2CameraFactory.getCamera (Camera2CameraFactory.java)
  at androidx.camera.core.impl.CameraRepository.init (CameraRepository.java:49)
  at androidx.camera.core.CameraX.lambda$initAndRetryRecursively$2 (CameraX.java)
  at androidx.camera.camera2.internal.compat.CameraCaptureSessionCompat$CaptureCallbackExecutorWrapper$$InternalSyntheticLambda$8$44f3b3505978b6ed93f62d918bccdd3aaac62bd10e5c81e454c5e6523140bd24$0.run$bridge (CameraCaptureSessionCompat.java)
  at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1167)
  at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:641)
  at java.lang.Thread.run (Thread.java:923)

Comments

da...@gmail.com <da...@gmail.com> #2May 7, 2022 05:21PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit a071470cde57cf7eb6e39a67d331d716a0f8b842
Author: Ian Lake <ilake@google.com>
Date: Wed Jan 15 14:01:37 2020

Add SpecialEffectsController infrastructure

"Special Effects" is the blanket term for
Animation, Animator, framework Transition, and
AndroidX Transition that Fragments currently support.

Add the infrastructure around a SpecialEffectsController
which would operate on the container (i.e., ViewGroup)
level and coordinate all of the special effects.

This will eventually allow us to track the
postponed state at the container level.

Currently all APIs are purposefully package private
as they should not be used by external developers
at this time.

BUG: 147749580
Test: new test suites pass
Change-Id: I1e209bb4f55c21aa01afb49035d41f5b8d7d3eca

A fragment/fragment/src/androidTest/java/androidx/fragment/app/DefaultSpecialEffectsControllerTest.kt
A fragment/fragment/src/androidTest/java/androidx/fragment/app/SpecialEffectsControllerTest.kt
A fragment/fragment/src/main/java/androidx/fragment/app/DefaultSpecialEffectsController.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentManager.java
A fragment/fragment/src/main/java/androidx/fragment/app/SpecialEffectsController.java
A fragment/fragment/src/main/java/androidx/fragment/app/SpecialEffectsControllerFactory.java
M fragment/fragment/src/main/res/values/ids.xml
M jetifier/jetifier/migration.config

https://android-review.googlesource.com/1209367

wu...@google.com <wu...@google.com> #3May 9, 2022 02:53AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit d990bffa6728272e139e1b5d58e764e9197b1adc
Author: Ian Lake <ilake@google.com>
Date: Thu Jan 16 14:05:14 2020

Find the appropriate SpecialEffectsControllerFactory

Instead of requiring each call to
getOrCreateController() require passing in the
factory, use findFragmentManager() to find the
containing FragmentManager and get the factory from
it.

Test: new test passes
BUG: 147749580
Change-Id: I44f747ba84feb2911a8d0811e40d1b46644429a5

M fragment/fragment/src/androidTest/java/androidx/fragment/app/DefaultSpecialEffectsControllerTest.kt
M fragment/fragment/src/main/java/androidx/fragment/app/SpecialEffectsController.java

https://android-review.googlesource.com/1210675

sc...@google.com <sc...@google.com> #4May 9, 2022 03:31AM

Reassigned to mi...@google.com.

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 62e13263d9ee368a255cfb7db73eba061ca85f9c
Author: Ian Lake <ilake@google.com>
Date: Fri Jan 17 13:09:12 2020

Allow cancellation of special effects operations

Pipe through a CancellationSignal to Operation
so that a SpecialEffectsController can handle
cancellation.

Test: new test passes
BUG: 147749580
Change-Id: Iff9525a3d3e079a3b0b6818e43e6efdb16207a7a

M fragment/fragment/src/androidTest/java/androidx/fragment/app/SpecialEffectsControllerTest.kt
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentStateManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/SpecialEffectsController.java

https://android-review.googlesource.com/1212445

sh...@google.com <sh...@google.com> #5May 9, 2022 05:03PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 9ea0884726565254629ff5c99ec919f56e0af1d9
Author: Ian Lake <ilake@google.com>
Date: Fri Jan 17 11:09:14 2020

Allow enqueuing special effects operations

Build the initial infrastructure for enqueuing
ADD and REMOVE operations to a SpecialEffectsController.

This currently doesn't change any runtime behavior
since DefaultSpecialEffectsController doesn't actually
do anything and FragmentStateManager isn't used to
move Fragments beyond CREATED with moveToExpectedState()
in production code.

Cancellation will be handled in a follow up CL.

Test: new SpecialEffectsControllerTest tests pass
BUG: 147749580
Change-Id: I6a3009de80419fd878f2f838bf8e234c20b933ae

M fragment/fragment/src/androidTest/java/androidx/fragment/app/SpecialEffectsControllerTest.kt
M fragment/fragment/src/main/java/androidx/fragment/app/DefaultSpecialEffectsController.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentStateManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/SpecialEffectsController.java

https://android-review.googlesource.com/1212444

da...@gmail.com <da...@gmail.com> #6May 9, 2022 05:17PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit d67221893efc073415220d4d3635f159e728b905
Author: Ian Lake <ilake@google.com>
Date: Tue Jan 21 16:49:50 2020

Add isAwaitingCompletion() API

Build the infrastructure for allowing the
FragmentStateManager to check whether a
SpecialEffectsController is still awaiting
completion of queued up / executing
operations.

Test: updated tests to call new API
BUG: 147749580
Change-Id: I6ebd563c505dc2e7b65ced4932316986c478a8cf

M fragment/fragment/src/androidTest/java/androidx/fragment/app/SpecialEffectsControllerTest.kt
M fragment/fragment/src/main/java/androidx/fragment/app/SpecialEffectsController.java

https://android-review.googlesource.com/1214213

da...@gmail.com <da...@gmail.com> #7May 9, 2022 05:18PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 974d9e0f0f1ca4108fa653302c239f9025dcd0d9
Author: Ian Lake <ilake@google.com>
Date: Fri Feb 07 15:46:17 2020

Add animations to DefaultSpecialEffectsController

Build the base for running Animations / Animators
in DefaultSpecialEffectsController, correctly only
calling complete() when they finish.

Test: visual inspection, comparing the code to the existing code
BUG: 147749580
Change-Id: Ib0eaeb34bfac7ca5e32b3feea9a2730388dcfd60

M fragment/fragment/src/main/java/androidx/fragment/app/DefaultSpecialEffectsController.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentAnim.java

https://android-review.googlesource.com/1230307

sc...@google.com <sc...@google.com> #8May 10, 2022 04:26AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit d08aad2c5c5da3127969d4243e93ea8f0e8eccd3
Author: Ian Lake <ilake@google.com>
Date: Tue Feb 11 15:36:30 2020

Add initial infrastructure for transitions

Allow DefaultSpecialEffectsController to
start framework or AndroidX transitions.

This just pipes the Fragment APIs through to
beginDelayedTransition(). It does not currently
handle shared element transitions.

Test: visual inspection, comparing the code to the existing code
BUG: 147749580
Change-Id: Ieb9b9de9bafd59d4a6b9036df389cff263cd737e

M fragment/fragment/src/main/java/androidx/fragment/app/DefaultSpecialEffectsController.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentTransition.java

https://android-review.googlesource.com/1232694

tr...@google.com <tr...@google.com> #9May 12, 2022 01:32AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 9c458049e7543914de98760102017b10dd04068e
Author: Ian Lake <ilake@google.com>
Date: Tue Feb 18 12:57:03 2020

Add cancelAllOperations() to SpecialEffectsController

Provide an API for cancelling all operations
(both pending and being executed) for a
SpecialEffectsController.

Test: new tests pass
BUG: 147749580
Change-Id: Id90f270869470bd44d73163a84343cf2cda3cd47

M fragment/fragment/src/androidTest/java/androidx/fragment/app/SpecialEffectsControllerTest.kt
M fragment/fragment/src/main/java/androidx/fragment/app/SpecialEffectsController.java

https://android-review.googlesource.com/1238926

sc...@google.com <sc...@google.com> #10May 12, 2022 03:25AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 4d3bac24c5a77c5ab5624795afa709b3f265bf74
Author: Ian Lake <ilake@google.com>
Date: Tue Feb 25 15:47:06 2020

Add infrastructure to fully switch to FragmentStateManager

Add the initial logic to fully replace all usages of
moveToState() with FragmentStateManager and its
moveToExpectedState().

Enabling this flag currently causes many tests to fail
(due to work not being completed on

b/147749580

), but
this infrastructure will allow us to confirm our fixes
are actually applied while iterating through those
outstanding issues.

Test: tests all pass with USE_STATE_MANAGER = false
BUG: 139536619
Change-Id: I84340091ce1b0bcc3e26051e25a12d7b38b860c4

M fragment/fragment/src/main/java/androidx/fragment/app/BackStackRecord.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentStore.java

https://android-review.googlesource.com/1243715

mi...@google.com <mi...@google.com> #11May 12, 2022 04:00AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 56a9dc625ea8a9f35569549b4c2f24c1fbc7ff48
Author: Ian Lake <ilake@google.com>
Date: Mon Mar 02 15:32:16 2020

Add infrastructure for awaiting enter/exit effects

Add new states for awaiting enter effects
(which block going to RESUMED) and awaiting
exit effects (which block going to CREATED).

Changed isAwaitingCompletion() to
getAwaitingCompletionType() to know what
type of Operation is uncompleted.

Test: existing tests pass
BUG: 147749580
Change-Id: Ifc28f62dfaadbcd222e3f2d8cf4a21e7eacf6091

M fragment/fragment/src/androidTest/java/androidx/fragment/app/SpecialEffectsControllerTest.kt
M fragment/fragment/src/main/java/androidx/fragment/app/Fragment.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentStateManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/SpecialEffectsController.java

https://android-review.googlesource.com/1248227

ap...@google.com <ap...@google.com> #12May 18, 2022 04:06AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 5a7e88f9d605f0a735d922e73463d1c603683e22
Author: Ian Lake <ilake@google.com>
Date: Thu Mar 05 16:02:57 2020

Don't call postpone when using FragmentStateManager

Avoid calling through to postponePostponableTransactions
at all when USE_STATE_MANAGER is true as it will
be responsible for taking care of postponement in
the future.

Tightened up the state management in FragmentStateManager
to avoid crashes in the test suite due to bad
transitions between states.

Test: all tests pass with USE_STATE_MANAGER=false
Test: test suite doesn't crash with USE_STATE_MANAGER=true
BUG: 147749580
Change-Id: I6ff2c3fbe325c82ea5e33dd5c0cb07c908bbd613

M fragment/fragment/src/main/java/androidx/fragment/app/Fragment.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentStateManager.java

https://android-review.googlesource.com/1252296

mi...@google.com <mi...@google.com> #13May 18, 2022 04:59AM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 20090a3bd0d8392a165f56b3a9da4efd1a86875c
Author: Ian Lake <ilake@google.com>
Date: Mon Mar 09 15:04:48 2020

Start transitions before starting animations

Ensure that transitions start before animations
so that they properly capture the current state.

Test: existing tests pass
BUG: 147749580
Change-Id: Icc5bfae9b60554225307dadb278d7c2700527dfd

M fragment/fragment/src/main/java/androidx/fragment/app/DefaultSpecialEffectsController.java

https://android-review.googlesource.com/1253639

da...@gmail.com <da...@gmail.com> #14May 18, 2022 05:27AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit ea897029344b96b61a86de8a0607d307c281ab3d
Author: Ian Lake <ilake@google.com>
Date: Mon Mar 09 16:44:41 2020

Correctly add/remove view using SpecialEffectsController

Ensure that DefaultSpecialEffectsController
correctly applies shows the View or removes
the View based on the appropriate
operation type.

Test: existing tests pass
Test: more tests pass when using USE_STATE_MANAGER
BUG: 147749580
Change-Id: I5dc190adc85a357a74862e25567769575dd135ef

M fragment/fragment/src/main/java/androidx/fragment/app/DefaultSpecialEffectsController.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentStateManager.java

https://android-review.googlesource.com/1253010

mi...@google.com <mi...@google.com> #15May 18, 2022 07:55AM

Adding on to this, I get a `Trying to set a replacement startPostponedEnterTransition on {fragment}` when performing the following behavior:

Quickly open two (new) fragment instances of the same fragment, and exit/open them repeatedly. For example, via a double click, or simply calling a method that creates a fragment transaction to show the fragment twice. After opening/closing the said fragment several times, the exception will occur. I believe this has to do with the aforementioned issues with the FragmentManager getting into a unpredictable state.
Some extra things to be noted:
- I use `setReorderingAllowed(true)` and `replace` in the Fragment transaction
- I call `postponeEnterTransition()` in `onCreate` of the Fragment
- I call `startPostponedEnterTransition()` after `onViewCreated`, in the Observer of a LiveData. The LiveData's `setValue` is called before the fragment transaction, so it should be called as soon as the livedata observer is attached and observing.
- The Fragment that is being instantiated and opened twice is opened from another Fragment that follows the same steps, such that the stack looks like so: Fragment A is added, the user clicks on a button to open Fragment B, and two new instances of Fragment B are added to the stack.

Some notable behavior that occurs: After opening Fragment B, Fragment A and two instances of Fragment B are on the stack. Upon pressing the "back" button, all three fragments are "popped" from the stack. When setting `setReorderingAllowed(false)`, only one of the instances of Fragment B are popped from the stack, which is expected.

If there are any more details or clarity I can provide, let me know, as this is a bit of a strange/obscure bug.

ju...@google.com <ju...@google.com> #16Dec 7, 2022 06:36PM

Re #15 - having a sample project that reproduces your issue would be very helpful in confirming that it no longer occurs in the new infrastructure.