From 1df9fdd4d5ba2c1a0c4731dada51780e5b5968a7 Mon Sep 17 00:00:00 2001 From: Tim Peters Date: Thu, 13 Feb 2003 03:13:40 +0000 Subject: socket_inet_aton(): ip_addr was left undefined before use in the !HAVE_INET_ATON case. Repaired that, and tried to repair what looked like out-of-date comments. --- Modules/socketmodule.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/Modules/socketmodule.c b/Modules/socketmodule.c index f7bbeae..1b3321f 100644 --- a/Modules/socketmodule.c +++ b/Modules/socketmodule.c @@ -2710,16 +2710,19 @@ socket_inet_aton(PyObject *self, PyObject *args) #ifndef INADDR_NONE #define INADDR_NONE (-1) #endif - - /* Have to use inet_addr() instead */ - char *ip_addr; #ifdef HAVE_INET_ATON struct in_addr buf; +#else + /* Have to use inet_addr() instead */ + unsigned long packed_addr; +#endif + char *ip_addr; - if (!PyArg_ParseTuple(args, "s:inet_aton", &ip_addr)) { + if (!PyArg_ParseTuple(args, "s:inet_aton", &ip_addr)) return NULL; - } + +#ifdef HAVE_INET_ATON if (inet_aton(ip_addr, &buf)) return PyString_FromStringAndSize((char *)(&buf), sizeof(buf)); @@ -2728,11 +2731,9 @@ socket_inet_aton(PyObject *self, PyObject *args) "illegal IP address string passed to inet_aton"); return NULL; -#else /* In case you don't have inet_aton() */ +#else /* ! HAVE_INET_ATON */ /* XXX Problem here: inet_aton('255.255.255.255') raises an exception while it should be a valid address. */ - unsigned long packed_addr; - packed_addr = inet_addr(ip_addr); if (packed_addr == INADDR_NONE) { /* invalid address */ @@ -2740,7 +2741,6 @@ socket_inet_aton(PyObject *self, PyObject *args) "illegal IP address string passed to inet_aton"); return NULL; } - return PyString_FromStringAndSize((char *) &packed_addr, sizeof(packed_addr)); #endif -- cgit v0.12