commit 2df30e5cee2565c74145af421b1fd6fc9a743c84
author Ben Lin <linben@google.com> Fri Apr 22 16:12:50 2016 -0700
committer Ben Lin <linben@google.com> Fri Apr 22 16:13:08 2016 -0700
tree 52efb731d67943d8daab4e72441b7f09fb1ba1f3
parent 63aed6ad1150bba639538998ae758189c23fe048

Safe check so we don't move cursor beyond possible pos scope.

Bug: 28278372
Change-Id: I7c66b5229ae3a6a1b99bdf0f02bf1f0b8a1c79b2

src/com/android/documentsui/dirlist/Model.java

diff --git a/src/com/android/documentsui/dirlist/Model.java b/src/com/android/documentsui/dirlist/Model.java
index e2a28ad..ab45a9f 100644
--- a/src/com/android/documentsui/dirlist/Model.java
+++ b/src/com/android/documentsui/dirlist/Model.java
@@ -163,7 +163,10 @@
 
         mCursor.moveToPosition(-1);
         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;
 
             // Generates a Model ID for a cursor entry that refers to a document. The Model ID is a