From 954900a3f98a8c0dea14dd575490237f3f8626b3 Mon Sep 17 00:00:00 2001 From: bggardner Date: Thu, 12 Sep 2019 06:02:48 -0400 Subject: [PATCH] closes bpo-37405: Make socket.getsockname() always return a tuple for AF_CAN. (GH-14392) This fixes a regression from 3.5. In recent releases, `getsockname()` in the AF_CAN case has returned a string. --- Lib/test/test_socket.py | 4 +++- .../next/Library/2019-09-11-20-27-41.bpo-37405.MG5xiY.rst | 2 ++ Modules/socketmodule.c | 2 +- 3 files changed, 6 insertions(+), 2 deletions(-) create mode 100644 Misc/NEWS.d/next/Library/2019-09-11-20-27-41.bpo-37405.MG5xiY.rst diff --git a/Lib/test/test_socket.py b/Lib/test/test_socket.py index b855c523f0..b74549024b 100755 --- a/Lib/test/test_socket.py +++ b/Lib/test/test_socket.py @@ -1948,7 +1948,9 @@ class BasicCANTest(unittest.TestCase): def testBindAny(self): with socket.socket(socket.PF_CAN, socket.SOCK_RAW, socket.CAN_RAW) as s: - s.bind(('', )) + address = ('', ) + s.bind(address) + self.assertEqual(s.getsockname(), address) def testTooLongInterfaceName(self): # most systems limit IFNAMSIZ to 16, take 1024 to be sure diff --git a/Misc/NEWS.d/next/Library/2019-09-11-20-27-41.bpo-37405.MG5xiY.rst b/Misc/NEWS.d/next/Library/2019-09-11-20-27-41.bpo-37405.MG5xiY.rst new file mode 100644 index 0000000000..09e1097342 --- /dev/null +++ b/Misc/NEWS.d/next/Library/2019-09-11-20-27-41.bpo-37405.MG5xiY.rst @@ -0,0 +1,2 @@ +Fixed regression bug for socket.getsockname() for non-CAN_ISOTP AF_CAN +address family sockets by returning a 1-tuple instead of string. diff --git a/Modules/socketmodule.c b/Modules/socketmodule.c index 3548b0c878..c649fa3c98 100644 --- a/Modules/socketmodule.c +++ b/Modules/socketmodule.c @@ -1563,7 +1563,7 @@ makesockaddr(SOCKET_T sockfd, struct sockaddr *addr, size_t addrlen, int proto) #endif /* CAN_ISOTP */ default: { - return Py_BuildValue("O&", PyUnicode_DecodeFSDefault, + return Py_BuildValue("(O&)", PyUnicode_DecodeFSDefault, ifname); } } -- 2.40.0