Re: [PATCH] Fix gcc-4 warning in accept() call
- From
- H. Peter Anvin <hpa@zytor.com>
- Date
- Sep 30, 2005, 04:46 UTC
- Message-ID
- <433CC31A.9090808@zytor.com>
- In-Reply-To
- <20050930000917.rrwggw4g0gogs8w4@webmail.spamcop.net>
Pavel Roskin wrote:
Show 14 quoted lines
> Quoting Horst von Brand <vonbrand@inf.utfsm.cl>: > > >>>+ unsigned int sslen = sizeof(ss); >> >>Shouldn't this be size_t? > > > No. That would actually break things on big-endian 64-bit platforms. > > You can see "info libc" for the whole story. My (limited!) understanding is > that it should be 32-bit for compatibility reasons, so socklen_t was a trick to > fool size_t proponents from the POSIX committee :-) >
Formally it should be "socklen_t" or "int". It should *not* be "unsigned int"!
-hpa