Charles-François Natali neolo...@free.fr added the comment:
A 2.7 OS X buildbot segfaults in test_unicode since 0cd197f13400 :
http://www.python.org/dev/buildbot/all/builders/AMD64 Snow Leopard 2
2.7/builds/409/steps/test/logs/stdio
test_unicode
make: *** [buildbottest] Segmentation fault
STINNER Victor victor.stin...@haypocalc.com added the comment:
A 2.7 OS X buildbot segfaults in test_unicode since 0cd197f13400
http://www.python.org/dev/buildbot/all/builders/AMD64 Snow Leopard 2
2.7/builds/409/steps/test/logs/stdio
Hum, I'm unable to reproduce the crash on Linux or Mac OS
Ned Deily n...@acm.org added the comment:
I am able to reproduce the problem on 2.7 OS X 10.7 64-bit.
unicode._encodedecimal is gobbling up memory. Looks like length is incorrect.
Breakpoint 1, unicode_encodedecimal (self=0x0, args=0x1007ce0d0) at
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 3ecddf168f1f by Victor Stinner in branch '2.7':
Issue #13093: Fix _testcapi.unicode_encodedecimal()
http://hg.python.org/cpython/rev/3ecddf168f1f
--
___
Python tracker
STINNER Victor victor.stin...@haypocalc.com added the comment:
Looks like length is incorrect.
Oh ok, _testcapimodule.c is sssize_t safe in Python 3, not in Python 2. Can
you please try with the last tip?
--
___
Python tracker
Ned Deily n...@acm.org added the comment:
That fixes it.
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13093
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset a20fae95618c by Victor Stinner in branch 'default':
Close #13093: PyUnicode_EncodeDecimal() doesn't support error handlers
http://hg.python.org/cpython/rev/a20fae95618c
--
stage: patch review -
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
Removed message: http://bugs.python.org/msg148348
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13093
___
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 52fecdc1c5d8 by Charles-François Natali in branch 'default':
Issue #13093: Perform a real merge.
http://hg.python.org/cpython/rev/52fecdc1c5d8
--
___
Python tracker
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset f29d7d597fae by Victor Stinner in branch '3.2':
Issue #13093: Fix error handling on PyUnicode_EncodeDecimal()
http://hg.python.org/cpython/rev/f29d7d597fae
New changeset bc53c11804ab by Victor Stinner in branch
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 0cd197f13400 by Victor Stinner in branch '2.7':
Issue #13093: Fix error handling on PyUnicode_EncodeDecimal()
http://hg.python.org/cpython/rev/0cd197f13400
--
___
Python
STINNER Victor victor.stin...@haypocalc.com added the comment:
I fixed the issue in Python 2.7, 3.2 and 3.3.
See also the changeset 849e9277906a (Python 3.3).
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
STINNER Victor victor.stin...@haypocalc.com added the comment:
The code is also present in Python 2.7 and 3.2.
--
nosy: +haypo
versions: +Python 2.7, Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13093
STINNER Victor victor.stin...@haypocalc.com added the comment:
PyUnicode_EncodeDecimal() was used in Python 2 by int, long and complex
constructors. In Python 3, the function is no more used: it has been replaced
by _PyUnicode_TransformDecimalAndSpaceToASCII().
--
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
nosy: +loewis
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13093
___
___
Stefan Krah stefan-use...@bytereef.org added the comment:
Looking at it again, the intention was probably to increment
collend so that it points to the first non-garbage character
(or '\0'). If that's the case, the loop should be something
like this:
while (collend end) {
if ((0
Martin v. Löwis mar...@v.loewis.de added the comment:
AFAICT, this code was introduced in 0337dad8403e, implementing PEP 293 (see
#432401). The intention clearly was what Stefan figured out: compute the list
of unencodable characters, to pass the longest run of unencodable characters to
the
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +facundobatista, mark.dickinson, rhettinger
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13093
___
New submission from Stefan Krah stefan-use...@bytereef.org:
I can't see what this code is supposed to accomplish (see patch):
while (collend end) {
if ((0 *collend *collend 256) ||
!Py_UNICODE_ISSPACE(*collend) ||
Py_UNICODE_TODECIMAL(*collend))
break;
19 matches
Mail list logo