From 27a4ac535f112b87d91f433eb9edcd0ae9988354 Mon Sep 17 00:00:00 2001 From: Ronald Oussoren Date: Fri, 24 May 2013 13:47:37 +0200 Subject: [PATCH] Issue #17269: Workaround for a platform bug in getaddrinfo on OSX Without this patch socket.getaddrinfo crashed when called with some unusual argument combinations. --- Lib/test/test_socket.py | 3 +++ Misc/NEWS | 3 +++ Modules/socketmodule.c | 9 +++++++++ 3 files changed, 15 insertions(+) diff --git a/Lib/test/test_socket.py b/Lib/test/test_socket.py index d20e37bff6f..546a10d23a7 100644 --- a/Lib/test/test_socket.py +++ b/Lib/test/test_socket.py @@ -1166,6 +1166,9 @@ class GeneralModuleTests(unittest.TestCase): # Issue #6697. self.assertRaises(UnicodeEncodeError, socket.getaddrinfo, 'localhost', '\uD800') + # Issue 17269 + socket.getaddrinfo("localhost", None, 0, 0, 0, socket.AI_NUMERICSERV) + def test_getnameinfo(self): # only IP addresses are allowed self.assertRaises(socket.error, socket.getnameinfo, ('mail.python.org',0), 0) diff --git a/Misc/NEWS b/Misc/NEWS index a4e7101e017..125b153ee70 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -40,6 +40,9 @@ Library - Issue #17968: Fix memory leak in os.listxattr(). +- Issue #17269: Workaround for socket.getaddrinfo crash on MacOS X + with port None or "0" and flags AI_NUMERICSERV. + IDLE ---- diff --git a/Modules/socketmodule.c b/Modules/socketmodule.c index 92000024af3..679143cc936 100644 --- a/Modules/socketmodule.c +++ b/Modules/socketmodule.c @@ -5041,6 +5041,15 @@ socket_getaddrinfo(PyObject *self, PyObject *args, PyObject* kwargs) PyErr_SetString(PyExc_OSError, "Int or String expected"); goto err; } +#ifdef __APPLE__ + if ((flags & AI_NUMERICSERV) && (pptr == NULL || (pptr[0] == '0' && pptr[1] == 0))) { + /* On OSX upto at least OSX 10.8 getaddrinfo crashes + * if AI_NUMERICSERV is set and the servname is NULL or "0". + * This workaround avoids a segfault in libsystem. + */ + pptr = "00"; + } +#endif memset(&hints, 0, sizeof(hints)); hints.ai_family = family; hints.ai_socktype = socktype;