Issue #14172: Fix reference leak when marshalling a buffer-like object (other than a bytes object).

This commit is contained in:
Antoine Pitrou 2012-03-02 18:16:38 +01:00
commit b2677c7397
3 changed files with 30 additions and 4 deletions

View file

@ -409,11 +409,12 @@ w_object(PyObject *v, WFILE *p)
else if (PyObject_CheckBuffer(v)) {
/* Write unknown buffer-style objects as a string */
char *s;
PyBufferProcs *pb = v->ob_type->tp_as_buffer;
Py_buffer view;
if ((*pb->bf_getbuffer)(v, &view, PyBUF_SIMPLE) != 0) {
if (PyObject_GetBuffer(v, &view, PyBUF_SIMPLE) != 0) {
w_byte(TYPE_UNKNOWN, p);
p->depth--;
p->error = WFERR_UNMARSHALLABLE;
return;
}
w_byte(TYPE_STRING, p);
n = view.len;
@ -425,8 +426,7 @@ w_object(PyObject *v, WFILE *p)
}
w_long((long)n, p);
w_string(s, (int)n, p);
if (pb->bf_releasebuffer != NULL)
(*pb->bf_releasebuffer)(v, &view);
PyBuffer_Release(&view);
}
else {
w_byte(TYPE_UNKNOWN, p);