Re: [PATCH 8/9] xfs: catch bad block numbers freeing extents.

xfs

Re: [PATCH 8/9] xfs: catch bad block numbers freeing extents.

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 8/9] xfs: catch bad block numbers freeing extents.
From:	Alex Elder <aelder@xxxxxxx>
Date:	Thu, 07 Apr 2011 16:16:46 -0500
Cc:	xfs@xxxxxxxxxxx
In-reply-to:	<1302141445-27457-9-git-send-email-david@xxxxxxxxxxxxx>
References:	<1302141445-27457-1-git-send-email-david@xxxxxxxxxxxxx> <1302141445-27457-9-git-send-email-david@xxxxxxxxxxxxx>
Reply-to:	aelder@xxxxxxx

On Thu, 2011-04-07 at 11:57 +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
> 
> A fuzzed filesystem crashed a kernel when freeing an extent with a
> block number beyond the end of the filesystem. Convert all the debug
> asserts in xfs_free_extent() to active checks so that we catch bad
> extents and return that the filesytsem is corrupted rather than
> crashing.
> 
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>

Looks good.

Reviewed-by: Alex Elder <aelder@xxxxxxx>

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
Re: [PATCH 3/9] xfs: convert ENOSPC inode flushing to use new syncd workqueue, (continued) Re: [PATCH 3/9] xfs: convert ENOSPC inode flushing to use new syncd workqueue, Alex Elder [PATCH 7/9] xfs: push the AIL from memory reclaim and periodic sync, Dave Chinner Re: [PATCH 7/9] xfs: push the AIL from memory reclaim and periodic sync, Alex Elder [PATCH 6/9] xfs: clean up code layout in xfs_trans_ail.c, Dave Chinner Re: [PATCH 6/9] xfs: clean up code layout in xfs_trans_ail.c, Alex Elder [PATCH 5/9] xfs: convert the xfsaild threads to a workqueue, Dave Chinner Re: [PATCH 5/9] xfs: convert the xfsaild threads to a workqueue, Alex Elder [PATCH 9/9] xfs: convert log tail checking to a warning, Dave Chinner Re: [PATCH 9/9] xfs: convert log tail checking to a warning, Alex Elder [PATCH 8/9] xfs: catch bad block numbers freeing extents., Dave Chinner Re: [PATCH 8/9] xfs: catch bad block numbers freeing extents., Alex Elder <=

Previous by Date:	Re: [PATCH 6/9] xfs: clean up code layout in xfs_trans_ail.c, Alex Elder
Next by Date:	Re: [PATCH 7/9] xfs: push the AIL from memory reclaim and periodic sync, Alex Elder
Previous by Thread:	[PATCH 8/9] xfs: catch bad block numbers freeing extents., Dave Chinner
Next by Thread:	[PATCH 0/2] xfs: write back inodes during reclaim, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]