[PATCH] Fix bug #10587 - Opening directories on SLES's cifsd and Apple's smbx succeeds.
Stefan (metze) Metzmacher
metze at samba.org
Fri May 9 15:12:34 MDT 2014
Am 09.05.2014 06:39, schrieb Jeremy Allison:
> On Thu, May 08, 2014 at 07:54:05PM -0700, Jeremy Allison wrote:
>> On Thu, May 08, 2014 at 07:47:34PM -0700, Jeremy Allison wrote:
>>> On Fri, May 09, 2014 at 12:19:11AM +0200, Stefan (metze) Metzmacher wrote:
>>>> Please avoid the ugly _internal_recv() function,
>>>> just expose the attrs in the normal _recv and the sync function.
>>> That means changing the ABI to the cli_ntcreate_recv().
>>> I thought this was already exposed to external callers,
>>> but checking I guess we're not guarenteeing this
>>> interface externally.
>>> OK, I'll fix it up to be the same as the smb2
>>> version of the function that returns a struct
>>> containing the full data returned from the
>>> open in the _recv call.
>> FYI - If I do it this way it's a *much*
>> bigger change, as it means changing
>> every caller of cli_ntcreate() and
>> also cli_nttrans_create() as well
>> (as we'll need the same return values
> All right Metze, here's the goddam rewrite
> you insisted upon :-).
> Add a return parameter of struct smb_create_returns *cr to
> and fixes everything up... It's now a 4-patch set.
> Please review. *Now* will you be happy ! :-).
Thanks very much, this is much much nicer :-)
I think it will be very useful to have smb_create_returns in some more
places in future.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 263 bytes
Desc: OpenPGP digital signature
More information about the samba-technical