Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Safe check so we don\'t move cursor beyond possible pos scope." into nyc-dev

Browse Source 
am: 2360aa3

* commit '2360aa33db829121a25be271edbaaaf26e5b093e':
  Safe check so we don't move cursor beyond possible pos scope.

Change-Id: I6794563ae70782e773bdb83390758120f930fed8
This commit is contained in:
Ben Lin
2016-04-27 20:41:21 +00:00
committed by android-build-merger
parent 0ce1ff89af 2360aa33db
commit 88dc2f020e
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
packages/DocumentsUI/src/com/android/documentsui/dirlist/Model.java
View File

@@ -149,7 +149,10 @@ public class Model {
mCursor.moveToPosition(-1); mCursor.moveToPosition(-1);
for (int pos = 0; pos < mCursorCount; ++pos) { for (int pos = 0; pos < mCursorCount; ++pos) {
mCursor.moveToNext(); if (!mCursor.moveToNext()) {
Log.e(TAG, "Fail to move cursor to next pos: " + pos);
return;
}
positions[pos] = pos; positions[pos] = pos;
// Generates a Model ID for a cursor entry that refers to a document. The Model ID is a // Generates a Model ID for a cursor entry that refers to a document. The Model ID is a