]> granicus.if.org Git - python/commitdiff
Fix os.confstr(): the result type of the C function is size_t, not int
authorVictor Stinner <victor.stinner@gmail.com>
Tue, 25 Jun 2013 21:13:47 +0000 (23:13 +0200)
committerVictor Stinner <victor.stinner@gmail.com>
Tue, 25 Jun 2013 21:13:47 +0000 (23:13 +0200)
Modules/posixmodule.c

index fd4628c76379d4853738e427231b9b693c1cb5a7..226a4d510b120c18173d73e67821054285283726 100644 (file)
@@ -9140,7 +9140,7 @@ posix_confstr(PyObject *self, PyObject *args)
     PyObject *result = NULL;
     int name;
     char buffer[255];
-    int len;
+    size_t len;
 
     if (!PyArg_ParseTuple(args, "O&:confstr", conv_confstr_confname, &name))
         return NULL;
@@ -9157,7 +9157,7 @@ posix_confstr(PyObject *self, PyObject *args)
         }
     }
 
-    if ((unsigned int)len >= sizeof(buffer)) {
+    if (len >= sizeof(buffer)) {
         char *buf = PyMem_Malloc(len);
         if (buf == NULL)
             return PyErr_NoMemory();