]> git.ipfire.org Git - thirdparty/xfsprogs-dev.git/commit
xfs_repair: allow '/' in attribute names
authorEric Sandeen <sandeen@redhat.com>
Mon, 28 Jan 2019 19:03:03 +0000 (13:03 -0600)
committerEric Sandeen <sandeen@redhat.com>
Mon, 28 Jan 2019 19:03:03 +0000 (13:03 -0600)
commit45571fd5885d00520a5c1d3bb743634267920c19
tree5e1ca03bf6075ae1256f7da7992f0829921f67ff
parentb32d0eb62d2452ded08506cbeee0131caca45697
xfs_repair: allow '/' in attribute names

For some reason, since the earliest days of XFS, a '/' character
in an extended attribute name has been treated as corruption by
xfs_repair.  This despite nothing in other userspace tools or the
kernel having this restriction.

My best guess is that this was an unintentional leftover from
common code between dirs & attrs in the "da" code, and there has
never been a good reason for it.

Since userspace and kernelspace allow such a name to be set,
listed, and read, it seems wrong to flag it as corruption.
So, make this test conditional on whether we're validating a name
in a dir, as opposed to the name of an attr.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
repair/attr_repair.c
repair/da_util.c
repair/da_util.h
repair/dir2.c