diff options
| author | Andrew Dalke <dalke@dalkescientific.com> | 2006-05-25 17:53:00 (GMT) |
|---|---|---|
| committer | Andrew Dalke <dalke@dalkescientific.com> | 2006-05-25 17:53:00 (GMT) |
| commit | 8c9091074b6a0e7e48eb2cce03a4eb193cb993be (patch) | |
| tree | 00401c251222ba1b77739198ccf1d583fca17285 /Objects | |
| parent | da53afa1b09218c0fb7ce8803cd783a43ee9d319 (diff) | |
| download | cpython-8c9091074b6a0e7e48eb2cce03a4eb193cb993be.zip cpython-8c9091074b6a0e7e48eb2cce03a4eb193cb993be.tar.gz cpython-8c9091074b6a0e7e48eb2cce03a4eb193cb993be.tar.bz2 | |
Fixed problem identified by Georg. The special-case in-place code for replace
made a copy of the string using PyString_FromStringAndSize(s, n) and modify
the copied string in-place. However, 1 (and 0) character strings are shared
from a cache. This cause "A".replace("A", "a") to change the cached version
of "A" -- used by everyone.
Now may the copy with NULL as the string and do the memcpy manually. I've
added regression tests to check if this happens in the future. Perhaps
there should be a PyString_Copy for this case?
Diffstat (limited to 'Objects')
| -rw-r--r-- | Objects/stringobject.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/Objects/stringobject.c b/Objects/stringobject.c index aed0c23..b0c4640 100644 --- a/Objects/stringobject.c +++ b/Objects/stringobject.c @@ -2692,10 +2692,11 @@ replace_single_character_in_place(PyStringObject *self, } /* Need to make a new string */ - result = (PyStringObject *) PyString_FromStringAndSize(self_s, self_len); + result = (PyStringObject *) PyString_FromStringAndSize(NULL, self_len); if (result == NULL) return NULL; result_s = PyString_AS_STRING(result); + memcpy(result_s, self_s, self_len); /* change everything in-place, starting with this one */ start = result_s + (next-self_s); @@ -2745,10 +2746,12 @@ replace_substring_in_place(PyStringObject *self, } /* Need to make a new string */ - result = (PyStringObject *) PyString_FromStringAndSize(self_s, self_len); + result = (PyStringObject *) PyString_FromStringAndSize(NULL, self_len); if (result == NULL) return NULL; result_s = PyString_AS_STRING(result); + memcpy(result_s, self_s, self_len); + /* change everything in-place, starting with this one */ start = result_s + offset; |