Message122445
The new "s*" code for PyArg_ParseTuple is used to fill a Py_buffer object from the arguments. This object must be relased using PyBuffer_Release() after use.
However, if the object in the tuple does not support the new buffer interface, the old buffer interface is queried and the Py_buffer object is manually filled in. For this case, the source object is _not_ increfed and buffer.obj remains set to 0.
This causes different semantics in the function for objects that are passed in: If the Py_buffer interface is supported directly, then it is safe for the function to store this and release this at a later time. If it isn't supported, then no extra reference to the object is got and the function cannot safely keep the Py_buffer object around.
The Fix is as follows: Change line 1402 of getargs.c from:
PyBuffer_FillInfo(view, NULL, buf, count, 1, 0);
to
PyBuffer_FillInfo(view, arg, buf, count, 1, 0); |
|
Date |
User |
Action |
Args |
2010-11-26 08:07:33 | kristjan.jonsson | set | recipients:
+ kristjan.jonsson |
2010-11-26 08:07:33 | kristjan.jonsson | set | messageid: <1290758853.31.0.608837110631.issue10538@psf.upfronthosting.co.za> |
2010-11-26 08:07:31 | kristjan.jonsson | link | issue10538 messages |
2010-11-26 08:07:31 | kristjan.jonsson | create | |
|