You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
test_ctypes failure on AIX in PyEval_CallObjectWithKeywords #62809
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee=Noneclosed_at=<Date2013-08-13.23:14:06.489>created_at=<Date2013-07-31.21:24:37.101>labels= ['interpreter-core', 'type-crash']
title='test_ctypes failure on AIX in PyEval_CallObjectWithKeywords'updated_at=<Date2013-10-29.10:34:20.487>user='https://bugs.python.org/DavidEdelsohn'
This issue is not a bug, but a new assertion added by myself for the issue bpo-18408. The idea is to not call a Python function with an exception set, because the exception may be cleared or replaced (ex: hasattr() clears the current exception if an attribute is not found).
I didn't see this specific issue because AIX is using a special locale setting. Its locale encoding is an alias of the ISO-8859-1 which is not known by PyUnicode_Decode().
I'm waiting for the buildbot to see if my changeset fixed the issue. If it does, I will also add a fast-path for the "iso8859-1" alias of the ISO-8859-1 codec in PyUnicode_Decode(). So the AIX buildbot will also use the C implementation of the codec.
DavidEdelsohn commented on Aug 2, 2013
DavidEdelsohnmannequin
MannequinAuthor
More actions
Thanks. The patch seems to return the tests to the expected failures and Python does not crash.
Activity
DavidEdelsohn commentedon Jul 31, 2013
test_ctypes now crashes on AIX after the ceval change to PyEval_CallObjectWithKeywords to check PyErr_Occurred().
python-dev commentedon Aug 1, 2013
New changeset de1ea7f84e08 by Victor Stinner in branch 'default':
Issue bpo-18609, bpo-18408: _ctypes_add_traceback() now clears the current exception
http://hg.python.org/cpython/rev/de1ea7f84e08
vstinner commentedon Aug 1, 2013
This issue is not a bug, but a new assertion added by myself for the issue bpo-18408. The idea is to not call a Python function with an exception set, because the exception may be cleared or replaced (ex: hasattr() clears the current exception if an attribute is not found).
I didn't see this specific issue because AIX is using a special locale setting. Its locale encoding is an alias of the ISO-8859-1 which is not known by PyUnicode_Decode().
I'm waiting for the buildbot to see if my changeset fixed the issue. If it does, I will also add a fast-path for the "iso8859-1" alias of the ISO-8859-1 codec in PyUnicode_Decode(). So the AIX buildbot will also use the C implementation of the codec.
DavidEdelsohn commentedon Aug 2, 2013
Thanks. The patch seems to return the tests to the expected failures and Python does not crash.
python-dev commentedon Oct 29, 2013
New changeset 31f9c1481cfa by Victor Stinner in branch 'default':
Issue bpo-18609: Add a fast-path for "iso8859-1" encoding
http://hg.python.org/cpython/rev/31f9c1481cfa