Inserting rows in reverse order could throw an ArrayIndexOutOfBoundsException in…

Inserting rows in reverse order could throw an ArrayIndexOutOfBoundsException in some cases (specially rows larger than the page size, which is 2 KB by default). Issue 226.

h2/src/main/org/h2/index/PageData.java

@@ -214,7 +214,7 @@ abstract class PageData extends Page {
      * @param key the key
      * @return the row
      */
-    abstract Row getRow(long key);
+    abstract Row getRowWithKey(long key);
 
     /**
      * Get the estimated memory size.

h2/src/main/org/h2/index/PageDataIndex.java

@@ -380,7 +380,7 @@ public class PageDataIndex extends PageIndex {
     }
 
     public Row getRow(Session session, long key) {
-        return getRow(key);
+        return getRowWithKey(key);
     }
 
     /**
@@ -389,9 +389,9 @@ public class PageDataIndex extends PageIndex {
      * @param key the key
      * @return the row
      */
-    public Row getRow(long key) {
+    public Row getRowWithKey(long key) {
         PageData root = getPage(rootPageId, 0);
-        return root.getRow(key);
+        return root.getRowWithKey(key);
     }
 
     PageStore getPageStore() {

h2/src/main/org/h2/index/PageDataLeaf.java

@@ -270,7 +270,7 @@ public class PageDataLeaf extends PageData {
         if (!SysProperties.OPTIMIZE_UPDATE) {
             readAllRows();
         }
-        Row r = getRow(i);
+        Row r = getRowAt(i);
         if (r != null) {
             memoryChange(false, r);
         }
@@ -278,10 +278,13 @@ public class PageDataLeaf extends PageData {
         if (entryCount < 0) {
             DbException.throwInternalError();
         }
-        freeOverflow();
-        firstOverflowPageId = 0;
-        overflowRowSize = 0;
-        rowRef = null;
+        if (firstOverflowPageId != 0) {
+            start -= 4;
+            freeOverflow();
+            firstOverflowPageId = 0;
+            overflowRowSize = 0;
+            rowRef = null;
+        }
         int keyOffsetPairLen = 2 + Data.getVarLongLen(keys[i]);
         int startNext = i > 0 ? offsets[i - 1] : index.getPageStore().getPageSize();
         int rowLength = startNext - offsets[i];
@@ -430,7 +433,7 @@ public class PageDataLeaf extends PageData {
         }
     }
 
-    Row getRow(long key) {
+    Row getRowWithKey(long key) {
         int at = find(key);
         return getRowAt(at);
     }

h2/src/main/org/h2/index/PageDataNode.java

@@ -264,10 +264,10 @@ public class PageDataNode extends PageData {
         }
     }
 
-    Row getRow(long key) {
+    Row getRowWithKey(long key) {
         int at = find(key);
         PageData page = index.getPage(childPageIds[at], getPos());
-        return page.getRow(key);
+        return page.getRowWithKey(key);
     }
 
     int getRowCount() {