pythoncapi
diff --git a/‎Lib/test/test_call.py‎
Lines changed: 26 additions & 0 deletions b/‎Lib/test/test_call.py‎
Lines changed: 26 additions & 0 deletions
diff --git a/‎Modules/_hashopenssl.c‎
Lines changed: 2 additions & 2 deletions b/‎Modules/_hashopenssl.c‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎Modules/_struct.c‎
Lines changed: 14 additions & 6 deletions b/‎Modules/_struct.c‎
Lines changed: 14 additions & 6 deletions
diff --git a/‎Python/bltinmodule.c‎
Lines changed: 6 additions & 6 deletions b/‎Python/bltinmodule.c‎
Lines changed: 6 additions & 6 deletions
@@ -5,6 +5,8 @@
     import _testcapi
 except ImportError:
     _testcapi = None
+import struct
+import collections
 
 # The test cases here cover several paths through the function calling
 # code.  They depend on the METH_XXX flag that is used to define a C
@@ -160,6 +162,30 @@ def test_varargs5_kw(self):
         msg = r"^hasattr\(\) takes no keyword arguments$"
         self.assertRaisesRegex(TypeError, msg, hasattr, x=2)
 
+    def test_varargs6_kw(self):
+        msg = r"^getattr\(\) takes no keyword arguments$"
+        self.assertRaisesRegex(TypeError, msg, getattr, x=2)
+
+    def test_varargs7_kw(self):
+        msg = r"^next\(\) takes no keyword arguments$"
+        self.assertRaisesRegex(TypeError, msg, next, x=2)
+
+    def test_varargs8_kw(self):
+        msg = r"^pack\(\) takes no keyword arguments$"
+        self.assertRaisesRegex(TypeError, msg, struct.pack, x=2)
+
+    def test_varargs9_kw(self):
+        msg = r"^pack_into\(\) takes no keyword arguments$"
+        self.assertRaisesRegex(TypeError, msg, struct.pack_into, x=2)
+
+    def test_varargs10_kw(self):
+        msg = r"^index\(\) takes no keyword arguments$"
+        self.assertRaisesRegex(TypeError, msg, collections.deque().index, x=2)
+
+    def test_varargs11_kw(self):
+        msg = r"^pack\(\) takes no keyword arguments$"
+        self.assertRaisesRegex(TypeError, msg, struct.Struct.pack, struct.Struct(""), x=2)
+
     def test_oldargs0_1(self):
         msg = r"keys\(\) takes no arguments \(1 given\)"
         self.assertRaisesRegex(TypeError, msg, {}.keys, 0)
 
@@ -931,11 +931,11 @@ generate_hash_name_list(void)
         Py_buffer view = { 0 }; \
         PyObject *ret_obj; \
      \
