socketmodule on OpenBSD/sparc64 (64bit machine)

[pelotas]

BPO	992055
Nosy	@loewis
Files	patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2004-07-17.08:12:38.000>
created_at = <Date 2004-07-16.00:47:33.000>
labels = []
title = 'socketmodule on OpenBSD/sparc64 (64bit machine)'
updated_at = <Date 2004-07-17.08:12:38.000>
user = 'https://bugs.python.org/pelotas'

bugs.python.org fields:

activity = <Date 2004-07-17.08:12:38.000>
actor = 'loewis'
assignee = 'none'
closed = True
closed_date = None
closer = None
components = ['None']
creation = <Date 2004-07-16.00:47:33.000>
creator = 'pelotas'
dependencies = []
files = ['6091']
hgrepos = []
issue_num = 992055
keywords = ['patch']
message_count = 3.0
messages = ['46376', '46377', '46378']
nosy_count = 2.0
nosy_names = ['loewis', 'pelotas']
pr_nums = []
priority = 'normal'
resolution = 'rejected'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue992055'
versions = []

[pelotas]

perky@users.sourceforge.net
<perky@users.sourceforge.net> wrote:

Update of /cvsroot/python/python/dist/src/Modules

In directory sc8-pr-cvs1:/tmp/cvs-serv3617

Modified Files:

  socketmodule.c                                

Log Message:

Fix erroneus argument parsing of socket.htons() on
64bit big endian

machines.

This is cool but not enough. There is another place
where 64 bit machines fail. Here is diff. With it my
OpenBSD/sparc64 passes all socket tests (test_socket).
Of course OpenBSD/i386 is still working OK.

[diff attached]

Hye-Shik Chang had already replied. Here is what he said:

[...] But return value type of inet_addr(3) and

        its size has complex issues because it is

used for not only OpenBSD but
also BeOS,
IRIX, old BSDs, Windows and etc. that I don't have an
access.
I think you'd better to submit it to
SF tracker [...]

So I'm submiting it.

[pelotas]

Logged In: YES
user_id=883833

Yuck. This looks ugly. Close this patch and I'll resubmit
it. Sorry :/

[loewis]

Logged In: YES
user_id=21627

Ok, closed.