Closed Bug 1317631 Opened 3 years ago Closed 3 years ago

Crash in android.database.CursorIndexOutOfBoundsException: Index -1 requested, with a size of 10 at android.database.AbstractCursor.checkPosition(AbstractCursor.java)

Categories

(Firefox for Android :: Awesomescreen, defect, critical)

52 Branch
Unspecified
Android
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Firefox 53
Iteration:
1.9
Tracking Status
firefox51 --- unaffected
firefox52 --- disabled
firefox53 --- fixed

People

(Reporter: ahunt, Assigned: ahunt)

References

Details

(Keywords: crash, Whiteboard: [MobileAS])

Crash Data

Attachments

(2 files)

This bug was filed from the Socorro interface and is 
report bp-336a7a1a-35d6-4911-a5a9-bc0112161114.
=============================================================

STR:
1. Enable Activity Stream
2. Tap the "highlights" title
Introduced in Bug 1312434
Assignee: nobody → ahunt
Status: NEW → ASSIGNED
Iteration: --- → 1.9
Depends on: 1312434
Whiteboard: [MobileAS]
Comment on attachment 8811098 [details]
Bug 1317631 - Only handle clicks for highlights

https://reviewboard.mozilla.org/r/93314/#review93350
Attachment #8811098 - Flags: review?(s.kaspari) → review+
Comment on attachment 8811099 [details]
Bug 1317631 - Post: add more cursor position sanity checks

https://reviewboard.mozilla.org/r/93316/#review93352
Attachment #8811099 - Flags: review?(s.kaspari) → review+
Pushed by ahunt@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c6f5da9786e9
Only handle clicks for highlights r=sebastian
https://hg.mozilla.org/integration/autoland/rev/399dea6e9200
Post: add more cursor position sanity checks r=sebastian
https://hg.mozilla.org/mozilla-central/rev/c6f5da9786e9
https://hg.mozilla.org/mozilla-central/rev/399dea6e9200
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
Looks like this should get uplifted to 52 as well? Please request Aurora approval on the patches when you get a chance.
Blocks: 1312434
No longer depends on: 1312434
Flags: needinfo?(ahunt)
Version: 51 Branch → 52 Branch
(In reply to Ryan VanderMeulen [:RyanVM] from comment #8)
> Looks like this should get uplifted to 52 as well? Please request Aurora
> approval on the patches when you get a chance.

This is for an incomplete and currently still disabled feature that definitely won't be shipped or enabled in 52. I.e. in this case there's no need to uplift it because no user will ever run this code (there's no harm in uplifting either - just more effort). (This is part Activity Stream on Android, which will probably ride the trains in a future release - we're still actively developing and testing on nightly. In 52 that code is protected by a build flag.)

I wonder if there's some way to tag crashes that wouldn't need uplifting, would changing the Version field be enough to not make it seem like uplift is needed?
Flags: needinfo?(ahunt)
Status flags are the answer you're looking for, thanks for the info!
You need to log in before you can comment on or make changes to this bug.