[PATCH]: s3: smbd: Fix delete-on-close after smb2_find

[Jeremy Allison]

On Fri, Nov 03, 2017 at 11:59:02AM -0700, Jeremy Allison via samba-technical wrote:
> On Fri, Nov 03, 2017 at 08:39:35AM -0700, Jeremy Allison via samba-technical wrote:
> > > I was able to recreate the panic on my test system with the attached
> > > 'smb2.delete-on-close-perms.FIND and set DOC.FIND and set DOC'
> > > smbtorture test. Please see also my proposed fix. Both patches apply
> > > on master.
> > 
> > Thanks *VERY* much for the test.
> > 
> > I'm trying to avoid adding calls that use by-pathname semantics,
> > so I'll take a look at the patch to see if we can fix it another
> > way first.
> 
> Nope, we can't. Your patch is good :-). Not only that
> it fixes an underlying issue I missed where calling
> can_delete_directory_fsp() would certainly move the
> directory pointer internally, thus causing a standard
> directory listing to miss files.
> 
> I've logged bug:
> 
> https://bugzilla.samba.org/show_bug.cgi?id=13118
> 
> to track this.

And here is your fix (slight reworking to remove
compile warning and commit message).

Can I get a second Team reviewer ?

Great work Ralph, thanks !

Jeremy.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: master-13118.patch
Type: text/x-diff
Size: 5003 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20171103/f37cf915/master-13118.diff>