dos filemode: a stale lock since 3.4

[Jeremy Allison]

On Tue, Dec 28, 2010 at 07:17:42PM +0300, Dmitry Butskoy wrote:
> Dmitry Butskoy wrote:
> >I've discovered an issue since the Samba 3.4, when "dos filemode =
> >yes" and a "not-owner with write access" tries to change file
> >attributes. A stale lock appear (it may vary depending on a
> >particular user case, the simplified test see below).
> >
> >Comparing the function  "smbd/open.c:open_file_fchmod()"  between
> >the versions 3.2.15 and 3.4.8, the call of "open_file" was changed
> >to "SMB_VFS_CREATE_FILE()", but the correspond
> >"smbd/open.c:close_file_fchmod()"  seems not changed in this way.
> >I guess this change causes the issue -- a stale WRONLY lock have
> >appeared.
> 
> Well, the correspond git change is:
> http://git.samba.org/?p=samba.git;a=commitdiff;h=6f7f4b428a86921e9532c62856873a19202f286d
> 
> When I revert it (for samba 3.4.9) the problem disappears (no any
> locks pending).
> 
> Any comments? Is this a correct fix?

I've logged bug #7892:

https://bugzilla.samba.org/show_bug.cgi?id=7892

to track this. Feel free to add yourself to this
bug report, it's where I'll attach the patches etc.

Jeremy.