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)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: liuche, Assigned: bnicholson)

References

Details

Attachments

(1 file)

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
Blocks: 709385
OS: Mac OS X → Android
Hardware: x86 → ARM
Blocks a beta blocker.
blocking-fennec1.0: --- → ?
Priority: -- → P1
Assignee: nobody → bnicholson
blocking-fennec1.0: ? → beta+
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
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: