[PATCH] Final removal of lp_posix_pathnames() from the smbd server main code paths.
Jeremy Allison
jra at samba.org
Tue Mar 22 15:34:57 UTC 2016
On Tue, Mar 22, 2016 at 12:17:22PM +0100, Ralph Boehme wrote:
> Hi!
>
> On Sun, Mar 20, 2016 at 09:31:40PM -0700, Jeremy Allison wrote:
> > 4). Add uint32_t flags field to struct smb_filename. Only
> > defined field currently is SMB_FILENAME_POSIX_PATH.
>
> we already have
>
> #define FSP_POSIX_FLAGS_OPEN 0x01
> #define FSP_POSIX_FLAGS_RENAME 0x02
> #define FSP_POSIX_FLAGS_PATHNAMES 0x04
>
> in vfs.h. Can we somehow align/reuse the flags?
>
> I'm afraid that having essentially the same flags for struct
> smb_filename and files_struct is confusing in the long term.
Err. Sure - the names don't match though, due
to the FSP_XXX component.
But I can certainly define the SMB_FILENAME_XX_FLAG
in terms of the FSP_POSIX_FLAGS_XX.
The only one that applies is FSP_POSIX_FLAGS_PATHNAMES,
as the others cover attributes of open files (locking,
renames etc.) that simply don't apply to pure pathnames.
Jeremy.
More information about the samba-technical
mailing list