uid_wrapper: Add uwrap_setresuid().
Stefan (metze) Metzmacher
metze at samba.org
Sat Oct 8 02:30:44 MDT 2011
Am 08.10.2011 10:01, schrieb Andrew Bartlett:
> On Sat, 2011-10-08 at 09:44 +0200, Andreas Schneider wrote:
>> On Saturday 08 October 2011 10:42:45 Andrew Bartlett wrote:
>>> On Fri, 2011-10-07 at 12:05 +0200, Andreas Schneider wrote:
>>>> - Log -----------------------------------------------------------------
>>>> commit 4493c578b0da44ae2100cc0d4d6acc714bf39a3f
>>>> Author: Andreas Schneider <asn at samba.org>
>>>> Date: Fri Oct 7 10:30:23 2011 +0200
>>>> uid_wrapper: Add uwrap_setresuid().
>>>> Autobuild-User: Andreas Schneider <asn at cryptomilk.org>
>>>> Autobuild-Date: Fri Oct 7 12:04:05 CEST 2011 on sn-devel-104
>>> It's really great to see UID wrapper being ported into the source3 code,
>>> but I just wanted to mention that this seems to have broken the Solaris
>>> I'm sure the fix is pretty simple, I just wanted to bring it to your
>> there is more stuff broken than I thought, and I think I've fixed it now.
>> I just wanted that someone looks over it before I push it.
> Thanks. I'm OK with these patches, except for
> The problem I have this this patch is if overriding the get/set uid
> functions requires that we include a particular header, but the build
> can continue without that header specified, we can get in a lot of
> trouble quickly. We should ensure that we do not obtain the system
> prototypes for the replaced functions except via the header that has the
> uid_wrapper replacements.
> If we don't, we risk subtle, painful to debug issues in our test
> environment (the same applies for socket_wrapper, and it is from that
> experience that I give this warning).
> I do realise this is difficult with geteuid() being defined via
> unistd.h, but this is yet another reason why all Samba code should only
> include system/ headers, not system headers directly.
I think Andreas should push this changes to make it work at all.
Then we can think about a good generic strategie how to prevent such
for all wrapper libraries.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 262 bytes
Desc: OpenPGP digital signature
More information about the samba-technical