summaryrefslogtreecommitdiffstats
path: root/Python
diff options
context:
space:
mode:
authorVictor Stinner <victor.stinner@haypocalc.com>2010-05-19 00:54:06 (GMT)
committerVictor Stinner <victor.stinner@haypocalc.com>2010-05-19 00:54:06 (GMT)
commit93b5513cf1d6ed328bbf2e90377fee7a487b545d (patch)
treeecbc226c399ae3cae5a0d36653bb0fe71f99b5d1 /Python
parent386fe71de1d0f35f53f082a88762e90d5b1c66bb (diff)
downloadcpython-93b5513cf1d6ed328bbf2e90377fee7a487b545d.zip
cpython-93b5513cf1d6ed328bbf2e90377fee7a487b545d.tar.gz
cpython-93b5513cf1d6ed328bbf2e90377fee7a487b545d.tar.bz2
Issue #6697: Fix a crash if a keyword contains a surrogate
Diffstat (limited to 'Python')
-rw-r--r--Python/getargs.c15
1 files changed, 9 insertions, 6 deletions
diff --git a/Python/getargs.c b/Python/getargs.c
index 4fa2b5b..e8efb3c 100644
--- a/Python/getargs.c
+++ b/Python/getargs.c
@@ -1755,18 +1755,21 @@ vgetargskeywords(PyObject *args, PyObject *keywords, const char *format,
"keywords must be strings");
return cleanreturn(0, freelist);
}
+ /* check that _PyUnicode_AsString() result is not NULL */
ks = _PyUnicode_AsString(key);
- for (i = 0; i < len; i++) {
- if (!strcmp(ks, kwlist[i])) {
- match = 1;
- break;
+ if (ks != NULL) {
+ for (i = 0; i < len; i++) {
+ if (!strcmp(ks, kwlist[i])) {
+ match = 1;
+ break;
+ }
}
}
if (!match) {
PyErr_Format(PyExc_TypeError,
- "'%s' is an invalid keyword "
+ "'%U' is an invalid keyword "
"argument for this function",
- ks);
+ key);
return cleanreturn(0, freelist);
}
}