Move len boundary assertions before using len.

Anastasia Belova · Anastasia Belova · commit da720c10f6d8 · 2025-03-20T16:22:40.000+03:00
diff --git a/Modules/_io/bytesio.c b/Modules/_io/bytesio.c
@@ -583,9 +583,9 @@ _io_BytesIO_readinto_impl(bytesio *self, Py_buffer *buffer)
             len = 0;
     }
 
-    memcpy(buffer->buf, PyBytes_AS_STRING(self->buf) + self->pos, len);
     assert(self->pos + len < PY_SSIZE_T_MAX);
     assert(len >= 0);
+    memcpy(buffer->buf, PyBytes_AS_STRING(self->buf) + self->pos, len);
     self->pos += len;
 
     return PyLong_FromSsize_t(len);

Original file line number	Diff line number	Diff line change
`@@ -583,9 +583,9 @@ _io_BytesIO_readinto_impl(bytesio self, Py_buffer buffer)`
`583`	`583`	`len = 0;`
`584`	`584`	`}`
`585`	`585`
`586`		`- memcpy(buffer->buf, PyBytes_AS_STRING(self->buf) + self->pos, len);`
`587`	`586`	`assert(self->pos + len < PY_SSIZE_T_MAX);`
`588`	`587`	`assert(len >= 0);`
	`588`	`+ memcpy(buffer->buf, PyBytes_AS_STRING(self->buf) + self->pos, len);`
`589`	`589`	`self->pos += len;`
`590`	`590`
`591`	`591`	`return PyLong_FromSsize_t(len);`