Serhiy Storchaka added the comment:
IĀ forgot mention the issue number in commit messages. See changeset3570e04f4ea9
and changesetbf347198fbaf.
Is the issue fixed now?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9242
Serhiy Storchaka added the comment:
changeset 3570e04f4ea9 and changeset bf347198fbaf
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9242
___
Stefan Krah added the comment:
The utf_32_le_decode invalid access is gone; for Valgrind the issue
is fixed (I didn't look at the patch, no time ATM, sorry).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9242
Changes by Serhiy Storchaka storch...@gmail.com:
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9242
___
Serhiy Storchaka added the comment:
3.2 is affected too. The PEP-393 changes fixes this bug in 3.3+.
Here is a simple patch.
--
assignee: - serhiy.storchaka
keywords: +patch
nosy: +serhiy.storchaka
stage: - patch review
versions: +Python 3.2
Added file:
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9242
___
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9242
___
___
Stefan Krah stefan-use...@bytereef.org added the comment:
Here is a minimal example how to reproduce this issue, extracted from
UTF32LETest.
valgrind --db-attach=yes --suppressions=Misc/valgrind-python.supp ./python
uninitialized.py
It seems that in Lib/codecs.py the equivalent of
New submission from Stefan Krah stefan-use...@bytereef.org:
Not sure if this is valid or if there is some internal trickery that Valgrind
isn't aware of. If it's the latter, perhaps an entry to
Misc/valgrind.supp could be added.
test_bug1175396 (__main__.UTF32Test) ... ==26861== Conditional
Marc-Andre Lemburg m...@egenix.com added the comment:
Stefan Krah wrote:
New submission from Stefan Krah stefan-use...@bytereef.org:
Not sure if this is valid or if there is some internal trickery that Valgrind
isn't aware of. If it's the latter, perhaps an entry to
Misc/valgrind.supp
Stefan Krah stefan-use...@bytereef.org added the comment:
const int iorder[4] = {0, 1, 2, 3};
const isn't possible, iorder is modified later on. Adding the array
dimension did not change anything.
Making everything const (see below) did not change anything either.
I presume that Valgrind
11 matches
Mail list logo