diff options
| author | Antoine Pitrou <solipsis@pitrou.net> | 2010-01-17 12:43:00 (GMT) |
|---|---|---|
| committer | Antoine Pitrou <solipsis@pitrou.net> | 2010-01-17 12:43:00 (GMT) |
| commit | 20d6c153275de051f0800a266d511ea5bef782ce (patch) | |
| tree | e9c0538a98a39344b93946bba1ea4c6ae84a4aa4 /Include | |
| parent | 1b64331f49881f23fbbdbab8a9e152aa2188d5b6 (diff) | |
| download | cpython-20d6c153275de051f0800a266d511ea5bef782ce.zip cpython-20d6c153275de051f0800a266d511ea5bef782ce.tar.gz cpython-20d6c153275de051f0800a266d511ea5bef782ce.tar.bz2 | |
Merged revisions 77576 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k
................
r77576 | antoine.pitrou | 2010-01-17 13:38:54 +0100 (dim., 17 janv. 2010) | 12 lines
Merged revisions 77573 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r77573 | antoine.pitrou | 2010-01-17 13:26:20 +0100 (dim., 17 janv. 2010) | 6 lines
Issue #7561: Operations on empty bytearrays (such as `int(bytearray())`)
could crash in many places because of the PyByteArray_AS_STRING() macro
returning NULL. The macro now returns a statically allocated empty
string instead.
........
................
Diffstat (limited to 'Include')
| -rw-r--r-- | Include/bytearrayobject.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/Include/bytearrayobject.h b/Include/bytearrayobject.h index 265b4bb..8702e5a 100644 --- a/Include/bytearrayobject.h +++ b/Include/bytearrayobject.h @@ -44,9 +44,13 @@ PyAPI_FUNC(char *) PyByteArray_AsString(PyObject *); PyAPI_FUNC(int) PyByteArray_Resize(PyObject *, Py_ssize_t); /* Macros, trading safety for speed */ -#define PyByteArray_AS_STRING(self) (assert(PyByteArray_Check(self)),((PyByteArrayObject *)(self))->ob_bytes) +#define PyByteArray_AS_STRING(self) \ + (assert(PyByteArray_Check(self)), \ + Py_SIZE(self) ? ((PyByteArrayObject *)(self))->ob_bytes : _PyByteArray_empty_string) #define PyByteArray_GET_SIZE(self) (assert(PyByteArray_Check(self)),Py_SIZE(self)) +extern char _PyByteArray_empty_string[]; + #ifdef __cplusplus } #endif |