diff options
| author | Eric Sandeen <sandeen@sandeen.net> | 2017-10-10 06:08:06 +0200 |
|---|---|---|
| committer | Darrick J. Wong <darrick.wong@oracle.com> | 2017-10-11 19:21:07 +0200 |
| commit | 93e8befc17f6d6ea92b0aee3741ceac8bca4590f (patch) | |
| tree | 19412a90dafb857dd468dcddfbe4bd6f1ca14180 /include/keys | |
| parent | xfs: reinit btree pointer on attr tree inactivation walk (diff) | |
| download | linux-93e8befc17f6d6ea92b0aee3741ceac8bca4590f.tar.xz linux-93e8befc17f6d6ea92b0aee3741ceac8bca4590f.zip | |
xfs: handle error if xfs_btree_get_bufs fails
Jason reported that a corrupted filesystem failed to replay
the log with a metadata block out of bounds warning:
XFS (dm-2): _xfs_buf_find: Block out of range: block 0x80270fff8, EOFS 0x9c40000
_xfs_buf_find() and xfs_btree_get_bufs() return NULL if
that happens, and then when xfs_alloc_fix_freelist() calls
xfs_trans_binval() on that NULL bp, we oops with:
BUG: unable to handle kernel NULL pointer dereference at 00000000000000f8
We don't handle _xfs_buf_find errors very well, every
caller higher up the stack gets to guess at why it failed.
But we should at least handle it somehow, so return
EFSCORRUPTED here.
Reported-by: Jason L Tibbitts III <tibbs@math.uh.edu>
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to 'include/keys')
0 files changed, 0 insertions, 0 deletions