[PATCH] swrap: Use the loaded libc open() directly.

Stefan (metze) Metzmacher metze at samba.org
Mon May 5 03:30:05 MDT 2014

Hi Volker,

>>> Andreas' socket wrapper repo contains a ciritical bug fix
>>> that should go into Samba master asap. Attached find the
>>> version that applies to master. Please review & push!
>>> Thanks,
>>> Volker
>> Please do not push this patch. Use the patch from
>> https://git.samba.org/?p=asn/samba.git;a=shortlog;h=refs/heads/swrap
> -1 on this one.
> Here we lose the commit history with the individual
> bugfixes. Just "socket wrapper 1.0.2" is not sufficient to
> find the individual patches. Please rebase the individual
> patches from socket wrapper.

The history is available at
The happens similar to an import of heimdal. We only import the files we
(socket_wrapper.c in this case) of a wellknown snapshot of the external
And 'wscript' is only part of the samba tree.

gets my review because the content of socket_wrapper.c (95643aa) is the
same as
last change in socket wrapper 1.0.2, see

I'm fine if we would also add the hash of the socket_wrapper-1.0.2 tag
and a reference to the repository url into the commit message...


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140505/02e79ee1/attachment.pgp>

More information about the samba-technical mailing list