diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2016-10-04 15:17:22 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2016-10-04 15:17:22 (GMT) |
commit | 09f3d080fe0cadab0db6380c58dd4968db20287d (patch) | |
tree | 9693ba1154b71e939f2097522672f6cdf1851de4 /Objects | |
parent | b47c9d29d763c9c24542e6de9191ef50da021ce6 (diff) | |
download | cpython-09f3d080fe0cadab0db6380c58dd4968db20287d.zip cpython-09f3d080fe0cadab0db6380c58dd4968db20287d.tar.gz cpython-09f3d080fe0cadab0db6380c58dd4968db20287d.tar.bz2 |
Issue #28350: String constants with null character no longer interned.
Diffstat (limited to 'Objects')
-rw-r--r-- | Objects/codeobject.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/Objects/codeobject.c b/Objects/codeobject.c index e50f730..9df87b0 100644 --- a/Objects/codeobject.c +++ b/Objects/codeobject.c @@ -11,21 +11,21 @@ static int all_name_chars(PyObject *o) { static char ok_name_char[256]; - static unsigned char *name_chars = (unsigned char *)NAME_CHARS; - PyUnicodeObject *u = (PyUnicodeObject *)o; - const unsigned char *s; + static const unsigned char *name_chars = (unsigned char *)NAME_CHARS; + const unsigned char *s, *e; - if (!PyUnicode_Check(o) || PyUnicode_READY(u) == -1 || - PyUnicode_MAX_CHAR_VALUE(u) >= 128) + if (!PyUnicode_Check(o) || PyUnicode_READY(o) == -1 || + !PyUnicode_IS_ASCII(o)) return 0; if (ok_name_char[*name_chars] == 0) { - unsigned char *p; + const unsigned char *p; for (p = name_chars; *p; p++) ok_name_char[*p] = 1; } - s = PyUnicode_1BYTE_DATA(u); - while (*s) { + s = PyUnicode_1BYTE_DATA(o); + e = s + PyUnicode_GET_LENGTH(o); + while (s != e) { if (ok_name_char[*s++] == 0) return 0; } |