py/sequence: Fix boundary errors when slicing with a negative step.

dpgeorge · dpgeorge · commit eb4c37f7a47f · 2017-05-18T17:32:42.000+10:00
diff --git a/py/sequence.c b/py/sequence.c
@@ -88,15 +88,22 @@ bool mp_seq_get_fast_slice_indexes(mp_uint_t len, mp_obj_t slice, mp_bound_slice
     if (start < 0) {
         start = len + start;
         if (start < 0) {
-            start = 0;
+            if (indexes->step < 0) {
+                start = -1;
+            } else {
+                start = 0;
+            }
         }
     } else if (indexes->step > 0 && (mp_uint_t)start > len) {
         start = len;
-    } else if (indexes->step < 0 && (mp_uint_t)start > len - 1) {
+    } else if (indexes->step < 0 && (mp_uint_t)start >= len) {
         start = len - 1;
     }
     if (stop < 0) {
         stop = len + stop;
+        if (stop < 0) {
+            stop = -1;
+        }
         if (indexes->step < 0) {
             stop += 1;
         }
