Re: [PATCH v2 0/7] Allow to choose symlink and socket type

From: Pali Rohár
Date: Sun Oct 13 2024 - 04:59:29 EST


Well, if server explicitly disallow user to create symlink due to
missing permissions, I am not sure if it is a good idea to obey it by
creating symlink in format unsupported by the server...

On Saturday 12 October 2024 23:18:13 Steve French wrote:
> after doing more experiments with native windows symlinks (and how
> difficult it is to get permission to set them over the wire to Windows),
> was wondering if we should allow fall back strategy if creating windows
> style symlinks fails with STATUS_PRIVILEGE_NOT_HELD then we should try NFS
> reparse point symlink. Any opinions?
>
> On Sat, Oct 12, 2024 at 3:53 AM Pali Rohár <pali@xxxxxxxxxx> wrote:
>
> > This patch series improves choosing reparse format when creating new
> > special files.
> >
> > Changes since v1:
> > * Instead of new -o reparse= mount option is now a new -o symlink= mount
> > option for choosing symlink type during creation, and new option
> > -o nonativesocket for choosing socket type
> >
> > Pali Rohár (7):
> > cifs: Add mount option -o symlink= for choosing symlink create type
> > cifs: Add mount option -o reparse=none
> > cifs: Add support for creating native Windows sockets
> > cifs: Add support for creating NFS-style symlinks
> > cifs: Improve guard for excluding $LXDEV xattr
> > cifs: Add support for creating WSL-style symlinks
> > cifs: Validate content of WSL reparse point buffers
> >
> > fs/smb/client/cifsfs.c | 4 +
> > fs/smb/client/cifsglob.h | 36 +++++++
> > fs/smb/client/connect.c | 4 +
> > fs/smb/client/fs_context.c | 82 +++++++++++++++
> > fs/smb/client/fs_context.h | 19 ++++
> > fs/smb/client/link.c | 60 ++++++++---
> > fs/smb/client/reparse.c | 201 +++++++++++++++++++++++++++++++------
> > fs/smb/client/reparse.h | 2 +
> > 8 files changed, 364 insertions(+), 44 deletions(-)
> >
> > --
> > 2.20.1
> >
> >
> >
>
> --
> Thanks,
>
> Steve