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

pali · gregkh · commit 10cbae84fed8 · 2025-03-22T12:50:49.000-07:00
[ Upstream commit cad3fc0 ] This would help to track and detect by caller if the reparse point type was processed or not. Signed-off-by: Pali Rohár <pali@kernel.org> Signed-off-by: Steve French <stfrench@microsoft.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
diff --git a/fs/smb/client/reparse.c b/fs/smb/client/reparse.c
@@ -656,13 +656,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
`@@ -656,13 +656,12 @@ int parse_reparse_point(struct reparse_data_buffer *buf,`
`656`	`656`	`le32_to_cpu(buf->ReparseTag));`
`657`	`657`	`return -EIO;`
`658`	`658`	`}`
`659`		`- break;`
	`659`	`+ return 0;`
`660`	`660`	`default:`
`661`	`661`	`cifs_tcon_dbg(VFS \| ONCE, "unhandled reparse tag: 0x%08x\n",`
`662`	`662`	`le32_to_cpu(buf->ReparseTag));`
`663`		`- break;`
	`663`	`+ return -EOPNOTSUPP;`
`664`	`664`	`}`
`665`		`- return 0;`
`666`	`665`	`}`
`667`	`666`
`668`	`667`	`int smb2_parse_reparse_point(struct cifs_sb_info *cifs_sb,`