This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author EdSchouten
Recipients EdSchouten
Date 2016-11-05.11:37:20
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1478345841.6.0.691091432417.issue28619@psf.upfronthosting.co.za>
In-reply-to
Content
Modern C code should use inet_ntop()/inet_pton() as opposed to inet_addr()/inet_aton()/inet_ntoa().

Though the former functions may typically act as drop-in replacements for the latter, the inet_addr()/inet_aton() functions still have the advantage over inet_pton() of allowing you to parse IPv4 addresses that don't use the dotted quad notation (e.g. '0x0a000001' for 10.0.0.1).

There is no difference between inet_ntop() and inet_ntoa(), as they both always print the address in dotted quad form. inet_ntop() does have the advantage of being thread-safe, as inet_ntoa() uses internal storage for the return value. In other words, we'd better not use inet_ntoa() at all.

Attached is a patch for Python's socketmodule that changes the existing call to inet_ntoa() to use inet_ntop() when available. This has the advantage of fixing the build on CloudABI (https://mail.python.org/pipermail/python-dev/2016-July/145708.html), which intentionally omits any APIs that are thread-unsafe.
History
Date User Action Args
2016-11-05 11:37:21EdSchoutensetrecipients: + EdSchouten
2016-11-05 11:37:21EdSchoutensetmessageid: <1478345841.6.0.691091432417.issue28619@psf.upfronthosting.co.za>
2016-11-05 11:37:21EdSchoutenlinkissue28619 messages
2016-11-05 11:37:20EdSchoutencreate