diff options
author | Victor Stinner <victor.stinner@gmail.com> | 2018-01-18 10:15:25 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-01-18 10:15:25 (GMT) |
commit | 05d68a8bd84cb141be9f9335f5b3540f15a989c4 (patch) | |
tree | a4e48a3b3c5b46422f83ffb9b4314eb8ea344325 /Python | |
parent | ab74504346a6e2569b3255b7b621c589716888c4 (diff) | |
download | cpython-05d68a8bd84cb141be9f9335f5b3540f15a989c4.zip cpython-05d68a8bd84cb141be9f9335f5b3540f15a989c4.tar.gz cpython-05d68a8bd84cb141be9f9335f5b3540f15a989c4.tar.bz2 |
bpo-9566: Fix size_t=>int downcast warnings (#5230)
* Use wider types (int => Py_ssize_t) to avoid integer overflows.
* Fix gc.get_freeze_count(): use Py_ssize_t type rather than int, since gc_list_size() returns a Py_ssize_t.
Diffstat (limited to 'Python')
-rw-r--r-- | Python/ast_opt.c | 2 | ||||
-rw-r--r-- | Python/bltinmodule.c | 4 | ||||
-rw-r--r-- | Python/pathconfig.c | 8 |
3 files changed, 7 insertions, 7 deletions
diff --git a/Python/ast_opt.c b/Python/ast_opt.c index f98a28a..65cf3c1 100644 --- a/Python/ast_opt.c +++ b/Python/ast_opt.c @@ -397,7 +397,7 @@ fold_compare(expr_ty node, PyArena *arena, int optimize) { asdl_int_seq *ops; asdl_seq *args; - int i; + Py_ssize_t i; ops = node->v.Compare.ops; args = node->v.Compare.comparators; diff --git a/Python/bltinmodule.c b/Python/bltinmodule.c index 844548f..7fc2261 100644 --- a/Python/bltinmodule.c +++ b/Python/bltinmodule.c @@ -52,9 +52,9 @@ _Py_IDENTIFIER(stderr); #include "clinic/bltinmodule.c.h" static PyObject* -update_bases(PyObject *bases, PyObject *const *args, int nargs) +update_bases(PyObject *bases, PyObject *const *args, Py_ssize_t nargs) { - int i, j; + Py_ssize_t i, j; PyObject *base, *meth, *new_base, *result, *new_bases = NULL; PyObject *stack[1] = {bases}; assert(PyTuple_Check(bases)); diff --git a/Python/pathconfig.c b/Python/pathconfig.c index 7ebd69b..6de5481 100644 --- a/Python/pathconfig.c +++ b/Python/pathconfig.c @@ -367,13 +367,12 @@ _Py_FindEnvConfigValue(FILE *env_file, const wchar_t *key, fseek(env_file, 0, SEEK_SET); while (!feof(env_file)) { char * p = fgets(buffer, MAXPATHLEN*2, env_file); - wchar_t *tmpbuffer; - int n; if (p == NULL) { break; } - n = strlen(p); + + size_t n = strlen(p); if (p[n - 1] != '\n') { /* line has overflowed - bail */ break; @@ -382,7 +381,8 @@ _Py_FindEnvConfigValue(FILE *env_file, const wchar_t *key, /* Comment - skip */ continue; } - tmpbuffer = _Py_DecodeUTF8_surrogateescape(buffer, n); + + wchar_t *tmpbuffer = _Py_DecodeUTF8_surrogateescape(buffer, n); if (tmpbuffer) { wchar_t * state; wchar_t * tok = wcstok(tmpbuffer, L" \t\r\n", &state); |