diff mbox series

[SMB3,client] fix creating sockets when using sfu mount options

Message ID CAH2r5mvv6GS07jY2W_5zXV3YpcHup7rZghRqmpoeSDndgrkDEA@mail.gmail.com
State New
Headers show
Series [SMB3,client] fix creating sockets when using sfu mount options | expand

Commit Message

Steve French May 29, 2024, 11:30 p.m. UTC
cifs: fix creating sockets when using sfu mount options

    When running fstest generic/423 with sfu mount option, it
    was being skipped due to inability to create sockets:

      generic/423  [not run] cifs does not support mknod/mkfifo

    which can also be easily reproduced with their af_unix tool:

      ./src/af_unix /mnt1/socket-two bind: Operation not permitted

    Fix sfu mount option to allow creating and reporting sockets.

See attached

Comments

Steve French May 31, 2024, 4:03 p.m. UTC | #1
Updated to remove setting major/minor number for sockets since they
are not relevant (and are not reported by stat command e.g.).

Suggested by Paulo.

On Wed, May 29, 2024 at 6:30 PM Steve French <smfrench@gmail.com> wrote:
>
>     cifs: fix creating sockets when using sfu mount options
>
>     When running fstest generic/423 with sfu mount option, it
>     was being skipped due to inability to create sockets:
>
>       generic/423  [not run] cifs does not support mknod/mkfifo
>
>     which can also be easily reproduced with their af_unix tool:
>
>       ./src/af_unix /mnt1/socket-two bind: Operation not permitted
>
>     Fix sfu mount option to allow creating and reporting sockets.
>
> See attached
>
>
> --
> Thanks,
>
> Steve
diff mbox series

Patch

From 7e476a83fc35f825759d079f4a0ce349bd927ca9 Mon Sep 17 00:00:00 2001
From: Steve French <stfrench@microsoft.com>
Date: Wed, 29 May 2024 18:16:56 -0500
Subject: [PATCH] cifs: fix creating sockets when using sfu mount options

When running fstest generic/423 with sfu mount option, it
was being skipped due to inability to create sockets:

  generic/423  [not run] cifs does not support mknod/mkfifo

which can also be easily reproduced with their af_unix tool:

  ./src/af_unix /mnt1/socket-two bind: Operation not permitted

Fix sfu mount option to allow creating and reporting sockets.

Cc: stable@vger.kernel.org
Signed-off-by: Steve French <stfrench@microsoft.com>
---
 fs/smb/client/cifspdu.h |  2 +-
 fs/smb/client/inode.c   | 13 +++++++++++++
 fs/smb/client/smb2ops.c |  5 +++++
 3 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/fs/smb/client/cifspdu.h b/fs/smb/client/cifspdu.h
index c46d418c1c0c..a2072ab9e586 100644
--- a/fs/smb/client/cifspdu.h
+++ b/fs/smb/client/cifspdu.h
@@ -2574,7 +2574,7 @@  typedef struct {
 
 
 struct win_dev {
-	unsigned char type[8]; /* IntxCHR or IntxBLK or LnxFIFO*/
+	unsigned char type[8]; /* IntxCHR or IntxBLK or LnxFIFO or LnxSOCK */
 	__le64 major;
 	__le64 minor;
 } __attribute__((packed));
diff --git a/fs/smb/client/inode.c b/fs/smb/client/inode.c
index 262576573eb5..6e4fac5f64bc 100644
--- a/fs/smb/client/inode.c
+++ b/fs/smb/client/inode.c
@@ -602,6 +602,19 @@  cifs_sfu_type(struct cifs_fattr *fattr, const char *path,
 				/* we have enough to decode dev num */
 				__u64 mjr; /* major */
 				__u64 mnr; /* minor */
+				mjr = le64_to_cpu(*(__le64 *)(pbuf+8));
+				mnr = le64_to_cpu(*(__le64 *)(pbuf+16));
+				fattr->cf_rdev = MKDEV(mjr, mnr);
+			}
+		} else if (memcmp("LnxSOCK", pbuf, 8) == 0) {
+			cifs_dbg(FYI, "Socket\n");
+			fattr->cf_mode |= S_IFSOCK;
+			fattr->cf_dtype = DT_SOCK;
+			if (bytes_read == 24) {
+				/* we have enough to decode dev num */
+				__u64 mjr; /* major */
+				__u64 mnr; /* minor */
+
 				mjr = le64_to_cpu(*(__le64 *)(pbuf+8));
 				mnr = le64_to_cpu(*(__le64 *)(pbuf+16));
 				fattr->cf_rdev = MKDEV(mjr, mnr);
diff --git a/fs/smb/client/smb2ops.c b/fs/smb/client/smb2ops.c
index 4ce6c3121a7e..477d594c2fc3 100644
--- a/fs/smb/client/smb2ops.c
+++ b/fs/smb/client/smb2ops.c
@@ -4997,6 +4997,11 @@  static int __cifs_sfu_make_node(unsigned int xid, struct inode *inode,
 		pdev.major = cpu_to_le64(MAJOR(dev));
 		pdev.minor = cpu_to_le64(MINOR(dev));
 		break;
+	case S_IFSOCK:
+		strscpy(pdev.type, "LnxSOCK");
+		pdev.major = cpu_to_le64(MAJOR(dev));
+		pdev.minor = cpu_to_le64(MINOR(dev));
+		break;
 	case S_IFIFO:
 		strscpy(pdev.type, "LnxFIFO");
 		break;
-- 
2.43.0