MultiParagraph.getBoundingBox cannot find a valid offset in its ParagraphInfoList [356080063]

Assigned

Bug

Status Update

No update yet.

Description

ha...@google.com

created issue #1

Jul 29, 2024 09:58AM

This issue was originally reported by Meta. The stackrace is the only information we have on the crash.

android_crash:java.lang.ArrayIndexOutOfBoundsException:androidx.compose.ui.text.MultiParagraph.getBoundingBox
stack_trace:	java.lang.ArrayIndexOutOfBoundsException: length=10; index=-2
	at java.util.ArrayList.get(ArrayList.java:439)
	at androidx.compose.ui.text.MultiParagraph.getBoundingBox(MultiParagraph.kt:508) [inlined]
	at androidx.compose.ui.text.TextLayoutResult.getBoundingBox(TextLayoutResult.kt:527)
	at androidx.compose.ui.platform.AndroidComposeViewAccessibilityDelegateCompat.addExtraDataToAccessibilityNodeInfoHelper(AndroidComposeViewAccessibilityDelegateCompat.android.kt:1994)
	at androidx.compose.ui.platform.AndroidComposeViewAccessibilityDelegateCompat.access$addExtraDataToAccessibilityNodeInfoHelper(AndroidComposeViewAccessibilityDelegateCompat.android.kt:134) [inlined]
	at androidx.compose.ui.platform.AndroidComposeViewAccessibilityDelegateCompat$ComposeAccessibilityNodeProvider.addExtraDataToAccessibilityNodeInfo(AndroidComposeViewAccessibilityDelegateCompat.android.kt:3404)
	at android.view.AccessibilityInteractionController$AccessibilityNodePrefetcher.prefetchAccessibilityNodeInfos(AccessibilityInteractionController.java:1104)
	at android.view.AccessibilityInteractionController.findAccessibilityNodeInfoByAccessibilityIdUiThread(AccessibilityInteractionController.java:342)
	at android.view.AccessibilityInteractionController.access$400(AccessibilityInteractionController.java:75)
	at android.view.AccessibilityInteractionController$PrivateHandler.handleMessage(AccessibilityInteractionController.java:1460)
	at android.os.Handler.dispatchMessage(Handler.java:106)
	at android.os.Looper.loop(Looper.java:223)
	at android.app.ActivityThread.main(ActivityThread.java:7948)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:603)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)

Comments

cl...@google.com <cl...@google.com> #2Aug 1, 2024 12:39PM

P1 as this is causing crashes in production

cl...@google.com <cl...@google.com> #3Aug 2, 2024 01:16PM

Can we have someone assigned to look at this please?

ha...@google.com <ha...@google.com> #4Aug 2, 2024 02:10PM

Reassigned to ha...@google.com.

I spent some time on this to understand how the final values in the crash report could come to be. Honestly I'm drawing a blank. All the leading code paths are guarded against any configuration that can produce these values. I'm thinking that there's a somewhat complex AnnotatedString calculation with a couple of ParagraphStyles but I can't think of any particular scenario that leads to this crash.

ha...@google.com <ha...@google.com> #5Aug 2, 2024 02:12PM

We need the reporter to provide more information about the screen or any kind of complex AnnotatedString construction that they are doing.

ha...@google.com <ha...@google.com> #6Aug 5, 2024 11:18AM

se...@google.com <se...@google.com> #7Aug 27, 2024 04:37PM

Tracked P1

gr...@google.com <gr...@google.com> #8Sep 10, 2024 06:40PM

Any updates here?

ha...@google.com <ha...@google.com> #9Sep 24, 2024 09:38AM

I requested more info from the original reporter again. Nothing else changed on our side after comment#4

ha...@google.com <ha...@google.com> #10Oct 2, 2024 10:35AM

Meta's last response was "we didn't have much information on this issue, after the 1.7 stable version, the crash number is very very small". So I'm lowering the priority on this bug to P2 as long as we don't get more information.

ap...@google.com <ap...@google.com> #11Oct 22, 2024 02:57PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Anastasia Soboleva <soboleva@google.com>
Link: https://android-review.googlesource.com/3317132

Add more debug info into findParagraphByIndex

Expand for full commit details

Add more debug info into findParagraphByIndex

Bug: 356080063, 374115892
Test: N/A
Change-Id: I9d7eef189bb5e8aa5a9b56e8fc2642a2634a2a45

Files:

M compose/ui/ui-text/src/commonMain/kotlin/androidx/compose/ui/text/MultiParagraph.kt

Hash: d24dad0b0db0cc7bf4c6062f40fadd38c04d4d43
Date: Tue Oct 22 14:28:01 2024

so...@google.com <so...@google.com> #12Oct 25, 2024 07:59AM

Reassigned to so...@google.com.

This might be the same root cause as in b/374115892, will verify