[PATCH V2] xfs: don't ASSERT on corrupt ftype

[Eric Sandeen <sandeen@xxxxxxxxxxx>]

xfs_dir3_data_get_ftype() gets the file type off
disk, but ASSERTs if it's invalid:

    ASSERT(type < XFS_DIR3_FT_MAX);

We shouldn't ASSERT on bad values read from disk.
V3 dirs are CRC-protected, but V2 dirs + ftype
are not.

Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
---

diff --git a/fs/xfs/libxfs/xfs_da_format.c b/fs/xfs/libxfs/xfs_da_format.c
index c9aee52..7e42fdf 100644
--- a/fs/xfs/libxfs/xfs_da_format.c
+++ b/fs/xfs/libxfs/xfs_da_format.c
@@ -270,7 +270,6 @@ xfs_dir3_data_get_ftype(
 {
     __uint8_t    ftype = dep->name[dep->namelen];
 
-    ASSERT(ftype < XFS_DIR3_FT_MAX);
     if (ftype >= XFS_DIR3_FT_MAX)
         return XFS_DIR3_FT_UNKNOWN;
     return ftype;

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs