diff options
author | Batuhan Taskaya <isidentical@gmail.com> | 2020-06-06 12:44:16 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-06-06 12:44:16 (GMT) |
commit | 68874a8502da440a1dc4746cf73262648b870aee (patch) | |
tree | 0e9a81285188e30842515122f2490b2ac197a0ce /Python | |
parent | 5552850f8e6ad6bf610c2633c74ed42dacc81b46 (diff) | |
download | cpython-68874a8502da440a1dc4746cf73262648b870aee.zip cpython-68874a8502da440a1dc4746cf73262648b870aee.tar.gz cpython-68874a8502da440a1dc4746cf73262648b870aee.tar.bz2 |
bpo-40870: Invalidate usage of some constants with ast.Name (GH-20649)
Diffstat (limited to 'Python')
-rw-r--r-- | Python/ast.c | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/Python/ast.c b/Python/ast.c index c524b8e..408591f 100644 --- a/Python/ast.c +++ b/Python/ast.c @@ -22,6 +22,25 @@ static int validate_stmt(stmt_ty); static int validate_expr(expr_ty, expr_context_ty); static int +validate_name(PyObject *name) +{ + assert(PyUnicode_Check(name)); + static const char * const forbidden[] = { + "None", + "True", + "False", + NULL + }; + for (int i = 0; forbidden[i] != NULL; i++) { + if (_PyUnicode_EqualToASCIIString(name, forbidden[i])) { + PyErr_Format(PyExc_ValueError, "Name node can't be used with '%s' constant", forbidden[i]); + return 0; + } + } + return 1; +} + +static int validate_comprehension(asdl_seq *gens) { Py_ssize_t i; @@ -173,6 +192,9 @@ validate_expr(expr_ty exp, expr_context_ty ctx) actual_ctx = exp->v.Starred.ctx; break; case Name_kind: + if (!validate_name(exp->v.Name.id)) { + return 0; + } actual_ctx = exp->v.Name.ctx; break; case List_kind: |