From: Guido van Rossum <guido@python.org>
Date: Thu, 9 Aug 2007 21:06:37 +0000 (+0000)
Subject: Fix two bugs in socket_getaddr():
X-Git-Tag: v3.0a1~477
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9ce8e3809906a0e7a01b548883b85d0e323d2f6d;p=python

Fix two bugs in socket_getaddr():
(a) 'single' shouldn't be DECREF'ed in the cleanup code;
(b) the fallback case in makesockaddr() should use y# instead of s#
    in the format, since the data is bytes, not UTF-8 text.
---

diff --git a/Modules/socketmodule.c b/Modules/socketmodule.c
index d9e984486c..27c2152fd9 100644
--- a/Modules/socketmodule.c
+++ b/Modules/socketmodule.c
@@ -1174,8 +1174,8 @@ makesockaddr(int sockfd, struct sockaddr *addr, int addrlen, int proto)
 
 	default:
 		/* If we don't know the address family, don't raise an
-		   exception -- return it as a tuple. */
-		return Py_BuildValue("is#",
+		   exception -- return it as an (int, bytes) tuple. */
+		return Py_BuildValue("iy#",
 				     addr->sa_family,
 				     addr->sa_data,
 				     sizeof(addr->sa_data));
@@ -3743,7 +3743,6 @@ socket_getaddrinfo(PyObject *self, PyObject *args)
 	int family, socktype, protocol, flags;
 	int error;
 	PyObject *all = (PyObject *)NULL;
-	PyObject *single = (PyObject *)NULL;
 	PyObject *idna = NULL;
 
 	family = socktype = protocol = flags = 0;
@@ -3797,6 +3796,7 @@ socket_getaddrinfo(PyObject *self, PyObject *args)
 	if ((all = PyList_New(0)) == NULL)
 		goto err;
 	for (res = res0; res; res = res->ai_next) {
+		PyObject *single;
 		PyObject *addr =
 			makesockaddr(-1, res->ai_addr, res->ai_addrlen, protocol);
 		if (addr == NULL)
@@ -3818,7 +3818,6 @@ socket_getaddrinfo(PyObject *self, PyObject *args)
 		freeaddrinfo(res0);
 	return all;
  err:
-	Py_XDECREF(single);
 	Py_XDECREF(all);
 	Py_XDECREF(idna);
 	if (res0)