-        if (!_PyArg_ParseStack(args, nargs, "|O:" #NAME , &data_obj)) { \
+        if (!_PyArg_NoStackKeywords(#NAME, kwnames)) { \
             return NULL; \
         } \
      \
-        if (!_PyArg_NoStackKeywords(#NAME, kwnames)) { \
+        if (!_PyArg_ParseStack(args, nargs, "|O:" #NAME , &data_obj)) { \
             return NULL; \
         } \
      \
 
@@ -1823,6 +1823,9 @@ s_pack(PyObject *self, PyObject **args, Py_ssize_t nargs, PyObject *kwnames)
     PyObject *result;
 
     /* Validate arguments. */
+    if (!_PyArg_NoStackKeywords("pack", kwnames)) {
+        return NULL;
+    }
     soself = (PyStructObject *)self;
     assert(PyStruct_Check(self));
     assert(soself->s_codes != NULL);
@@ -1832,9 +1835,6 @@ s_pack(PyObject *self, PyObject **args, Py_ssize_t nargs, PyObject *kwnames)
             "pack expected %zd items for packing (got %zd)", soself->s_len, nargs);
         return NULL;
     }
-    if (!_PyArg_NoStackKeywords("pack", kwnames)) {
-        return NULL;
-    }
 
     /* Allocate a new buffer */
     result = PyBytes_FromStringAndSize((char *)NULL, soself->s_size);
@@ -1866,6 +1866,9 @@ s_pack_into(PyObject *self, PyObject **args, Py_ssize_t nargs, PyObject *kwnames
     Py_ssize_t offset;
 
     /* Validate arguments.  +1 is for the first arg as buffer. */
+    if (!_PyArg_NoStackKeywords("pack_into", kwnames)) {
+        return NULL;
+    }
     soself = (PyStructObject *)self;
     assert(PyStruct_Check(self));
     assert(soself->s_codes != NULL);
@@ -1886,9 +1889,6 @@ s_pack_into(PyObject *self, PyObject **args, Py_ssize_t nargs, PyObject *kwnames
         }
         return NULL;
     }
-    if (!_PyArg_NoStackKeywords("pack_into", kwnames)) {
-        return NULL;
-    }
 
     /* Extract a writable memory buffer from the first argument */
     if (!PyArg_Parse(args[0], "w*", &buffer))
@@ -2131,6 +2131,10 @@ pack(PyObject *self, PyObject **args, Py_ssize_t nargs, PyObject *kwnames)
     PyObject *s_object = NULL;
     PyObject *format, *result;
 
+    if (!_PyArg_NoStackKeywords("pack", kwnames)) {
+        return NULL;
+    }
+
     if (nargs == 0) {
         PyErr_SetString(PyExc_TypeError, "missing format argument");
         return NULL;
@@ -2159,6 +2163,10 @@ pack_into(PyObject *self, PyObject **args, Py_ssize_t nargs, PyObject *kwnames)
     PyObject *s_object = NULL;
     PyObject *format, *result;
 
+    if (!_PyArg_NoStackKeywords("pack_into", kwnames)) {
+        return NULL;
+    }
+
     if (nargs == 0) {
         PyErr_SetString(PyExc_TypeError, "missing format argument");
         return NULL;
 
@@ -997,13 +997,13 @@ builtin_getattr(PyObject *self, PyObject **args, Py_ssize_t nargs,
     PyObject *v, *result, *dflt = NULL;
     PyObject *name;
 
-    if (!_PyArg_UnpackStack(args, nargs, "getattr", 2, 3, &v, &name, &dflt))
-        return NULL;
-
     if (!_PyArg_NoStackKeywords("getattr", kwnames)) {
         return NULL;
     }
 
+    if (!_PyArg_UnpackStack(args, nargs, "getattr", 2, 3, &v, &name, &dflt))
+        return NULL;
+
     if (!PyUnicode_Check(name)) {
         PyErr_SetString(PyExc_TypeError,
                         "getattr(): attribute name must be string");
@@ -1307,13 +1307,13 @@ builtin_next(PyObject *self, PyObject **args, Py_ssize_t nargs,
     PyObject *it, *res;
     PyObject *def = NULL;
 
-    if (!_PyArg_UnpackStack(args, nargs, "next", 1, 2, &it, &def))
-        return NULL;
-
     if (!_PyArg_NoStackKeywords("next", kwnames)) {
         return NULL;
     }
 
+    if (!_PyArg_UnpackStack(args, nargs, "next", 1, 2, &it, &def))
+        return NULL;
+
     if (!PyIter_Check(it)) {
         PyErr_Format(PyExc_TypeError,
             "'%.200s' object is not an iterator",
Original file line number	Diff line number	Diff line change
`@@ -931,11 +931,11 @@ generate_hash_name_list(void)`
`931`	`931`	`Py_buffer view = { 0 }; \`
`932`	`932`	`PyObject *ret_obj; \`
`933`	`933`	`\`
`934`		`- if (!_PyArg_ParseStack(args, nargs, "\|O:" #NAME , &data_obj)) { \`
	`934`	`+ if (!_PyArg_NoStackKeywords(#NAME, kwnames)) { \`
`935`	`935`	`return NULL; \`
`936`	`936`	`} \`
`937`	`937`	`\`
`938`		`- if (!_PyArg_NoStackKeywords(#NAME, kwnames)) { \`
	`938`	`+ if (!_PyArg_ParseStack(args, nargs, "\|O:" #NAME , &data_obj)) { \`
`939`	`939`	`return NULL; \`
`940`	`940`	`} \`
`941`	`941`	`\`