diff --git a/mercurial/cext/revlog.c b/mercurial/cext/revlog.c
index 781b2720d2ac005e2806b46d8cb91abacfe4b901_bWVyY3VyaWFsL2NleHQvcmV2bG9nLmM=..a3dacabd476b6bed04932b08962d7aa2fd2fad41_bWVyY3VyaWFsL2NleHQvcmV2bG9nLmM= 100644
--- a/mercurial/cext/revlog.c
+++ b/mercurial/cext/revlog.c
@@ -158,8 +158,8 @@
 	Py_ssize_t length = index_length(self) + 1;
 	PyObject *entry;
 
-	if (pos == -1 || pos == length - 1) {
+	if (pos == -1) {
 		Py_INCREF(nullentry);
 		return nullentry;
 	}
 
@@ -162,8 +162,8 @@
 		Py_INCREF(nullentry);
 		return nullentry;
 	}
 
-	if (pos < 0 || pos >= length) {
+	if (pos < 0 || pos >= length - 1) {
 		PyErr_SetString(PyExc_IndexError, "revlog index out of range");
 		return NULL;
 	}
diff --git a/mercurial/revlog.py b/mercurial/revlog.py
index 781b2720d2ac005e2806b46d8cb91abacfe4b901_bWVyY3VyaWFsL3JldmxvZy5weQ==..a3dacabd476b6bed04932b08962d7aa2fd2fad41_bWVyY3VyaWFsL3JldmxvZy5weQ== 100644
--- a/mercurial/revlog.py
+++ b/mercurial/revlog.py
@@ -792,7 +792,7 @@
 
 class revlogoldindex(list):
     def __getitem__(self, i):
-        if i == -1 or i == len(self):
+        if i == -1:
             return (0, 0, 0, -1, -1, -1, -1, nullid)
         return list.__getitem__(self, i)
 
diff --git a/tests/test-parseindex2.py b/tests/test-parseindex2.py
index 781b2720d2ac005e2806b46d8cb91abacfe4b901_dGVzdHMvdGVzdC1wYXJzZWluZGV4Mi5weQ==..a3dacabd476b6bed04932b08962d7aa2fd2fad41_dGVzdHMvdGVzdC1wYXJzZWluZGV4Mi5weQ== 100644
--- a/tests/test-parseindex2.py
+++ b/tests/test-parseindex2.py
@@ -61,9 +61,6 @@
     e[0] = offset_type(0, type)
     index[0] = tuple(e)
 
-    # add the magic null revision at -1
-    index.append((0, 0, 0, -1, -1, -1, -1, nullid))
-
     return index, cache
 
 data_inlined = (