[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