Fix reference leak noted in test_types:

Check for a[:] = a _before_ calling PySequence_Fast on a.
release23-maint candidate
Reference leak doesn't happen with head of release22-maint.

1 files changed, 9 insertions, 9 deletions

diff --git a/Objects/listobject.c b/Objects/listobject.c
index b059420..ed28200 100644
--- a/Objects/listobject.c
+++ b/Objects/listobject.c
@@ -472,15 +472,6 @@ list_ass_slice(PyListObject *a, int ilow, int ihigh, PyObject *v)
 		n = 0;
 	else {
 		char msg[256];
-		PyOS_snprintf(msg, sizeof(msg),
-			      "must assign sequence"
-			      " (not \"%.200s\") to slice",
-			      v->ob_type->tp_name);
-		v_as_SF = PySequence_Fast(v, msg);
-		if(v_as_SF == NULL)
-			return -1;
-		n = PySequence_Fast_GET_SIZE(v_as_SF);
-
 		if (a == b) {
 			/* Special case "a[i:j] = a" -- copy b first */
 			int ret;
@@ -491,6 +482,15 @@ list_ass_slice(PyListObject *a, int ilow, int ihigh, PyObject *v)
 			Py_DECREF(v);
 			return ret;
 		}
+
+		PyOS_snprintf(msg, sizeof(msg),
+			      "must assign sequence"
+			      " (not \"%.200s\") to slice",
+			      v->ob_type->tp_name);
+		v_as_SF = PySequence_Fast(v, msg);
+		if(v_as_SF == NULL)
+			return -1;
+		n = PySequence_Fast_GET_SIZE(v_as_SF);
 	}
 	if (ilow < 0)
 		ilow = 0;