[Python-checkins] cpython: Fix sock_recvfrom_guts(): recvfrom() size is limited to an int on Windows, not

victor.stinner python-checkins at python.org
Sat Nov 16 00:34:19 CET 2013


http://hg.python.org/cpython/rev/7cd4c3e9e310
changeset:   87134:7cd4c3e9e310
user:        Victor Stinner <victor.stinner at gmail.com>
date:        Sat Nov 16 00:18:58 2013 +0100
summary:
  Fix sock_recvfrom_guts(): recvfrom() size is limited to an int on Windows, not
on other OSes!

files:
  Modules/socketmodule.c |  16 ++++++++--------
  1 files changed, 8 insertions(+), 8 deletions(-)


diff --git a/Modules/socketmodule.c b/Modules/socketmodule.c
--- a/Modules/socketmodule.c
+++ b/Modules/socketmodule.c
@@ -840,7 +840,7 @@
         return siz;
     }
     /* special-case broadcast - inet_addr() below can return INADDR_NONE for
-     * this */ 
+     * this */
     if (strcmp(name, "255.255.255.255") == 0 ||
         strcmp(name, "<broadcast>") == 0) {
         struct sockaddr_in *sin;
@@ -901,7 +901,7 @@
 #endif
             return 4;
         }
-    }   
+    }
 #endif /* HAVE_INET_PTON */
 
     /* perform a name resolution */
@@ -2833,7 +2833,7 @@
     memset(&addrbuf, 0, addrlen);
     timeout = internal_select_ex(s, 0, interval);
     if (!timeout) {
-#ifndef MS_WINDOWS
+#ifdef MS_WINDOWS
         if (len > INT_MAX)
             len = INT_MAX;
         n = recvfrom(s->sock_fd, cbuf, (int)len, flags,
@@ -4702,7 +4702,7 @@
     /* On UNIX, dup can be used to duplicate the file descriptor of a socket */
     newfd = _Py_dup(fd);
     if (newfd == INVALID_SOCKET)
-        return NULL; 
+        return NULL;
 #endif
 
     newfdobj = PyLong_FromSocket_t(newfd);
@@ -5093,7 +5093,7 @@
         return NULL;
     }
 
-    size = sizeof(addr); 
+    size = sizeof(addr);
     ret = WSAStringToAddressA(ip, af, NULL, (LPSOCKADDR)&addr, &size);
 
     if (ret) {
@@ -5101,10 +5101,10 @@
         return NULL;
     } else if(af == AF_INET) {
         struct sockaddr_in *addr4 = (struct sockaddr_in*)&addr;
-        return PyBytes_FromStringAndSize((const char *)&(addr4->sin_addr), 
+        return PyBytes_FromStringAndSize((const char *)&(addr4->sin_addr),
                                          sizeof(addr4->sin_addr));
     } else if (af == AF_INET6) {
-        return PyBytes_FromStringAndSize((const char *)&(addr.sin6_addr), 
+        return PyBytes_FromStringAndSize((const char *)&(addr.sin6_addr),
                                           sizeof(addr.sin6_addr));
     } else {
         PyErr_SetString(PyExc_OSError, "unknown address family");
@@ -5231,7 +5231,7 @@
     }
 
     retlen = sizeof(ip);
-    ret = WSAAddressToStringA((struct sockaddr*)&addr, addrlen, NULL, 
+    ret = WSAAddressToStringA((struct sockaddr*)&addr, addrlen, NULL,
                               ip, &retlen);
 
     if (ret) {

-- 
Repository URL: http://hg.python.org/cpython


More information about the Python-checkins mailing list