diff options
| author |   Darrick J. Wong <djwong@kernel.org> | 2023-04-11 19:00:29 -0700 |
|---|---|---|
| committer | Darrick J. Wong <djwong@kernel.org> | 2023-04-11 19:00:29 -0700 |
| commit | c12ad41468a5f4112b98566dcb1ee9096579106a (patch) | |
| tree | f3ad1477100d6fb821454c86bff095346af9f455 | |
| parent | xfs: check for reverse mapping records that could be merged (diff) | |
| download | wireguard-linux-c12ad41468a5f4112b98566dcb1ee9096579106a.tar.xz wireguard-linux-c12ad41468a5f4112b98566dcb1ee9096579106a.zip | |
xfs: xattr scrub should ensure one namespace bit per name
Check that each extended attribute exists in only one namespace.
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
| -rw-r--r-- | fs/xfs/scrub/attr.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/fs/xfs/scrub/attr.c b/fs/xfs/scrub/attr.c index 5573be3a3dfe..f6def9c864ab 100644 --- a/fs/xfs/scrub/attr.c +++ b/fs/xfs/scrub/attr.c @@ -128,10 +128,16 @@ xchk_xattr_listent( return; } + /* Only one namespace bit allowed. */ + if (hweight32(flags & XFS_ATTR_NSP_ONDISK_MASK) > 1) { + xchk_fblock_set_corrupt(sx->sc, XFS_ATTR_FORK, args.blkno); + goto fail_xref; + } + /* Does this name make sense? */ if (!xfs_attr_namecheck(name, namelen)) { xchk_fblock_set_corrupt(sx->sc, XFS_ATTR_FORK, args.blkno); - return; + goto fail_xref; } /* |