diff options
author | Dave Chinner <dchinner@redhat.com> | 2014-03-07 00:19:14 -0500 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2014-03-07 00:19:14 -0500 |
commit | ac75a1f7a4af4dddcc1ac3c0778f0e3f75dc8f32 (patch) | |
tree | ac44f40e109615cfda93e22fb79e8953f842393d /fs/xfs/xfs_mount.c | |
parent | 38dbfb59d1175ef458d006556061adeaa8751b72 (diff) |
xfs: don't leak EFSBADCRC to userspace
While the verifier routines may return EFSBADCRC when a buffer has
a bad CRC, we need to translate that to EFSCORRUPTED so that the
higher layers treat the error appropriately and we return a
consistent error to userspace. This fixes a xfs/005 regression.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs/xfs_mount.c')
-rw-r--r-- | fs/xfs/xfs_mount.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c index 02df7b408a26..5c670f55ef07 100644 --- a/fs/xfs/xfs_mount.c +++ b/fs/xfs/xfs_mount.c | |||
@@ -307,6 +307,9 @@ reread: | |||
307 | error = bp->b_error; | 307 | error = bp->b_error; |
308 | if (loud) | 308 | if (loud) |
309 | xfs_warn(mp, "SB validate failed with error %d.", error); | 309 | xfs_warn(mp, "SB validate failed with error %d.", error); |
310 | /* bad CRC means corrupted metadata */ | ||
311 | if (error == EFSBADCRC) | ||
312 | error = EFSCORRUPTED; | ||
310 | goto release_buf; | 313 | goto release_buf; |
311 | } | 314 | } |
312 | 315 | ||