Re: Current Samba master incorrectly returns STATUS_INVALID_HANDLE on copy_chunk

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/9/24 4:26 PM, Ralph Boehme wrote:
On 10/9/24 9:43 PM, Jeremy Allison wrote:
Can we just map (access_mask (FILE_APPEND_DATA|SYNCHRONIZE)) == (FILE_APPEND_DATA|SYNCHRONIZE))
to O_APPEND, regardless of POSIX mode ?

thinking about this a bit more, this seems doable, albeit only for POSIX mode. For non-POSIX mode we could potentially break WIndows application that open only with FILE_APPEND_DATA: I checked with a torture test that Windows doesn't enforce append behaviour for FILE_APPEND_DATA|SYNCHRONIZE.

For POSIX opens we should also allow combinations like FILE_READ_ATTRIBUTES|FILE_APPEND_DATA to map to O_APPEND, so clients can write in append mode to the handle and still are able to fstat() it.

https://gitlab.com/samba-team/samba/-/merge_requests/3863

oh, forgot to mention that this fixes the failing copy_chunk copy from the Linux kernel client in posix mode.

-slow




[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux