Crash: IndexOutOfBoundsException: 77, -75

sentry-io[bot] commented 4 years ago

Sentry Issue: https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/

IndexOutOfBoundsException: 77, -75
    at android.text.PackedIntVector.deleteAt(PackedIntVector.java:218)
    at android.text.DynamicLayout.reflow(DynamicLayout.java:311)
    at android.text.DynamicLayout.-wrap0(DynamicLayout.java)
    at android.text.DynamicLayout$ChangeWatcher.reflow(DynamicLayout.java:662)
    at android.text.DynamicLayout$ChangeWatcher.onSpanChanged(DynamicLayout.java:691)
...
(16 additional frame(s) were not displayed)

77, -75

loremattei commented 4 years ago

Also: https://sentry.io/share/issue/81b90713f6fe4893b463255d6895b21b/

designsimply commented 4 years ago

@loremattei can these reports be merged in Sentry?

Combined stats:

30-day impact: ~60 per day Users affected: 640 in the last 30d Last seen in: 12.7 (latest major point release at the time of this report)

shiki commented 4 years ago

I tried to reproduce this for an hour but ran out of time. It seems to be related to Aztec. The logs say that Gutenberg is turned off. Based on the crash log, I guessed and tried:

Copy, cutting, pasting with varying text
Typing the same content
Emojis
State restoration

No luck. 😞

mzorz commented 4 years ago

Opened a PR aiming at fixing these reports here https://github.com/wordpress-mobile/AztecEditor-Android/pull/834

designsimply commented 4 years ago

90-day impact: ~18 per day Users affected in the last 90 days: 366 Limited to: Android 7.1.2 and below

