ConcurrentModificationException on checking wake locks [121345393]

Fixed

Bug

Status Update

No update yet.

Description

ru...@gmail.com

created issue #1

Dec 21, 2018 01:35AM

Component used: WorkManager
Version used: alpha13
Devices/Android versions reproduced on: 5.1.1

Got this crash through crashlytics. App wasn't in background

Fatal Exception: java.util.ConcurrentModificationException
at java.util.WeakHashMap$HashIterator.next(WeakHashMap.java:165)
at androidx.work.impl.utils.WakeLocks.checkWakeLocks(WakeLocks.java:70)
at androidx.work.impl.background.systemalarm.SystemAlarmService.onAllCommandsCompleted(SystemAlarmService.java:69)
at androidx.work.impl.background.systemalarm.SystemAlarmDispatcher.dequeueAndCheckForCompletion(SystemAlarmDispatcher.java:215)
at androidx.work.impl.background.systemalarm.SystemAlarmDispatcher$DequeueAndCheckForCompletion.run(SystemAlarmDispatcher.java:315)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5675)
at java.lang.reflect.Method.invoke(Method.java)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:960)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:755)

Comments

ra...@google.com <ra...@google.com> Dec 21, 2018 07:24AM

Assigned to ra...@google.com.

ot...@gmail.com <ot...@gmail.com> #2Dec 26, 2018 01:44PM

There is a report about it in kotlinlang Slack: https://kotlinlang.slack.com/archives/C01D6HTPATV/p1706005056105739

ot...@gmail.com <ot...@gmail.com> #3Dec 26, 2018 01:45PM

Thanks for the report!

su...@google.com <su...@google.com> #4Jan 2, 2019 09:31PM

Marked as fixed.

The release notes documentation has been edited to clarify this change in behavior for line height.

To support non-standard text sizes, we encourage users to follow the Material design system and use a different type scale rather than changing the font size directly. Alternatively, users can overwrite the line height like so: style = LocalTextStyle.current.copy(lineHeight = TextUnit.Unspecified), or create a custom Typography entirely.

ru...@gmail.com <ru...@gmail.com> #5Jan 7, 2019 11:09AM

deleted

ra...@google.com <ra...@google.com> #6Jan 7, 2019 04:30PM

In my case, I have multiple font sizes in the same Text (using SpanStyle in AnnotatedString). There are legitimate reasons for this. For example, when combining Chinese and English (phonetic) together (for language-learning purposes).

ru...@gmail.com <ru...@gmail.com> #7Jan 17, 2019 03:28PM

Seems like it's still not fixed:

Fatal Exception: java.util.ConcurrentModificationException
at java.util.WeakHashMap$HashIterator.next(WeakHashMap.java:165)
at androidx.work.impl.utils.WakeLocks.checkWakeLocks(WakeLocks.java:78)
at androidx.work.impl.background.systemalarm.SystemAlarmService.onAllCommandsCompleted(SystemAlarmService.java:69)
at androidx.work.impl.background.systemalarm.SystemAlarmDispatcher.dequeueAndCheckForCompletion(SystemAlarmDispatcher.java:217)
at androidx.work.impl.background.systemalarm.SystemAlarmDispatcher$DequeueAndCheckForCompletion.run(SystemAlarmDispatcher.java:317)
at android.os.Handler.handleCallback(Handler.java:810)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:189)
at android.app.ActivityThread.main(ActivityThread.java:5532)
at java.lang.reflect.Method.invoke(Method.java)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:950)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:745)

ru...@gmail.com <ru...@gmail.com> #8Jan 17, 2019 03:29PM

Still happened on beta02, Android 5.0.2

ru...@gmail.com <ru...@gmail.com> #9Jan 17, 2019 04:52PM

Sample app here:

https://issuetracker.google.com/issues/121345393

It's hard to reproduce with SystemAlarmService.onAllCommandsCompleted. Only managed to do it once but it was just luck. The sample app crashes more often though.

ru...@gmail.com <ru...@gmail.com> #10Jan 17, 2019 04:52PM

I meant this url:

https://github.com/rubensousa/WorkManagerConcurrent

ra...@google.com <ra...@google.com> #11Jan 17, 2019 06:03PM

Status: Assigned (reopened)

Thanks for the bug report and the sample app. Will take another look.

su...@google.com <su...@google.com> #12Jan 17, 2019 09:00PM

Why are you calling a hidden method (WakeLocks#checkWakeLocks) yourself?

ru...@gmail.com <ru...@gmail.com> #13Jan 17, 2019 09:05PM

Just to reproduce the crash faster. I don't call this on the production app.

su...@google.com <su...@google.com> Jan 17, 2019 09:25PM

Reassigned to su...@google.com.

su...@google.com <su...@google.com> #14Jan 18, 2019 10:01PM

Marked as fixed.

The fix will be available in beta03:

https://android-review.googlesource.com/c/platform/frameworks/support/+/876992

ru...@gmail.com <ru...@gmail.com> #15Jan 18, 2019 11:54PM

Thanks a lot for fixing it quickly ;)

md...@gmail.com <md...@gmail.com> #16Jan 22, 2019 07:58AM

It still seems to happen for me using beta02:

java.util.WeakHashMap$HashIterator.next (WeakHashMap.java:165)
androidx.work.impl.utils.WakeLocks.checkWakeLocks (WakeLocks.java:78)
androidx.work.impl.background.systemalarm.SystemAlarmService.onAllCommandsCompleted (SystemAlarmService.java:69)
androidx.work.impl.background.systemalarm.SystemAlarmDispatcher.dequeueAndCheckForCompletion (SystemAlarmDispatcher.java:217)
androidx.work.impl.background.systemalarm.SystemAlarmDispatcher$DequeueAndCheckForCompletion.run (SystemAlarmDispatcher.java:317)
android.os.Handler.handleCallback (Handler.java:815)
android.os.Handler.dispatchMessage (Handler.java:104)
android.os.Looper.loop (Looper.java:194)
android.app.ActivityThread.main (ActivityThread.java:5624)
java.lang.reflect.Method.invoke (Method.java)
java.lang.reflect.Method.invoke (Method.java:372)
com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run (ZygoteInit.java:959)
com.android.internal.os.ZygoteInit.main (ZygoteInit.java:754)

Stacktrace is from a X5pro running Android 5.1.

ra...@google.com <ra...@google.com> #17Jan 22, 2019 06:23PM

Another fix will be available in beta03.