diff options
| author | Victor Stinner <victor.stinner@haypocalc.com> | 2011-03-21 02:22:50 (GMT) |
|---|---|---|
| committer | Victor Stinner <victor.stinner@haypocalc.com> | 2011-03-21 02:22:50 (GMT) |
| commit | 451385d8f8e8bedd347d363fd0e01288226daee2 (patch) | |
| tree | 3c7769aa1c730f5b02f9462293ffd69f5ef1b001 /Python | |
| parent | e788ace5abccfd89f94a28630d6e0019921fe2cb (diff) | |
| download | cpython-451385d8f8e8bedd347d363fd0e01288226daee2.zip cpython-451385d8f8e8bedd347d363fd0e01288226daee2.tar.gz cpython-451385d8f8e8bedd347d363fd0e01288226daee2.tar.bz2 | |
Issue #8651: PyArg_Parse*() functions raise an OverflowError if the file
doesn't have PY_SSIZE_T_CLEAN define and the size doesn't fit in an int
(length bigger than 2^31-1 bytes).
Diffstat (limited to 'Python')
| -rw-r--r-- | Python/getargs.c | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/Python/getargs.c b/Python/getargs.c index 686eac5..0009b35 100644 --- a/Python/getargs.c +++ b/Python/getargs.c @@ -613,7 +613,17 @@ convertsimple(PyObject *arg, const char **p_format, va_list *p_va, int flags, #define FETCH_SIZE int *q=NULL;Py_ssize_t *q2=NULL;\ if (flags & FLAG_SIZE_T) q2=va_arg(*p_va, Py_ssize_t*); \ else q=va_arg(*p_va, int*); -#define STORE_SIZE(s) if (flags & FLAG_SIZE_T) *q2=s; else *q=s; +#define STORE_SIZE(s) \ + if (flags & FLAG_SIZE_T) \ + *q2=s; \ + else { \ + if (INT_MAX < s) { \ + PyErr_SetString(PyExc_OverflowError, \ + "size does not fit in an int"); \ + return converterr("", arg, msgbuf, bufsize); \ + } \ + *q=s; \ + } #define BUFFER_LEN ((flags & FLAG_SIZE_T) ? *q2:*q) const char *format = *p_format; |