^{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

designsimply commented 4 years ago

Noting that this should be checked on in the next Groundskeeping rotation and nudge for replies to the open PR as needed.

designsimply commented 4 years ago

90-day impact: ~22 per day Users affected in the last 90 days: 344 Limited to: Android 7.1.2 and below

^{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

mzorz commented 4 years ago

Limited to: Android 7.1.2 and below

Just a note that this also affects Android 8.0 as can be seen in this issue (don't know why Sentry takes these two as different issues) - first observed in https://github.com/wordpress-mobile/WordPress-Android/issues/9904#issuecomment-503021638

mzorz commented 4 years ago

Closing via https://github.com/wordpress-mobile/AztecEditor-Android/pull/834 - will monitor the performance of the fix and re-open this issue if it doesn't improve our numbers

mzorz commented 4 years ago

Reopening as per the reasons explained in reverting PR https://github.com/wordpress-mobile/AztecEditor-Android/pull/882 cc @designsimply

designsimply commented 4 years ago

IndexOutOfBoundsException: 704, -1
android.text.PackedIntVector in deleteAt

Users affected in the last 90 days: 258 Limited to: Android 7.1.2 and below _{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

IndexOutOfBoundsException: 126, -123
android.text.PackedIntVector in deleteAt

Users affected in the last 90 days: 209 Limited to: Android 8.0.0 _{https://sentry.io/share/issue/81b90713f6fe4893b463255d6895b21b/}

designsimply commented 4 years ago

IndexOutOfBoundsException: 704, -1
android.text.PackedIntVector in deleteAt

Users affected in the last 90 days: 228 Limited to: Android 7.1.2 and below _{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

IndexOutOfBoundsException: 126, -123
android.text.PackedIntVector in deleteAt

Users affected in the last 90 days: 168 Limited to: Android 8.0.0 _{https://sentry.io/share/issue/81b90713f6fe4893b463255d6895b21b/}

jd-alexander commented 4 years ago

I spent some time today trying to reproduce this error but I wasn't able to. However, I found something that happened in the previous version of Android that caused this. When the soft hyphen Unicode character was used it would cause a crash. Eg. https://github.com/andstatus/andstatus/issues/334 but it was on Android 6.0 and when I used the soft hyphen e.g. '\u00AD' with Aztec on API 23, 26 & 29 it works well. Nonetheless, I am hoping that this will somehow help to figure out what's causing this 🙏

designsimply commented 4 years ago

Users affected in the last 90 days: 157 Limited to: Android 7.1.2 and below _{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

Users affected in the last 90 days: 96 Limited to: Android 8.0.0 _{https://sentry.io/share/issue/81b90713f6fe4893b463255d6895b21b/}

10 and 2 events, respectively, have been tracked for this crash in 14.6.1 since it was released 9d ago on Apr 22.

designsimply commented 3 years ago

13 events have been tracked for this crash in 14.7 since it was released 15d ago on May 4.

Limited to: Android 7.1.2 and below Events in the last 90d: 3,700 Users affected in the last 90d: 148 _{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

3 events have been tracked for this crash in 14.7 since it was released 15d ago on May 4.

Limited to: Android 8.0.0 Events in the last 90d: 2,900 Users affected in the last 90d: 79 _{https://sentry.io/share/issue/81b90713f6fe4893b463255d6895b21b/}

designsimply commented 3 years ago

Events in the last 90d: 3,700 Events in the last 9d since 14.8 was released: 3 Users affected in the last 90d: 135 Also see: 90d graph _{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

Events in the last 90d: 2,900 Events in the last 9d since 14.8 was released: 7 Users affected in the last 90d: 80 Also see: 90d graph _{https://sentry.io/share/issue/81b90713f6fe4893b463255d6895b21b/}

designsimply commented 3 years ago

Events in the last 90d: 3,800 Users affected in the last 90d: 127 _{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

Events in the last 90d: 3,000 Users affected in the last 90d: 66 _{https://sentry.io/share/issue/81b90713f6fe4893b463255d6895b21b/}

designsimply commented 3 years ago

Events in the last 90d: 4,000 Users affected in the last 90d: 92 Limited to: Android 6.x & 7.x _{https://sentry.io/share/issue/25c5482018784560b498f2fddb147974/}

Events in the last 90d: 3,100 Users affected in the last 90d: 63 Limited to: Android 8.0.0 _{https://sentry.io/share/issue/81b90713f6fe4893b463255d6895b21b/}

Slight decrease in users affected overall in these two issues and it's still happening in latest releases but only in lower versions of Android, however, there are more issues with similar errors noted which are likely related. A search for IndexOutOfBoundsException shows a downward trend but still with pretty high numbers per day (~73/day) recently in the latest stable release (15.3) and Android 10.

designsimply commented 3 years ago

Seeing a slight decrease in frequency on a search for IndexOutOfBoundsException in Sentry. Still happening in the latest release, 15.8.1.

designsimply commented 3 years ago

Another slight decrease but daily numbers are still comparatively high and it's still happening in the latest release.

sentry-io[bot] commented 3 years ago

Sentry issue: WORDPRESS-ANDROID-2M

sentry-io[bot] commented 3 years ago

Sentry issue: WORDPRESS-ANDROID-ED

sentry-io[bot] commented 3 years ago

Sentry issue: WORDPRESS-ANDROID-6G

sentry-io[bot] commented 3 years ago

Sentry issue: WORDPRESS-ANDROID-83

sentry-io[bot] commented 3 years ago

Sentry issue: WORDPRESS-ANDROID-83

malinajirka commented 3 years ago

I've looked into this issue. Unfortunately, I wasn't able to reproduce it or pin down the root cause.

What I did:

linked issues I believe are related
compared the stacktraces to other similar crashes
created a query in sentry called "IndexOutOfBounds - PackedIntVector in deleteAt, Aztec Android 5-8" so we can check the trends
I tried to reproduce the issue both manually and using monkey but I failed

Findings

this crash is limited to Android 5-8
this crash is limited to Aztec (doesn't happen in Gutenberg)
Some interesting notes can be found here

A search for IndexOutOfBoundsException shows a downward trend but still with pretty high numbers per day (~73/day)

I think IndexOutOfBoundsException is too generic. It aggregates a bunch of unrelated crashes. I tried to create a query which might more closely show the trend of this issue. It's in the Discover section (in Sentry). The query is called "IndexOutOfBounds - PackedIntVector in deleteAt, Aztec Android 5-8".

Considering the following:

the issue seems to be hidden in the OS (perhaps even just in a custom ROM)
happens only on Android 8 and older, so it should disappear in time
happens only in Aztec, so it should disappear in time
number of affected users doesn't seem to be too high (+- 10 per day)

I'd personally just keep checking the trends and ignore this issue. Wdyt?

designsimply commented 3 years ago

+1 for continuing to check trends for now based on how it's limited to Android 5-8 and Aztec. I'll remove it from the prioritized list for now. 👍

designsimply commented 3 years ago

A search for message:android.text.PackedIntVector in deleteAt in Sentry Discover returned 1,316 in the last 90 days:

designsimply commented 3 years ago

Numbers feel steady here, and they are lower than other crashes we are tracking so let's close this one in favor of working on the biggest crashes as priority. Please definitely re-open if you feel it needs to be checked again or if you spot an upward trend in these!

A search for message:android.text.PackedIntVector in deleteAt in Sentry Discover returned 1,311 in the last 90 days.

wordpress-mobile / WordPress-Android

Crash: IndexOutOfBoundsException: 77, -75 #9904