[PATCH] Fix bug #10587 - Opening directories on SLES's cifsd and Apple's smbx succeeds.

[Jeremy Allison]

On Thu, May 08, 2014 at 01:57:43PM -0700, Jeremy Allison wrote:
> On Thu, May 08, 2014 at 01:32:16PM -0700, Jeremy Allison wrote:
> > On Thu, May 08, 2014 at 09:58:19PM +0200, Stefan (metze) Metzmacher wrote:
> > > This is too much magic if the cli_foo() behaves differently as
> > > cli_foo_send/recv().
> > 
> > Yeah, I had qualms about it too, but it's the simplest
> > fix I could come up with - not being able to change
> > the gvfs or other libsmbclient callers.
> > 
> > Or do you mean putting this change inside libsmbclient ?
> > 
> > There is also precedence for this kind of change
> > based on the fallback code inside cli_open() that
> > tries ntcreateX first, and then fall back to
> > openX if the server returns specific error codes:
> 
> OK, how about the following. It puts all the
> uglyness inside cli_open(), which is the
> meta-call used by libsmbclient and smbclient.
> 
> As there is no cli_open_send() and cli_open_recv()
> I think we're safe having different behavior
> here.

And here's the version with the corrected
commit message that properly describes
what I'm doing here :-).

As a side-note, this change will also fixes
the bug if it happens in an SMB2 server as well,
which kind of hints that this is the correct layer
to fix it at (as that fix came for free :-).

Cheers,

	Jeremy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-s3-libsmbclient-Work-around-bugs-in-SLES-cifsd-and-A.patch
Type: text/x-diff
Size: 6120 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140508/1c740602/attachment.patch>