libadouble: allow FILE_SHARE_DELETE in ad_convert_xattr()
authorRalph Boehme <slow@samba.org>
Tue, 23 May 2023 15:28:33 +0000 (17:28 +0200)
committerJeremy Allison <jra@samba.org>
Thu, 25 May 2023 23:59:33 +0000 (23:59 +0000)
commit55bd10456486628cad2bd085618e873598401c3b
treecca3aae89b7d2b8c70be98e7fabf453a08c8cca0
parent06f0c070a0b18313f48362aa326e3e7e6a096492
libadouble: allow FILE_SHARE_DELETE in ad_convert_xattr()

Not specifying FILE_SHARE_DELETE wasn't done intentionally. Not setting the flag
triggers the following problem:

* client sends a CREATE with delete access

* this triggers a call to open_streams_for_delete() where we check for
conflicting opens on any of the streams of the file or directory

* if the file (or directory) has a stream like ":com.apple.quarantine" the
stream is opened with DELETE_ACCESS and kept open when the next step might:

* if the file (or directory) has a Mac specific :AFP_AfpInfo stream, the
ad_convert() routine in fruit_create_file() is triggered

* ad_convert() checks if the file (or ...) has a sidecar ._ AppleDouble file, if
it has:

* in ad_convert_xattr() we unpack any set of xattrs encoded in the AppleDouble
file and recreate them as streams with the VFS. Now, if any of these xattrs
happens to be converted to a stream that we still have open in
open_streams_for_delete() (see above) we get a NT_STATUS_SHARING_VIOLATION

This error gets passed up the stack back to open_streams_for_delete() so the
client CREATE request fails and the client is unhappy.

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

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
selftest/knownfail.d/samba3.vfs.fruit.delete_trigger_convert_sharing_violation [deleted file]
source3/lib/adouble.c