pythoncapi
diff --git a/‎Lib/test/test_slice.py‎
Lines changed: 3 additions & 0 deletions b/‎Lib/test/test_slice.py‎
Lines changed: 3 additions & 0 deletions
diff --git a/‎Objects/sliceobject.c‎
Lines changed: 5 additions & 0 deletions b/‎Objects/sliceobject.c‎
Lines changed: 5 additions & 0 deletions
@@ -26,6 +26,9 @@ def test_cmp(self):
         s3 = slice(1, 2, 4)
         self.assertEqual(s1, s2)
         self.assertNotEqual(s1, s3)
+        self.assertNotEqual(s1, None)
+        self.assertNotEqual(s1, (1, 2, 3))
+        self.assertNotEqual(s1, "")
 
         class Exc(Exception):
             pass
 
@@ -286,6 +286,11 @@ slice_richcompare(PyObject *v, PyObject *w, int op)
 	PyObject *t2;
 	PyObject *res;
 
+	if (!PySlice_Check(v) || !PySlice_Check(w)) {
+		Py_INCREF(Py_NotImplemented);
+		return Py_NotImplemented;
+	}
+
 	if (v == w) {
 		/* XXX Do we really need this shortcut?
 		   There's a unit test for it, but is that fair? */