RejectedExecutionException originating from VideoCapture.clearPipeline() [273713906]

Fixed

Bug

Status Update

No update yet.

Description

ri...@gmail.com

created issue #1

Mar 15, 2023 03:51PM

CAMERAX VERSION: 1.3.0-alpha03

CAMERA APPLICATION NAME AND VERSION: LinkedIn 4.1.793

ANDROID OS BUILD NUMBER: 8, 10, 11, 12, 13

DEVICE NAME: Several Samsung (A51, A11, A32..) and Xiaomi (Redmi 9) devices

DESCRIPTION:

Our crash monitoring reports a few instances of the following crash:

java.util.concurrent.RejectedExecutionException:Handler (android.os Handler) {85a71a1} is shutting down()
at androidx.camera.core.impl.utils.executor.HandlerScheduledExecutorService . HandlerScheduledExecutorService . createPostFailedException() (HandlerScheduledExecutorService.java:184)
at androidx.camera.core.impl.utils.executor.HandlerScheduledExecutorService . HandlerScheduledExecutorService . execute() (HandlerScheduledExecutorService.java:179)
at androidx.camera.core.processing.DefaultSurfaceProcessor . DefaultSurfaceProcessor . release() (DefaultSurfaceProcessor.java:148)
at androidx.camera.core.processing.SurfaceProcessorNode . SurfaceProcessorNode . release() (SurfaceProcessorNode.java:256)
at androidx.camera.video.VideoCapture . VideoCapture . clearPipeline() (VideoCapture.java:558)
at androidx.camera.video.VideoCapture . VideoCapture . onStateDetached() (VideoCapture.java:346)
at androidx.camera.camera2.internal.Camera2CameraImpl . Camera2CameraImpl . notifyStateDetachedToUseCases() (Camera2CameraImpl.java:821)
at androidx.camera.camera2.internal.Camera2CameraImpl . Camera2CameraImpl . detachUseCases() (Camera2CameraImpl.java:840)
at androidx.camera.core.internal.CameraUseCaseAdapter . CameraUseCaseAdapter . detachUseCases() (CameraUseCaseAdapter.java:366)
at androidx.camera.lifecycle.LifecycleCamera . LifecycleCamera . onStop() (LifecycleCamera.java:97)
at androidx.camera.lifecycle.LifecycleCamera . LifecycleCamera . suspend() (LifecycleCamera.java:140)
at androidx.camera.lifecycle.LifecycleCameraRepository . LifecycleCameraRepository . suspendUseCases() (LifecycleCameraRepository.java:449)
at androidx.camera.lifecycle.LifecycleCameraRepository . LifecycleCameraRepository . setInactive() (LifecycleCameraRepository.java:394)
at androidx.camera.lifecycle.LifecycleCameraRepository$LifecycleCameraRepositoryObserver . LifecycleCameraRepository . onStop() (LifecycleCameraRepository.java:522)
at java.lang.reflect.Method . Method . invoke() (Method.java:-2)
at androidx.lifecycle.ClassesInfoCache$MethodReference . ClassesInfoCache . invokeCallback() (ClassesInfoCache.java:225)
at androidx.lifecycle.ClassesInfoCache$CallbackInfo . ClassesInfoCache . invokeMethodsForEvent() (ClassesInfoCache.java:199)
at androidx.lifecycle.ClassesInfoCache$CallbackInfo . ClassesInfoCache . invokeCallbacks() (ClassesInfoCache.java:190)
at androidx.lifecycle.ReflectiveGenericLifecycleObserver . ReflectiveGenericLifecycleObserver . onStateChanged() (ReflectiveGenericLifecycleObserver.java:40)
at androidx.lifecycle.LifecycleRegistry$ObserverWithState . LifecycleRegistry . dispatchEvent() (LifecycleRegistry.java:360)
at androidx.lifecycle.LifecycleRegistry . LifecycleRegistry . backwardPass() (LifecycleRegistry.java:290)
at androidx.lifecycle.LifecycleRegistry . LifecycleRegistry . sync() (LifecycleRegistry.java:308)
at androidx.lifecycle.LifecycleRegistry . LifecycleRegistry . moveToState() (LifecycleRegistry.java:151)
at androidx.lifecycle.LifecycleRegistry . LifecycleRegistry . handleLifecycleEvent() (LifecycleRegistry.java:134)
at androidx.fragment.app.Fragment . Fragment . performStop() (Fragment.java:3318)

LIST ANY EXPERIMENTAL FEATURES: androidx.camera.view.video.ExperimentalVideo

ADDITIONAL INFORMATION: 8 external instances reported over the past week, no internal reports, not reproducible during our normal testing

CODE FRAGMENTS (this will help us troubleshoot your issues):

    onResume() {
        if (controller != null) {
            return
        }
        controller = LifecycleCameraController(activity)
        // set up target size, capture quality, etc
        controller.initializationFuture.addListener({
            try {
                controller.initializationFuture.get()
                controller.bindToLifecycle(fragment)
            } catch (e: ExecutionException) {
                // report error
            } catch (e: IllegalStateException) {
                // report error
            }
        }, mainExecutor)
    }

Comments

wu...@google.com <wu...@google.com> #2Mar 17, 2023 12:29AM

A couple of questions:

1. Have you saw crash in real device or only in simulators?
2. Do you use dynamic feature for language ID?

le...@google.com <le...@google.com> Mar 17, 2023 12:59AM

Reassigned to le...@google.com.

le...@google.com <le...@google.com> Mar 17, 2023 06:42AM

Accepted by le...@google.com.

ap...@google.com <ap...@google.com> #3Mar 21, 2023 02:07AM

Tested on Android 12 Emulator with custom executor, but cannot repro this issue.

le...@google.com <le...@google.com> #4Mar 21, 2023 10:26AM

Marked as fixed.

Second crash in the description is from a real device. Experienced it myself on two different Xiaomi phones, plus lots of crashes from users in the Google Play console.
Dynamic features are not used in the application.

As a wild guess, I have downgraded build tools from 31.0.0 to 30.0.3, compileSdk from 31 to 30, and moved all work with Language ID to the service in a separate process (just to be sure that crash can kill secondary process instead of main). This combination is in beta for 2 days by now and I don't see any SIGSEGV crashes.

pr...@google.com <pr...@google.com> #5Apr 19, 2023 06:13PM

Hmm, I feel the crash might be something related to separate/secondary process.

I also changed compileSdk and targetSDK to 31 but still cannot repro this issue.

Issue 273713906