commit b518d000f2c4b3ae81e57bab421ad43d91406469
author Edwin Kempin <ekempin@google.com> Wed Apr 06 11:17:46 2016 +0200
committer Edwin Kempin <ekempin@google.com> Thu Apr 07 11:48:01 2016 +0200
tree 0925052a31ebf8075ea09c823027fa1076395be3
parent 7ddcb129fb9d956700daf22998d3f1e83e53f3b0

Fix potential IndexOutOfBoundsException in QueryHandler

Change-Id: I3e0ddbb2f072914292e52ccf11dc6d22020e6a5a
Signed-off-by: Edwin Kempin <ekempin@google.com>

app/src/main/java/com/google/reviewit/app/QueryHandler.java

diff --git a/app/src/main/java/com/google/reviewit/app/QueryHandler.java b/app/src/main/java/com/google/reviewit/app/QueryHandler.java
index 0a87518..bb796e6 100644
--- a/app/src/main/java/com/google/reviewit/app/QueryHandler.java
+++ b/app/src/main/java/com/google/reviewit/app/QueryHandler.java
@@ -65,7 +65,8 @@
   public List<Change> next() throws RestApiException {
     query();
     List<Change> nextChanges =
-        result.subList(page * LIMIT_QUERY, (page + 1) * LIMIT_QUERY);
+        result.subList(page * LIMIT_QUERY,
+            Math.min((page + 1) * LIMIT_QUERY, result.size()));
     page++;
     return nextChanges;
   }