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

RESOLVED FIXED in Firefox 53

Status

()

--
critical
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: ahunt, Assigned: ahunt)

Tracking

({crash})

52 Branch
Firefox 53
Unspecified
Android
crash
Points:
---

Firefox Tracking Flags

(firefox51 unaffected, firefox52 disabled, firefox53 fixed)

Details

(Whiteboard: [MobileAS], crash signature)

Attachments

(2 attachments)

(Assignee)

Description

2 years ago
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
(Assignee)

Comment 1

2 years ago
Introduced in Bug 1312434
Assignee: nobody → ahunt
Status: NEW → ASSIGNED
Iteration: --- → 1.9
Depends on: 1312434
Whiteboard: [MobileAS]
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 4

2 years ago
mozreview-review
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 5

2 years ago
mozreview-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+

Comment 6

2 years ago
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

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c6f5da9786e9
https://hg.mozilla.org/mozilla-central/rev/399dea6e9200
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox53: --- → fixed
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
status-firefox51: --- → unaffected
status-firefox52: --- → affected
No longer depends on: 1312434
Flags: needinfo?(ahunt)
Version: 51 Branch → 52 Branch
(Assignee)

Comment 9

2 years ago
(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!
status-firefox52: affected → disabled
You need to log in before you can comment on or make changes to this bug.