summaryrefslogtreecommitdiffstats
path: root/Python
diff options
context:
space:
mode:
authorVictor Stinner <victor.stinner@gmail.com>2018-01-18 10:15:25 (GMT)
committerGitHub <noreply@github.com>2018-01-18 10:15:25 (GMT)
commit05d68a8bd84cb141be9f9335f5b3540f15a989c4 (patch)
treea4e48a3b3c5b46422f83ffb9b4314eb8ea344325 /Python
parentab74504346a6e2569b3255b7b621c589716888c4 (diff)
downloadcpython-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.c2
-rw-r--r--Python/bltinmodule.c4
-rw-r--r--Python/pathconfig.c8
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);