1.1.0-alpha01 crash with "Cannot invoke setValue on a background thread" [159983244]

Fixed

Bug

[AOSP] assigned

Status Update

No update yet.

Description

ma...@bokhorst.biz

created issue #1

Jun 27, 2020 06:34AM

java.lang.IllegalStateException: Cannot invoke setValue on a background thread at androidx.lifecycle.LiveData.assertMainThread(SourceFile:462) at androidx.lifecycle.LiveData.setValue(SourceFile:304) at androidx.lifecycle.MutableLiveData.setValue(SourceFile:50) at androidx.biometric.BiometricViewModel.setNegativeButtonPressPending(SourceFile:408) at androidx.biometric.BiometricViewModel$4.onClick(SourceFile:280) at android.hardware.biometrics.BiometricPrompt$1.lambda$onDialogDismissed$1(BiometricPrompt.java:231) at android.hardware.biometrics.-$$Lambda$BiometricPrompt$1$J5PqpiT8xZNiNN1gy9VraVgknaQ.run(Unknown:2) 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:764)

Comments

vi...@google.com <vi...@google.com> Jun 27, 2020 09:34AM

Assigned to vi...@google.com.

ma...@bokhorst.biz <ma...@bokhorst.biz> #2Jun 27, 2020 09:39AM

For the record: Pixel 3a XL with Android 11 beta 1.5

vi...@google.com <vi...@google.com> #3Jun 27, 2020 03:38PM

For us to further investigate this issue, please provide the following additional information:

Device used -- Device Make, Model, Android OS Version
Which device did you use to reproduce this issue?

Please provide sample project or apk to reproduce the issue. Also mention the steps to be followed for reproducing the issue with the given sample project or apk.

Expected output
What is the expected output?

Current output
What is the current output?

Android bug report (to be captured after reproducing the issue)
For steps to capture a bug report, please refer:

https://developer.android.com/studio/debug/bug-report#bugreportdevice

Alternate method
Navigate to “Developer options”, ensure “USB debugging” is enabled, then enable “Bug report shortcut”. Capture bug report by holding the power button and selecting the “Take bug report” option.

Screen Record of the Issue
Please capture screen record or video of the issue using following steps:
adb shell screenrecord /sdcard/video.mp4
Subsequently use following command to pull the recorded file:
adb pull /sdcard/video.mp4
Attach the file to this issue.

Note: Please upload the bug report and screenshot to google drive and share the folder to android-bugreport@google.com, then share the link here.

ma...@bokhorst.biz <ma...@bokhorst.biz> #4Jun 27, 2020 03:44PM

I think the crash is pretty obvious and doesn't require any additional information, so I am not going to put time into this.

ru...@gmail.com <ru...@gmail.com> #5Jun 29, 2020 09:10AM

I can't agree more "The crash is PRETTY OBVIOUS" .On emulator Pixel XL API 28 ，the bug occures each time i press negative Button which I can customize the name of it .I will upload my Demo next.You can also check my code by visiting my github repository

https://github.com/runfengai/Biometric 。

app-debug.apk

2.3 MB

Download

vi...@google.com <vi...@google.com> #6Jun 30, 2020 05:44PM

Reassigned to kc...@google.com.

Moving the issue to (androidx) > Biometric component to check this issue and provide inputs.

kc...@google.com <kc...@google.com> Jun 30, 2020 06:55PM

Reassigned to cu...@google.com.

cu...@google.com <cu...@google.com> Jul 6, 2020 08:43PM

Accepted by cu...@google.com.

eu...@gmail.com <eu...@gmail.com> #7Jul 7, 2020 11:19PM

Another instance of this was reported here: https://github.com/inorichi/tachiyomi/issues/3425

Library version: androidx.biometric:biometric:1.1.0-alpha01

Device info:

Device: Samsung Galaxy S8
Android version: 9 (SDK 28)
Android build ID: PPR1.180610.011.G950WVLS7CTC1
Device brand: samsung
Device manufacturer: samsung
Device name: dreamqltecan
Device model: SM-G950W
Device product name: dreamqltevl

ap...@google.com <ap...@google.com> #8Jul 8, 2020 07:15PM

Marked as fixed.

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

commit b525b6ebf4687dc789e4825be85a0458529d2309
Author: Curtis Belmonte <curtislb@google.com>
Date: Mon Jul 06 15:53:14 2020

Ensure biometric LiveData updates run on main thread

MutableLiveData requires value updates triggered by setValue() to run on
the main thread. Currently, the androidx.biometric library doesn't
enforce this, leading to runtime crashes. This commit changes the
behavior of BiometricViewModel to check whether the update occurs on the
main thread and, if not, to instead call postValue().

Test: Biometric integration test app on API 27-30.
Test: ./gradlew biometric:biometric:test
Test: ./gradlew biometric:biometric:connectedAndroidTest

Fixes: 159983244
Change-Id: Iec79da704651b7ce4c065b6f12471a50756e54b0

M biometric/biometric/build.gradle
M biometric/biometric/src/main/java/androidx/biometric/BiometricViewModel.java
A biometric/biometric/src/test/java/androidx/biometric/BiometricViewModelTest.java

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