Repository navigation
Commit e9d9673
Set default arguments in _testlimitedcapi/unicode.c (GH-159380)
Some optional arguments had no default value leading to undefined
behavior if they were omitted.
(cherry picked from commit e96d943)
Co-authored-by: Victor Stinner <vstinner@python.org>
1 parent 3396c34 commit e9d9673
1 file changed
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -946,7 +946,7 @@ unicode_encodecodepage(PyObject *self, PyObject *args) | |||
| 946 | 946 | { | |
| 947 | 947 | int code_page; | |
| 948 | 948 | PyObject *unicode; | |
| 949 | - const char *errors; | ||
| 949 | + const char *errors = NULL; | ||
| 950 | 950 | ||
| 951 | 951 | if (!PyArg_ParseTuple(args, "iO|z", &code_page, &unicode, &errors)) | |
| 952 | 952 | return NULL; | |
@@ -963,7 +963,7 @@ unicode_decodelocaleandsize(PyObject *self, PyObject *args) | |||
| 963 | 963 | { | |
| 964 | 964 | const char *data; | |
| 965 | 965 | Py_ssize_t size; | |
| 966 | - const char *errors; | ||
| 966 | + const char *errors = NULL; | ||
| 967 | 967 | ||
| 968 | 968 | if (!PyArg_ParseTuple(args, "y#|z", &data, &size, &errors)) | |
| 969 | 969 | return NULL; | |
@@ -977,7 +977,7 @@ unicode_decodelocale(PyObject *self, PyObject *args) | |||
| 977 | 977 | { | |
| 978 | 978 | const char *data; | |
| 979 | 979 | Py_ssize_t size; | |
| 980 | - const char *errors; | ||
| 980 | + const char *errors = NULL; | ||
| 981 | 981 | ||
| 982 | 982 | if (!PyArg_ParseTuple(args, "y#|z", &data, &size, &errors)) | |
| 983 | 983 | return NULL; | |
@@ -990,7 +990,7 @@ static PyObject * | |||
| 990 | 990 | unicode_encodelocale(PyObject *self, PyObject *args) | |
| 991 | 991 | { | |
| 992 | 992 | PyObject *unicode; | |
| 993 | - const char *errors; | ||
| 993 | + const char *errors = NULL; | ||
| 994 | 994 | ||
| 995 | 995 | if (!PyArg_ParseTuple(args, "O|z", &unicode, &errors)) | |
| 996 | 996 | return NULL; | |
@@ -1284,7 +1284,7 @@ unicode_comparewithasciistring(PyObject *self, PyObject *args) | |||
| 1284 | 1284 | { | |
| 1285 | 1285 | PyObject *left; | |
| 1286 | 1286 | const char *right = NULL; | |
| 1287 | - Py_ssize_t right_len; | ||
| 1287 | + Py_ssize_t right_len = 0; | ||
| 1288 | 1288 | int result; | |
| 1289 | 1289 | ||
| 1290 | 1290 | if (!PyArg_ParseTuple(args, "O|y#", &left, &right, &right_len)) | |
@@ -1323,7 +1323,7 @@ unicode_equaltoutf8andsize(PyObject *self, PyObject *args) | |||
| 1323 | 1323 | { | |
| 1324 | 1324 | PyObject *left; | |
| 1325 | 1325 | const char *right = NULL; | |
| 1326 | - Py_ssize_t right_len; | ||
| 1326 | + Py_ssize_t right_len = 0; | ||
| 1327 | 1327 | Py_ssize_t size = -100; | |
| 1328 | 1328 | int result; | |
| 1329 | 1329 | ||
0 commit comments