From: Guido van Rossum <guido@python.org>
Date: Tue, 26 Oct 1999 00:12:20 +0000 (+0000)
Subject: Fix PR117.  The error message is "keywords must be strings".  Perhaps
X-Git-Tag: v1.6a1~799
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=25da5bebd8875728f1719ddc0f70dde529cec224;p=thirdparty%2FPython%2Fcpython.git

Fix PR117.  The error message is "keywords must be strings".  Perhaps
not as descriptive as what Barry suggests, but this also catches the
(in my opinion important) case where some other C code besides apply()
constructs a kwdict that doesn't have the right format.  All the other
possibilities of getting it wrong (non-dict, wrong keywords etc) are
already caught so this makes sense to check here.
---

diff --git a/Python/ceval.c b/Python/ceval.c
index 4c1bf2f8b430..fe2e0a2910f6 100644
--- a/Python/ceval.c
+++ b/Python/ceval.c
@@ -482,6 +482,11 @@ eval_code2(co, globals, locals,
 			PyObject *keyword = kws[2*i];
 			PyObject *value = kws[2*i + 1];
 			int j;
+			if (keyword == NULL || !PyString_Check(keyword)) {
+				PyErr_SetString(PyExc_TypeError,
+						"keywords must be strings");
+				goto fail;
+			}
 			/* XXX slow -- speed up using dictionary? */
 			for (j = 0; j < co->co_argcount; j++) {
 				PyObject *nm = PyTuple_GET_ITEM(