tcl test change crashes AIX

[DavidEdelsohn]

BPO	21951
Nosy	@vstinner, @serhiy-storchaka
Files	tkinter_nomemory.patch tkinter_debug.patch asobj_empty_sequence.patch asobj_empty_sequence_2.patch tkinter_ckallock.patch

^{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 = 'https://github.com/serhiy-storchaka'
closed_at = <Date 2014-09-11.11:07:28.538>
created_at = <Date 2014-07-10.17:38:46.705>
labels = ['interpreter-core', 'expert-tkinter', 'type-crash']
title = 'tcl test change crashes AIX'
updated_at = <Date 2014-09-11.11:35:34.916>
user = 'https://bugs.python.org/DavidEdelsohn'

bugs.python.org fields:

activity = <Date 2014-09-11.11:35:34.916>
actor = 'serhiy.storchaka'
assignee = 'serhiy.storchaka'
closed = True
closed_date = <Date 2014-09-11.11:07:28.538>
closer = 'vstinner'
components = ['Interpreter Core', 'Tkinter']
creation = <Date 2014-07-10.17:38:46.705>
creator = 'David.Edelsohn'
dependencies = []
files = ['35944', '36530', '36535', '36544', '36582']
hgrepos = []
issue_num = 21951
keywords = ['patch']
message_count = 26.0
messages = ['222680', '222837', '222897', '222950', '222965', '222968', '222970', '222977', '224290', '224291', '226345', '226359', '226361', '226362', '226364', '226367', '226374', '226400', '226637', '226721', '226723', '226751', '226754', '226762', '226763', '226765']
nosy_count = 4.0
nosy_names = ['vstinner', 'python-dev', 'serhiy.storchaka', 'David.Edelsohn']
pr_nums = []
priority = 'high'
resolution = 'fixed'
stage = 'commit review'
status = 'closed'
superseder = None
type = 'crash'
url = 'https://bugs.python.org/issue21951'
versions = ['Python 2.7', 'Python 3.4', 'Python 3.5']

[DavidEdelsohn]

The patch for Issue bpo-21881 causes CPython test_tcl to crash on AIX.

$ ./python -m test -v test_tcl
== CPython 3.5.0a0 (default:d1f89eb9ea1e+, Jul 10 2014, 10:21:22) [GCC 4.8.1]
==   AIX-1-00F84C0C4C00-powerpc-32bit big-endian
==   hash algorithm: siphash24 32bit
==   /home/dje/src/cpython/build/test_python_27984522
Testing with flags: sys.flags(debug=0, inspect=0, interactive=0, optimize=0, dont_write_bytecode=0, no_user_site=0, no_site=0, ignore_environment=0, verbose=0, bytes_warning=0, quiet=0, hash_randomization=1, isolated=0)
[1/1] test_tcl
patchlevel = 8.4.7
testCall (test.test_tcl.TclTest) ... ok
testCallException (test.test_tcl.TclTest) ... ok
testCallException2 (test.test_tcl.TclTest) ... ok
testEval (test.test_tcl.TclTest) ... ok
testEvalException (test.test_tcl.TclTest) ... ok
testEvalException2 (test.test_tcl.TclTest) ... ok
testEvalFile (test.test_tcl.TclTest) ... ok
testEvalFileException (test.test_tcl.TclTest) ... ok
testGetVar (test.test_tcl.TclTest) ... ok
testGetVarArray (test.test_tcl.TclTest) ... ok
testGetVarArrayException (test.test_tcl.TclTest) ... ok
testGetVarException (test.test_tcl.TclTest) ... ok
testLoadWithUNC (test.test_tcl.TclTest) ... skipped 'Requires Windows'
testPackageRequireException (test.test_tcl.TclTest) ... ok
testSetVar (test.test_tcl.TclTest) ... ok
testSetVarArray (test.test_tcl.TclTest) ... ok
testUnsetVar (test.test_tcl.TclTest) ... ok
testUnsetVarArray (test.test_tcl.TclTest) ... ok
testUnsetVarException (test.test_tcl.TclTest) ... ok
test_eval_null_in_result (test.test_tcl.TclTest) ... ok
test_evalfile_null_in_result (test.test_tcl.TclTest) ... ok
test_exprboolean (test.test_tcl.TclTest) ... ok
test_exprdouble (test.test_tcl.TclTest) ... ok
test_exprlong (test.test_tcl.TclTest) ... ok
test_exprstring (test.test_tcl.TclTest) ... ok
test_getboolean (test.test_tcl.TclTest) ... ok
test_getdouble (test.test_tcl.TclTest) ... ok
test_getint (test.test_tcl.TclTest) ... ok
test_passing_values (test.test_tcl.TclTest) ... ok
test_split (test.test_tcl.TclTest) ... ok
test_splitlist (test.test_tcl.TclTest) ... ok
test_user_command (test.test_tcl.TclTest) ... Assertion failed: __EX, file  Objects/methodobject.c, line 94
Fatal Python error: Aborted

Current thread 0x00000001 (most recent call first):
File "/home/dje/src/cpython/Lib/test/test_tcl.py", line 426 in check
File "/home/dje/src/cpython/Lib/test/test_tcl.py", line 458 in test_user_command
File "/home/dje/src/cpython/Lib/unittest/case.py", line 577 in run
File "/home/dje/src/cpython/Lib/unittest/case.py", line 625 in __call__
File "/home/dje/src/cpython/Lib/unittest/suite.py", line 125 in run
File "/home/dje/src/cpython/Lib/unittest/suite.py", line 87 in __call__
File "/home/dje/src/cpython/Lib/unittest/suite.py", line 125 in run
File "/home/dje/src/cpython/Lib/unittest/suite.py", line 87 in __call__
File "/home/dje/src/cpython/Lib/unittest/runner.py", line 168 in run
File "/home/dje/src/cpython/Lib/test/support/init.py", line 1724 in _run_suite
File "/home/dje/src/cpython/Lib/test/support/init.py", line 1758 in run_unittest
File "/home/dje/src/cpython/Lib/test/test_tcl.py", line 601 in test_main
File "/home/dje/src/cpython/Lib/test/regrtest.py", line 1278 in runtest_inner
File "/home/dje/src/cpython/Lib/test/regrtest.py", line 978 in runtest
File "/home/dje/src/cpython/Lib/test/regrtest.py", line 763 in main
File "/home/dje/src/cpython/Lib/test/regrtest.py", line 1562 in main_in_temp_cwd
File "/home/dje/src/cpython/Lib/test/main.py", line 3 in <module>
File "/home/dje/src/cpython/Lib/runpy.py", line 85 in _run_code
File "/home/dje/src/cpython/Lib/runpy.py", line 170 in _run_module_as_main
IOT/Abort trap (core dumped)

[serhiy-storchaka]

Is it crashes when comment out "check((), '')"? Is it crashes when comment out two previous checks (for inf and -inf)?

[DavidEdelsohn]

If I comment out all three tests, it runs.

--- a/Lib/test/test_tcl.py      Thu Jul 10 01:17:11 2014 -0400
+++ b/Lib/test/test_tcl.py      Sat Jul 12 16:59:33 2014 -0700
@@ -455,9 +455,9 @@
         check(float('inf'), 'Inf', eq=float_eq)
         check(-float('inf'), '-Inf', eq=float_eq)
         # XXX NaN representation can be not parsable by float()
-        check((), '')
-        check((1, (2,), (3, 4), '5 6', ()), '1 2 {3 4} {5 6} {}')
-        check([1, [2,], [3, 4], '5 6', []], '1 2 {3 4} {5 6} {}')
+        #check((), '')
+        #check((1, (2,), (3, 4), '5 6', ()), '1 2 {3 4} {5 6} {}')
+        #check([1, [2,], [3, 4], '5 6', []], '1 2 {3 4} {5 6} {}')
 
     def test_splitlist(self):
         splitlist = self.interp.tk.splitlist

[serhiy-storchaka]

Is following test passed?

    check((1,), '1')

Is CPython crashes when change signature of testfunc?

--- a/Lib/test/test_tcl.py      Sat Jul 12 18:26:03 2014 +0300
+++ b/Lib/test/test_tcl.py      Sun Jul 13 19:21:55 2014 +0300
@@ -416,7 +416,7 @@
 
     def test_user_command(self):
         result = None
-        def testfunc(arg):
+        def testfunc(arg=None, *args):
             nonlocal result
             result = arg
             return arg

[DavidEdelsohn]

Is following test passed?

    check((1,), '1')

That test succeeds.

Is CPython crashes when change signature of testfunc?

--- a/Lib/test/test_tcl.py      Sat Jul 12 18:26:03 2014 +0300
+++ b/Lib/test/test_tcl.py      Sun Jul 13 19:21:55 2014 +0300
@@ -416,7 +416,7 @@
 
     def test_user_command(self):
         result = None
-        def testfunc(arg):
+        def testfunc(arg=None, *args):
             nonlocal result
             result = arg
             return arg

This change does not have any effect on the original failure. The assertion failure and crash continues to occur.

[DavidEdelsohn]

#5 0x1019bb90 in PyCFunction_Call (func=0x30533b5c, arg=0x305d8ab4, kw=0x0)
at Objects/methodobject.c:94
#6 0x1012534c in call_function (pp_stack=0x2ff16144, oparg=2)
at Python/ceval.c:4269

94 CHECK_RESULT(res);

(gdb) print res
$9 = (PyObject *) 0x0
(gdb) print meth
$10 = (PyCFunction) @0x20380888: 0xd8dd57dc <Tkapp_Call>
(gdb) print self
$11 = (PyObject *) 0x305d44d8

[serhiy-storchaka]

May be this patch will help.

[DavidEdelsohn]

No difference with the patch. The problem is not a a malloc() failure.

[python-dev]

New changeset 7de64def6565 by Serhiy Storchaka in branch '2.7':
Issue bpo-21951: Temporary skip crashing test_user_command on AIX.
http://hg.python.org/cpython/rev/7de64def6565

New changeset 31f4cb1fede9 by Serhiy Storchaka in branch '3.4':
Issue bpo-21951: Temporary skip crashing test_user_command on AIX.
http://hg.python.org/cpython/rev/31f4cb1fede9

New changeset de32cd419174 by Serhiy Storchaka in branch 'default':
Issue bpo-21951: Temporary skip crashing test_user_command on AIX.
http://hg.python.org/cpython/rev/de32cd419174

[serhiy-storchaka]

Link to log with a crash:

http://buildbot.python.org/all/builders/PPC64%20AIX%203.x/builds/2442/steps/test/logs/stdio

[vstinner]

The failing assertion means that the method raised an exception and returned a result, or failed but didn't raised an exception. According to the gdb trace, the function failed (res is NULL). You can display the current exception in gdb using: "print PyErr_Occurred()" (NULL if there is no exception).

Please run test_tcl on AIX with attached test_tkinter.patch applied.

[serhiy-storchaka]

As far as I remember, both res and PyErr_Occurred() were NULL.