summaryrefslogtreecommitdiffstats
path: root/Parser
diff options
context:
space:
mode:
authorPablo Galindo <Pablogsal@gmail.com>2021-06-08 23:54:29 (GMT)
committerGitHub <noreply@github.com>2021-06-08 23:54:29 (GMT)
commit9fd21f649d66dcb10108ee395fd68ed32c8239cd (patch)
treef04ca9dce4d4cad34d7833252326cd195031283a /Parser
parent2ea6d890281c415e0a2f00e63526e592da8ce3d9 (diff)
downloadcpython-9fd21f649d66dcb10108ee395fd68ed32c8239cd.zip
cpython-9fd21f649d66dcb10108ee395fd68ed32c8239cd.tar.gz
cpython-9fd21f649d66dcb10108ee395fd68ed32c8239cd.tar.bz2
bpo-44349: Fix edge case when displaying text from files with encoding in syntax errors (GH-26611)
Diffstat (limited to 'Parser')
-rw-r--r--Parser/pegen.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/Parser/pegen.c b/Parser/pegen.c
index 42a9922..e651819 100644
--- a/Parser/pegen.c
+++ b/Parser/pegen.c
@@ -456,10 +456,13 @@ _PyPegen_raise_error_known_location(Parser *p, PyObject *errtype,
goto error;
}
+ // PyErr_ProgramTextObject assumes that the text is utf-8 so we cannot call it with a file
+ // with an arbitrary encoding or otherwise we could get some badly decoded text.
+ int uses_utf8_codec = (!p->tok->encoding || strcmp(p->tok->encoding, "utf-8") == 0);
if (p->tok->fp_interactive) {
error_line = get_error_line(p, lineno);
}
- else if (p->start_rule == Py_file_input) {
+ else if (uses_utf8_codec && p->start_rule == Py_file_input) {
error_line = PyErr_ProgramTextObject(p->tok->filename, (int) lineno);
}
@@ -471,7 +474,7 @@ _PyPegen_raise_error_known_location(Parser *p, PyObject *errtype,
we're actually parsing from a file, which has an E_EOF SyntaxError and in that case
`PyErr_ProgramTextObject` fails because lineno points to last_file_line + 1, which
does not physically exist */
- assert(p->tok->fp == NULL || p->tok->fp == stdin || p->tok->done == E_EOF);
+ assert(p->tok->fp == NULL || p->tok->fp == stdin || p->tok->done == E_EOF || !uses_utf8_codec);
if (p->tok->lineno <= lineno) {
Py_ssize_t size = p->tok->inp - p->tok->buf;