diff options
author | Victor Stinner <victor.stinner@haypocalc.com> | 2010-05-19 00:54:06 (GMT) |
---|---|---|
committer | Victor Stinner <victor.stinner@haypocalc.com> | 2010-05-19 00:54:06 (GMT) |
commit | 93b5513cf1d6ed328bbf2e90377fee7a487b545d (patch) | |
tree | ecbc226c399ae3cae5a0d36653bb0fe71f99b5d1 /Python | |
parent | 386fe71de1d0f35f53f082a88762e90d5b1c66bb (diff) | |
download | cpython-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.c | 15 |
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); } } |