cifs: Throw -EOPNOTSUPP error on unsupported reparse point type from parse_reparse_point()

pali · smfrench · commit cad3fc0a4c8c · 2025-02-19T10:42:25.000-06:00
This would help to track and detect by caller if the reparse point type was
processed or not.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Signed-off-by: Steve French &lt;stfrench@microsoft.com&gt;
diff --git a/fs/smb/client/reparse.c b/fs/smb/client/reparse.c
@@ -1088,13 +1088,12 @@ int parse_reparse_point(struct reparse_data_buffer *buf,
 				 le32_to_cpu(buf->ReparseTag));
 			return -EIO;
 		}
-		break;
+		return 0;
 	default:
 		cifs_tcon_dbg(VFS | ONCE, "unhandled reparse tag: 0x%08x\n",
 			      le32_to_cpu(buf->ReparseTag));
-		break;
+		return -EOPNOTSUPP;
 	}
-	return 0;
 }
 
 int smb2_parse_reparse_point(struct cifs_sb_info *cifs_sb,

Original file line number	Diff line number	Diff line change
`@@ -1088,13 +1088,12 @@ int parse_reparse_point(struct reparse_data_buffer *buf,`
`1088`	`1088`	`le32_to_cpu(buf->ReparseTag));`
`1089`	`1089`	`return -EIO;`
`1090`	`1090`	`}`
`1091`		`- break;`
	`1091`	`+ return 0;`
`1092`	`1092`	`default:`
`1093`	`1093`	`cifs_tcon_dbg(VFS \| ONCE, "unhandled reparse tag: 0x%08x\n",`
`1094`	`1094`	`le32_to_cpu(buf->ReparseTag));`
`1095`		`- break;`
	`1095`	`+ return -EOPNOTSUPP;`
`1096`	`1096`	`}`
`1097`		`- return 0;`
`1098`	`1097`	`}`
`1099`	`1098`
`1100`	`1099`	`int smb2_parse_reparse_point(struct cifs_sb_info *cifs_sb,`