Patchset to add asynchronous open/close to master
Jeremy Allison
jra at samba.org
Thu Jun 21 16:31:42 MDT 2012
On Thu, Jun 21, 2012 at 06:15:03PM -0400, simo wrote:
>
> Why the getxxx ?
Paranoia :-).
> Don't we get an error back from the syscal if it fails ?
Sure, but we just want to make sure..
> However the problem I was hinting is at how to find that someone changed
> the id from under us, so I was wondering if we should have a geteuid()
> right before the open and crap out if it doesn't match what we need ...
Sure - but that is a race condition of course :-).
> maybe we could have that enabled only with configure.developer() to
> avoid adding more syscalls in production to the open path.
Once we've wrapped the setXXX calls in smbd with syscall, then I
don't see how it can get swapped out from under us.
Jeremy.
More information about the samba-technical
mailing list