[PATCH] Fix tsocket on solaris

Stefan (metze) Metzmacher metze at samba.org
Wed Apr 7 11:32:07 MDT 2010


Stefan (metze) Metzmacher schrieb:
> Stefan (metze) Metzmacher schrieb:
>> Stefan (metze) Metzmacher schrieb:
>>> Andrew Bartlett schrieb:
>>>> Solaris seems to error out (EINVAL) if sendto has a socklen parameter
>>>> that isn't the same as the sizeof() for the relevant socket type.
>>>>
>>>> That is, we can't use sizeof(struct sockaddr_storage).
>>>>
>>>> This does not show up on the build farm, as we use socket_wrapper, which
>>>> plays other games.
>>>>
>>>> Please find attached and review my proposed patch, which instead tries
>>>> to keep the length alongside the struct sockaddr at all times. 
>>> here's my full version of the patch.
>>>
>>> It compiles, but I haven't done any real testing.
>>> make test failed, but I'm not sure the failure is really related.
> 
> I think this version should be ok, it fixes a pointer vs. stack variable
> bug in the recvfrom_handler.
> 
> But it still fails with the same error, maybe it's not really related...
> I'll do more tests tomorrow.

Ok, this one seems to work better, but I'll do manual tests tomorrow.

metze
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-tsocket_bsd-Always-use-a-real-length-for-the-sa_sock.patch
Type: text/x-diff
Size: 8012 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20100407/f5fa0cc3/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 260 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20100407/f5fa0cc3/attachment.pgp>


More information about the samba-technical mailing list