1265118 - crash in java.lang.ArrayIndexOutOfBoundsException: length=12; index=-1 at java.util.ArrayList.get(ArrayList.java)

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: General, defect)

Description

[Chenxia Liu [:liuche]]

This bug was filed from the Socorro interface and is 
report bp-f3738901-e317-492c-93f0-6c6cb2160415.
=============================================================

Crash reported when tapping on the first synced item in the History panel.

I'll have to investigate this further since I can't reproduce it.

What's happening is the index of the item that was clicked is being passed to showChildView, but when it's being used to get() the client from a list, the index is -1 causing an ArrayIndexOutOfBounds error. This is a little confusing to me because the index shouldn't be 0 when it's passed from the click listener, and it's not modified until it's being used as a list index [1], where it's -1.

I'll investigate this further.

http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/home/CombinedHistoryAdapter.java#188

Comment 1

[Chenxia Liu [:liuche]]

Whoops, typo above, "the index shouldn't be * -1 * when it's passed from the click listener" (it should be 0 in this case).

Comment 2

[Chenxia Liu [:liuche]]

I haven't seen any other reports, and it seems like there was only the one crash report - the others in the CrashReporter were pre-48. Can reopen if this resurfaces.