Skip to content

gh-68114: Fix handling for removed PyArg_ParseTuple formatters #8204

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
encukou merged 11 commits into from
Apr 23, 2024
Merged

gh-68114: Fix handling for removed PyArg_ParseTuple formatters #8204

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
5a275c8
Fix handling for removed PyArg_ParseTuple formatters
Jul 9, 2018
cbc1897
Merge in the main branch
encukou Mar 18, 2024
65aec54
Merge comments
encukou Mar 18, 2024
6c948fb
Recreate the blurb
encukou Mar 18, 2024
c2a63a2
Fix the test
encukou Mar 18, 2024
4fbed15
Fix style
encukou Mar 18, 2024
3aff049
Ensure the invalid args result in SystemError
encukou Mar 18, 2024
ae9b1e2
Fix functionality, and add additional test
encukou Mar 18, 2024
30e7def
Unmark 'w#' as supported in test_skipitem_with_suffix
encukou Mar 19, 2024
2aee71c
Don't omit parameter names
encukou Mar 19, 2024
11eca72
Merge branch 'main' into remove-old-formatters
encukou Apr 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix the test 
- Don't use removed trivial helper raiseTestError
- Decref kwargs if it's created
- Export the test so that it is run
  • Loading branch information
@encukou
encukou committed Mar 18, 2024
commit c2a63a2beeb25dcd3ad17eadfa8aeb11c463f258
25 changes: 19 additions & 6 deletions Modules/_testcapi/getargs.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ getargs_w_star(PyObject *self, PyObject *args)

/* Test the old w and w# codes that no longer work */
static PyObject *
test_w_code_invalid(PyObject *self)
test_w_code_invalid(PyObject *, PyObject *)
{
static const char * const keywords[] = {"a", "b", "c", "d", NULL};
char *formats_3[] = {"O|w#$O",
Expand All @@ -167,7 +167,13 @@ test_w_code_invalid(PyObject *self)
return NULL;
}

if (!(kwargs = PyDict_New()) || PyDict_SetItemString(kwargs, "c", Py_None)) {
kwargs = PyDict_New();
if (!kwargs) {
Py_DECREF(args);
return NULL;
}

if (PyDict_SetItemString(kwargs, "c", Py_None)) {
Py_DECREF(args);
Py_XDECREF(kwargs);
return NULL;
Expand All @@ -178,8 +184,11 @@ test_w_code_invalid(PyObject *self)
(char**) keywords,
&tmp, &tmp, &tmp)) {
Py_DECREF(args);
return raiseTestError("test_w_code_invalid_suffix",
formats_3[n]);
Py_DECREF(kwargs);
PyErr_Format(PyExc_AssertionError,
"test_w_code_invalid_suffix: %s",
formats_3[n]);
return NULL;
}
else {
PyErr_Clear();
Expand All @@ -199,8 +208,11 @@ test_w_code_invalid(PyObject *self)
(char**) keywords,
&tmp, &tmp, &tmp, &tmp)) {
Py_DECREF(args);
return raiseTestError("test_w_code_invalid_suffix",
formats_4[n]);
Py_DECREF(kwargs);
PyErr_Format(PyExc_AssertionError,
"test_w_code_invalid_suffix: %s",
formats_4[n]);
return NULL;
}
else {
PyErr_Clear();
Expand Down Expand Up @@ -764,6 +776,7 @@ static PyMethodDef test_methods[] = {
{"getargs_z_star", getargs_z_star, METH_VARARGS},
{"parse_tuple_and_keywords", parse_tuple_and_keywords, METH_VARARGS},
{"gh_99240_clear_args", gh_99240_clear_args, METH_VARARGS},
{"test_w_code_invalid", test_w_code_invalid, METH_NOARGS},
{NULL},
};

Expand Down