Closed
Bug 743411
Opened 14 years ago
Closed 14 years ago
Cursor implementation does not conform to android.database.Cursor interface. Causes infinite loops.
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: liuche, Assigned: bnicholson)
References
Details
Attachments
(1 file)
7.59 KB,
text/plain
|
Details |
Specifically, moveToNext() needs advance the cursor even if the cursor is at the last row, because ifAfterLast() will never return true otherwise.
See android.database.Cursor interface. (http://developer.android.com/reference/android/database/Cursor.html#moveToNext%28%29)
Infinite loop triggers when iterating through cursor values.
Code that triggers infinite looping: https://github.com/mozilla-services/android-sync/blob/liuche/cursor-infinite/src/main/java/org/mozilla/gecko/sync/repositories/android/PasswordsRepositorySession.java#L623
Updated•14 years ago
|
OS: Mac OS X → Android
Hardware: x86 → ARM
Updated•14 years ago
|
Assignee: nobody → bnicholson
blocking-fennec1.0: ? → beta+
Reporter | ||
Comment 2•14 years ago
|
||
Found bug in the sync code, causing infinite loops, apologies for the stir.
Status: NEW → RESOLVED
blocking-fennec1.0: beta+ → ---
Closed: 14 years ago
Priority: P1 → --
Resolution: --- → INVALID
Updated•5 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•