diff options
author | Pablo Galindo <Pablogsal@gmail.com> | 2021-06-08 19:02:03 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-06-08 19:02:03 (GMT) |
commit | bafe0aade5741ab0d13143ee261711fdd65e8a1f (patch) | |
tree | 4fba23f314cadbdd155572cc59f0c158d2a6fe13 /Parser | |
parent | 8004c4570b1d1277ea8754e22b5eb60e63f5026c (diff) | |
download | cpython-bafe0aade5741ab0d13143ee261711fdd65e8a1f.zip cpython-bafe0aade5741ab0d13143ee261711fdd65e8a1f.tar.gz cpython-bafe0aade5741ab0d13143ee261711fdd65e8a1f.tar.bz2 |
bpo-44335: Ensure the tokenizer doesn't go into Python with the error set (GH-26608)
Diffstat (limited to 'Parser')
-rw-r--r-- | Parser/pegen.c | 20 |
1 files changed, 17 insertions, 3 deletions
diff --git a/Parser/pegen.c b/Parser/pegen.c index c69a042..42a9922 100644 --- a/Parser/pegen.c +++ b/Parser/pegen.c @@ -1251,9 +1251,14 @@ _PyPegen_check_tokenizer_errors(Parser *p) { return 0; } + PyObject *type, *value, *traceback; + PyErr_Fetch(&type, &value, &traceback); + Token *current_token = p->known_err_token != NULL ? p->known_err_token : p->tokens[p->fill - 1]; Py_ssize_t current_err_line = current_token->lineno; + int ret = 0; + for (;;) { const char *start; const char *end; @@ -1262,9 +1267,9 @@ _PyPegen_check_tokenizer_errors(Parser *p) { if (p->tok->level != 0) { int error_lineno = p->tok->parenlinenostack[p->tok->level-1]; if (current_err_line > error_lineno) { - PyErr_Clear(); raise_unclosed_parentheses_error(p); - return -1; + ret = -1; + goto exit; } } break; @@ -1276,7 +1281,16 @@ _PyPegen_check_tokenizer_errors(Parser *p) { break; } - return 0; + +exit: + if (PyErr_Occurred()) { + Py_XDECREF(value); + Py_XDECREF(type); + Py_XDECREF(traceback); + } else { + PyErr_Restore(type, value, traceback); + } + return ret; } void * |