| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: xfs_efi_item slab corruption. (v3.9-10936-g51a26ae) |
| From: | Dave Jones <davej@xxxxxxxxxx> |
| Date: | Tue, 7 May 2013 20:43:50 -0400 |
| Cc: | Mark Tinguely <tinguely@xxxxxxx>, CAI Qian <caiqian@xxxxxxxxxx>, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20130507235458.GG24635@dastard> |
| References: | <518954DE.4070803@xxxxxxx> <20130507193146.GA7539@xxxxxxxxxx> <51895CD7.7040806@xxxxxxx> <20130507195954.GA8384@xxxxxxxxxx> <51895E51.2050508@xxxxxxx> <20130507202217.GA9883@xxxxxxxxxx> <518962FC.2060509@xxxxxxx> <20130507222256.GD24635@dastard> <51898400.8000900@xxxxxxx> <20130507235458.GG24635@dastard> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Wed, May 08, 2013 at 09:54:58AM +1000, Dave Chinner wrote:
> Dave, the patch below should fix the problem.
>
> xfs: Don't reference the EFI after it is freed
>
> From: Dave Chinner <dchinner@xxxxxxxxxx>
>
> Checking the EFI for whether it is being released from recovery
> after we've already released the known active reference is a mistake
> worthy of a brown paper bag. Fix the (now) obvious use after free
> that it can cause.
Yup, that works for me.
thanks,
Dave
|
| Previous by Date: | Re: [PATCH 2/2] Make test 306 use null and zero device nodes on SCRATCH_MNT, Eric Sandeen |
|---|---|
| Next by Date: | [PATCH V2] xfstests: automatically add -F to xfs_io on non-xfs, Eric Sandeen |
| Previous by Thread: | Re: xfs_efi_item slab corruption. (v3.9-10936-g51a26ae), Dave Chinner |
| Next by Thread: | Re: xfs_efi_item slab corruption. (v3.9-10936-g51a26ae), Mark Tinguely |
| Indexes: | [Date] [Thread] [Top] [All Lists] |