CameraXBasic crash on Android 9.1 / InvocationTargetException [216667482]

Fixed

Bug

Status Update

No update yet.

Description

ma...@sensoriainc.com

created issue #1

Jan 28, 2022 01:33AM

Please note: This component is for the CameraX API used in Jetpack. Please DO NOT file Pixel Camera issues here.

Please describe your issue and include details such as the version of CameraX you are using and any relevant logs related to your issue.

// If at all possible, capture an Android logcat (

https://developer.android.com/studio/command-line/logcat) when you're experiencing the issue, preferably while the camera is still active.

CAMERAX VERSION 1.1.0-beta01, cameraview:1.0.0-alpha27 (NOTE: repro also on previous versions per official

https://github.com/android/camera-samples)

CAMERA APPLICATION NAME AND VERSION: CameraXSample

ANDROID OS BUILD NUMBER: (Settings > About > Build number) Android V9.1 / DM30_LEMP_V1.6_2021_0726

DEVICE NAME: (Nexus 5X, Samsung S6, etc) LEMP (an Android Smart Watch)

DESCRIPTION:
I can successfully run Camera2Basic

When I run CameraXBasic, I can observe a series of internal/managed exceptions up to this code:

/** Initialize CameraX, and prepare to bind the camera use cases */
private fun setUpCamera() {
val cameraProviderFuture = ProcessCameraProvider.getInstance(requireContext())
cameraProviderFuture.addListener(Runnable {

// CameraProvider
cameraProvider = cameraProviderFuture.get() <<<<<<<<<<

After this call, I get a java.lang.reflect.InvocationTargetException crashing the app.

See below for detailed logcat.

LIST ANY EXPERIMENTAL FEATURES: (As an example - @ExperimentalCamera2Interop)
None

STEPS TO REPRODUCE:
1. Launch CameraXBasic

OBSERVED RESULTS:
CRASH

EXPECTED RESULTS:
NOT TO CRASH :)

REPRODUCIBILITY: (5 of 5, 1 of 100, etc) 100%

ADDITIONAL INFORMATION:

Logcat in attachment. You can observe the set of internal exceptions up to before the call to cameraProvideFuture.get(), and the fatal crash.

CODE FRAGMENTS (this will help us troubleshoot your issues):
The crash happens at line 248 of CameraFragment.kt.

/** Initialize CameraX, and prepare to bind the camera use cases */
private fun setUpCamera() {
val cameraProviderFuture = ProcessCameraProvider.getInstance(requireContext())
cameraProviderFuture.addListener(Runnable {

// CameraProvider
cameraProvider = cameraProviderFuture.get()

CameraXBasic-crash-logcat.log

85 KB

View

Download

Comments

er...@google.com <er...@google.com> Jan 31, 2022 09:59PM

Reassigned to tr...@google.com.

tr...@google.com <tr...@google.com> #2Jan 31, 2022 11:51PM

Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.

ma...@sensoriainc.com <ma...@sensoriainc.com> #3Feb 1, 2022 12:02AM

Great! Thanks a lot, I'll look for the live updates soon!

ma...@sensoriainc.com <ma...@sensoriainc.com> #4Feb 1, 2022 11:32PM

When I add the log line above, the app crashes with this exception:

E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.android.example.camera2.basic, PID: 9034
java.nio.BufferUnderflowException
at java.nio.Buffer.nextGetIndex(Buffer.java:515)
at java.nio.HeapByteBuffer.get(HeapByteBuffer.java:111)
at android.hardware.camera2.marshal.impl.MarshalQueryableBoolean$MarshalerBoolean.unmarshal(MarshalQueryableBoolean.java:45)
at android.hardware.camera2.marshal.impl.MarshalQueryableBoolean$MarshalerBoolean.unmarshal(MarshalQueryableBoolean.java:32)
at android.hardware.camera2.impl.CameraMetadataNative.getBase(CameraMetadataNative.java:532)
at android.hardware.camera2.impl.CameraMetadataNative.get(CameraMetadataNative.java:404)
at android.hardware.camera2.impl.CameraMetadataNative.get(CameraMetadataNative.java:373)
at android.hardware.camera2.CameraCharacteristics.get(CameraCharacteristics.java:223)
at com.example.android.camera2.basic.fragments.CameraFragment$initializeCamera$1.invokeSuspend(CameraFragment.kt:201)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:56)
at android.os.Handler.handleCallback(Handler.java:873)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:193)
at android.app.ActivityThread.main(ActivityThread.java:6865)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:504)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)

tr...@google.com <tr...@google.com> #5Feb 3, 2022 07:17PM

Reassigned to ra...@google.com.

Thank you for trying that out. This seems to suggest there is a problem with the camera implementation on this device. If we can get a device in-hand, we may be able to put a workaround into CameraX.

ma...@sensoriainc.com <ma...@sensoriainc.com> #6Feb 3, 2022 07:50PM

Thanks for the feedback.
I can ship a loaner to you for repro and workaround.

Would that be feasible?

tr...@google.com <tr...@google.com> #7Feb 4, 2022 09:29PM

Thanks for the offer. We're attempting to source a device from our side. We'll keep this bug updated with progress.

ma...@sensoriainc.com <ma...@sensoriainc.com> #8Feb 11, 2022 09:43PM

do you have an update? Again, I can provide a loaner device.

tr...@google.com <tr...@google.com> #9Feb 12, 2022 12:03AM

We have a device in-transit and should arrive soon.

In the mean time, do you happen to have any other LEMFO devices besides the LEMP? If so, have you seen similar issues with these devices?

ma...@sensoriainc.com <ma...@sensoriainc.com> #10Feb 14, 2022 08:09PM

I don't have other types of LEMFO devices at hand

ma...@sensoriainc.com <ma...@sensoriainc.com> #11Feb 22, 2022 07:55PM

Hello. were you able to get the device in hand?

tr...@google.com <tr...@google.com> #12Feb 22, 2022 10:59PM

Accepted by tr...@google.com.

I just received the device today and have been able to reproduce the error. I'll keep this bug updated on progress of the workaround.

ap...@google.com <ap...@google.com> #13Feb 28, 2022 07:37PM

Project: platform/frameworks/support
Branch: androidx-main

commit 8873fdc0218c1440fc125cbd28403d385e365f7a
Author: Trevor McGuire <trevormcguire@google.com>
Date: Thu Feb 24 17:01:21 2022

Flash availability BufferUnderflowException quirk

Adds quirk and workaround for devices that throw a
BufferUnderflowExceptionwhile checking for flash availablility.
This allows the device to function without crashing when creating
TorchControl.

Relnote: "Workaround included for devices that crash when checking
availability of flash. These devices will not have torch available."
Bug: 216667482
Test: Manual testing of CameraXBasic and Core Test App with
problem/non-problem devices.

Change-Id: If26a3e436ce12dc82451f07ce49969ea237017c0

A camera/camera-camera2/src/main/java/androidx/camera/camera2/internal/compat/quirk/FlashAvailabilityBufferUnderflowQuirk.java
M camera/camera-camera2/src/main/java/androidx/camera/camera2/internal/TorchControl.java
M camera/camera-camera2/src/main/java/androidx/camera/camera2/internal/Camera2CameraInfoImpl.java
M camera/camera-camera2/src/main/java/androidx/camera/camera2/internal/compat/quirk/DeviceQuirksLoader.java
A camera/camera-camera2/src/main/java/androidx/camera/camera2/internal/compat/workaround/FlashAvailabilityChecker.java

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

tr...@google.com <tr...@google.com> #14Feb 28, 2022 08:31PM

Marked as fixed.

This is now fixed and should be part of the next release. If you'd like to try it out before the next release, you can use this snapshot build. See androidx.dev for more information on using snapshot builds.

ma...@sensoriainc.com <ma...@sensoriainc.com> #15Feb 28, 2022 08:49PM

Thanks Trevor.
is there a log for when the expected release will be?

tr...@google.com <tr...@google.com> #16Mar 1, 2022 11:20PM

I can't give an exact date on the release, but we try to keep to ~monthly release cadence. Since we just released on the 23rd, it will likely be sometime near the end of next month, barring major release blockers. You can get an idea of the release cadence from the release notes here