summaryrefslogtreecommitdiffstats
path: root/Objects
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2016-10-04 15:17:22 (GMT)
committerSerhiy Storchaka <storchaka@gmail.com>2016-10-04 15:17:22 (GMT)
commit09f3d080fe0cadab0db6380c58dd4968db20287d (patch)
tree9693ba1154b71e939f2097522672f6cdf1851de4 /Objects
parentb47c9d29d763c9c24542e6de9191ef50da021ce6 (diff)
downloadcpython-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.c16
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;
}