From d21fb4c2e097dd72d1772a6f0325cc45f9b20f30 Mon Sep 17 00:00:00 2001 From: Amaury Forgeot d'Arc Date: Wed, 5 Mar 2008 01:50:33 +0000 Subject: [PATCH] Issue#2238: some syntax errors from *args or **kwargs expressions would give bogus error messages, because of untested exceptions:: >>> f(**g(1=2)) XXX undetected error Traceback (most recent call last): File "", line 1, in TypeError: 'int' object is not iterable instead of the expected SyntaxError: keyword can't be an expression Will backport. --- Lib/test/test_grammar.py | 4 ++++ Misc/NEWS | 6 ++++++ Python/ast.c | 4 ++++ 3 files changed, 14 insertions(+) diff --git a/Lib/test/test_grammar.py b/Lib/test/test_grammar.py index e2bb3eb9e63..0e593677fd5 100644 --- a/Lib/test/test_grammar.py +++ b/Lib/test/test_grammar.py @@ -282,6 +282,10 @@ class GrammarTests(unittest.TestCase): def d32v((x,)): pass d32v((1,)) + # Check ast errors in *args and *kwargs + check_syntax_error(self, "f(*g(1=2))") + check_syntax_error(self, "f(**g(1=2))") + def testLambdef(self): ### lambdef: 'lambda' [varargslist] ':' test l1 = lambda : 0 diff --git a/Misc/NEWS b/Misc/NEWS index 4fac8599a6c..4992513fac6 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -9,6 +9,12 @@ What's New in Python 2.6 alpha 2? *Release date: XX-XXX-2008* +Core and builtins +----------------- + +- Issue #2238: Some syntax errors in *args and **kwargs expressions could give + bogus error messages. + What's New in Python 2.6 alpha 1? ================================= diff --git a/Python/ast.c b/Python/ast.c index 7a1b5bc13e1..e14ff3a3da1 100644 --- a/Python/ast.c +++ b/Python/ast.c @@ -1934,10 +1934,14 @@ ast_for_call(struct compiling *c, const node *n, expr_ty func) } else if (TYPE(ch) == STAR) { vararg = ast_for_expr(c, CHILD(n, i+1)); + if (!vararg) + return NULL; i++; } else if (TYPE(ch) == DOUBLESTAR) { kwarg = ast_for_expr(c, CHILD(n, i+1)); + if (!kwarg) + return NULL; i++; } }