commit 607189f2a56878cc9f35168d1fb487709b9351ac
author Dmitri Plotnikov <dplotnikov@google.com> Thu Feb 04 17:43:39 2010 -0800
committer Dmitri Plotnikov <dplotnikov@google.com> Thu Feb 04 17:43:39 2010 -0800
tree bb7e4b89bbbaa0a0d9439e309b931a0ef8d018dd
parent 1ea45f2fa000d6f35b3b2918d4d22dc5aa7b8f4f

Fixing CursorIndexOutOfBoundsException in call log UI

Bug: 2419501
Change-Id: I98bc08f60555d4d46685a95fa802d7c501737abb

src/com/android/contacts/GroupingListAdapter.java

diff --git a/src/com/android/contacts/GroupingListAdapter.java b/src/com/android/contacts/GroupingListAdapter.java
index 885007f..9d494a3 100644
--- a/src/com/android/contacts/GroupingListAdapter.java
+++ b/src/com/android/contacts/GroupingListAdapter.java
@@ -419,13 +419,20 @@
 
     public Object getItem(int position) {
         obtainPositionMetadata(mPositionMetadata, position);
-        mCursor.moveToPosition(mPositionMetadata.cursorPosition);
-        return mCursor;
+        if (mCursor.moveToPosition(mPositionMetadata.cursorPosition)) {
+            return mCursor;
+        } else {
+            return null;
+        }
     }
 
     public long getItemId(int position) {
-        getItem(position);
-        return mCursor.getLong(mRowIdColumnIndex);
+        Object item = getItem(position);
+        if (item != null) {
+            return mCursor.getLong(mRowIdColumnIndex);
+        } else {
+            return -1;
+        }
     }
 
     public View getView(int position, View convertView, ViewGroup parent) {