summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVictor Stinner <victor.stinner@haypocalc.com>2011-09-29 23:54:04 (GMT)
committerVictor Stinner <victor.stinner@haypocalc.com>2011-09-29 23:54:04 (GMT)
commit1fbcaeff55ce799b75b787726ef362dd7470f253 (patch)
tree1ba05ab8706391d15a311a13c3582da7bf1edf40
parenteb5657a0c5bd2a45881ca379a095298490014451 (diff)
downloadcpython-1fbcaeff55ce799b75b787726ef362dd7470f253.zip
cpython-1fbcaeff55ce799b75b787726ef362dd7470f253.tar.gz
cpython-1fbcaeff55ce799b75b787726ef362dd7470f253.tar.bz2
Fix array.array('u') constructor
-rw-r--r--Modules/arraymodule.c18
1 files changed, 12 insertions, 6 deletions
diff --git a/Modules/arraymodule.c b/Modules/arraymodule.c
index 4888f84..3f91289 100644
--- a/Modules/arraymodule.c
+++ b/Modules/arraymodule.c
@@ -2529,19 +2529,25 @@ array_new(PyTypeObject *type, PyObject *args, PyObject *kwds)
Py_DECREF(v);
}
else if (initial != NULL && PyUnicode_Check(initial)) {
- Py_ssize_t n = PyUnicode_GET_DATA_SIZE(initial);
+ Py_ssize_t n;
+ if (PyUnicode_READY(initial)) {
+ Py_DECREF(a);
+ return NULL;
+ }
+ n = PyUnicode_GET_LENGTH(initial);
if (n > 0) {
arrayobject *self = (arrayobject *)a;
- char *item = self->ob_item;
- item = (char *)PyMem_Realloc(item, n);
+ Py_UCS4 *item = (Py_UCS4 *)self->ob_item;
+ item = (char *)PyMem_Realloc(item, n * sizeof(Py_UCS4));
if (item == NULL) {
PyErr_NoMemory();
Py_DECREF(a);
return NULL;
}
- self->ob_item = item;
- Py_SIZE(self) = n / sizeof(Py_UCS4);
- memcpy(item, PyUnicode_AS_DATA(initial), n);
+ self->ob_item = (char*)item;
+ Py_SIZE(self) = n;
+ if (!PyUnicode_AsUCS4(initial, item, n, 0))
+ return NULL;
self->allocated = Py_SIZE(self);
}
}