Rafał Chłodnicki added the comment:
Maybe worth backporting to 3.3 and up? Especially if #20160 is to be backported
as they seem to be related.
--
nosy: +Rafał Chłodnicki
___
Python tracker
Roundup Robot added the comment:
New changeset 4c81b1846e14 by Steve Dower in branch 'default':
Issue #22733: MSVC ffi_prep_args doesn't handle 64-bit arguments properly
https://hg.python.org/cpython/rev/4c81b1846e14
--
nosy: +python-dev
___
Python
Roundup Robot added the comment:
New changeset 24f4569a1308 by Steve Dower in branch 'default':
Issue #22733: Added NEWS item
https://hg.python.org/cpython/rev/24f4569a1308
--
___
Python tracker rep...@bugs.python.org
Changes by Steve Dower steve.do...@microsoft.com:
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22733
___
Tim Golden added the comment:
Likewise.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22733
___
___
Python-bugs-list mailing list
Unsubscribe:
Steve Dower added the comment:
This patch also resolves this test issue:
[ 68/390/2] test_ttk_textonly
test test_ttk_textonly crashed -- Traceback (most recent call last):
File D:\...\lib\test\regrtest.py, line 1280, in runtest_inner
test_runner()
File
Zachary Ware added the comment:
I don't know near enough to comment on this one. If you feel confident in it,
I'd say go ahead.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22733
___
Steve Dower added the comment:
Nosying the other Windows guys - seems like the ctypes nosy list is inactive,
and this only affects MSVC.
--
nosy: +tim.golden, zach.ware
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22733
New submission from Steve Dower:
The ffi_prep_args function in libffi_msvc/ffi.c needs the attached patch to
handle 64-bit parameters with the correct padding.
Without this patch, garbage may appear in the top part of 64-bit arguments as
the values are not zeroed out by the memcpy.
I'm not