[issue26942] android: test_ctypes crashes on armv7 and aarch64

Changes by Xavier de Gaye : -- status: open -> closed ___ Python tracker ___ ___

[issue26942] android: test_ctypes crashes on armv7 and aarch64

Xavier de Gaye added the comment: Thanks Chi Hsuan Yen for your contributions with this issue. -- ___ Python tracker ___

[issue26942] android: test_ctypes crashes on armv7 and aarch64

Xavier de Gaye added the comment: At least for non-Darwin POSIX builds: * Building _ctypes with the bundled copy of libffi is deprecated in 3.6 and the default is to use a system copy of libffi, issue 27976. * The bundled libffi is removed in 3.7, issue 27979. As this crash happens with the

[issue26942] android: test_ctypes crashes on armv7 and aarch64

Chi Hsuan Yen added the comment: Since issue27976, this one can be closed as third-party, just like issue27323. -- ___ Python tracker ___

[issue26942] android: test_ctypes crashes on armv7 and aarch64

Xavier de Gaye added the comment: We should wait for the pull request to be merged in the libffi development repo before committing the patch. The PR is at https://github.com/libffi/libffi/pull/265. -- ___ Python tracker

[issue26942] android: test_ctypes crashes on armv7 and aarch64

Chi Hsuan Yen added the comment: You're right. I thought the default malloc() implementation is better, and now I think a unified implementation on Android brings less surprises. -- Added file: http://bugs.python.org/file43945/libffi-pr240.patch ___

[issue26942] android: test_ctypes crashes on armv7 and aarch64

Xavier de Gaye added the comment: > By msg264746, only ARM fails, so I patch libffi for arm and aarch64 triplets > only Why not for all Android architectures (*-linux-android*) as it is done in PR120 ? -- stage: -> patch review ___ Python tracker

[issue26942] android: test_ctypes crashes on armv7 and aarch64

Changes by Chi Hsuan Yen : -- title: android: test_ctypes crashes on armv7 -> android: test_ctypes crashes on armv7 and aarch64 ___ Python tracker

[issue26942] android: test_ctypes crashes on armv7

Xavier de Gaye added the comment: Nice, the patch fixes the problem when python is built with gcc :) Running test_ctypes on the Android emulator when python is built for the arm architecture or the armv7 architecture gives in both cases the same successfull result: Ran 456 tests in

[issue26942] android: test_ctypes crashes on armv7

Chi Hsuan Yen added the comment: Test results against patched libffi in Modules/_ctypes: shell@ASUS_Z00E_2:/data/local/tmp $ python3.6 -m test.test_ctypes

[issue26942] android: test_ctypes crashes on armv7

Chi Hsuan Yen added the comment: By msg264746, only ARM fails, so I patch libffi for arm and aarch64 triplets only -- keywords: +patch Added file: http://bugs.python.org/file43894/libffi-pr240.patch ___ Python tracker

[issue26942] android: test_ctypes crashes on armv7

Xavier de Gaye added the comment: Thanks for looking into that problem. Can you provide a patch ? -- ___ Python tracker ___

[issue26942] android: test_ctypes crashes on armv7

Chi Hsuan Yen added the comment: Found libffi PR240 that fixes closures on Android: shell@ASUS_Z00E_2:/data/local/tmp $ python3.6 -m test.test_ctypes

[issue26942] android: test_ctypes crashes on armv7

Chi Hsuan Yen added the comment: Maybe a libffi issue. The crash is still with libffi git-master and CPython hg-tip + `--with-system-libffi`. Reported to https://github.com/libffi/libffi/issues/262 -- nosy: +Chi Hsuan Yen ___ Python tracker

[issue26942] android: test_ctypes crashes on armv7

Xavier de Gaye added the comment: Running the following interactive statements [1]: >>> import unittest, ctypes.test.test_as_parameter >>> unittest.main(module=ctypes.test.test_as_parameter, >>> defaultTest='BasicWrapTestCase', verbosity=2) The corresponding attached gdb session: (gdb)

[issue26942] android: test_ctypes crashes on armv7

Xavier de Gaye added the comment: The crash occurs at the same line that the crash reported in issue #17786. Line 85 in ctypes/test/test_as_parameter.py was line 87 at changeset ae5c4a9118b8a3f490f77f2084d46163ca229aef. -- ___ Python tracker

[issue26942] android: test_ctypes crashes on armv7

New submission from Xavier de Gaye: test_ctypes crashes on an android emulator running an armv7 system image (but not on x86) at API level 21. 143|root@generic:/data/local/tmp # python -m test -v test_ctypes == CPython 3.6.0a0 (default:f4c6dab59cd8+, May 3