diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/ui/SelectPublicKeyFragment.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/ui/SelectPublicKeyFragment.java index eac5189ea..760485435 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/ui/SelectPublicKeyFragment.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/ui/SelectPublicKeyFragment.java @@ -174,16 +174,6 @@ public class SelectPublicKeyFragment extends RecyclerFragment onCreateLoader(int id, Bundle args) { Uri baseUri = KeyRings.buildUnifiedKeyRingsUri(); @@ -199,6 +189,9 @@ public class SelectPublicKeyFragment extends RecyclerFragment= getItemCount() || position < -1) { + if (position >= getItemCount() || position < -1) { throw new IndexOutOfBoundsException("Position: " + position + " is invalid for this data set!"); } - if(!mDataValid) { + if (!mDataValid) { throw new IllegalStateException("Attempt to move cursor over invalid data set!"); } - if(!mCursor.moveToPosition(position)) { + if (!mCursor.moveToPosition(position)) { throw new IllegalStateException("Couldn't move cursor from position: " + mCursor.getPosition() + " to position: " + position + "!"); } } public boolean moveCursor(int position) { - if(position >= getItemCount() || position < -1) { + if (position >= getItemCount() || position < -1) { Log.w(TAG, "Position: %d is invalid for this data set!"); return false; } - if(!mDataValid) { + if (!mDataValid) { Log.d(TAG, "Attempt to move cursor over invalid data set!"); } @@ -315,7 +315,7 @@ public abstract class CursorAdapter T wrap(Cursor cursor, Class type) { if (cursor != null) { @@ -356,10 +356,10 @@ public abstract class CursorAdapter 0L;