| To: | Mark Tinguely <tinguely@xxxxxxx> |
|---|---|
| Subject: | Re: xfs_efi_item slab corruption. (v3.9-10936-g51a26ae) |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Fri, 10 May 2013 11:38:30 +1000 |
| Cc: | Dave Jones <davej@xxxxxxxxxx>, CAI Qian <caiqian@xxxxxxxxxx>, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <518A5213.60307@xxxxxxx> |
| References: | <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> <518A5213.60307@xxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Wed, May 08, 2013 at 08:24:35AM -0500, Mark Tinguely wrote: > On 05/07/13 18:54, Dave Chinner wrote: > > > 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. > > Reported-by: Dave Jones <davej@xxxxxxxxxx> > Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx> > --- > fs/xfs/xfs_extfree_item.c | 14 +++++++++++++- > 1 file changed, 13 insertions(+), 1 deletion(-) > > Looks good. > > Reviewed-by: Mark Tinguely <tinguely@xxxxxxx> Zach pointed out that the fix is much more complex than it needs to be. I'll respin the patch and resend it later today. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx |
| Previous by Date: | Re: [PATCH TARBALL, DEV ONLY] xfsprogs: metadata CRC support V2, Rich Johnston |
|---|---|
| Next by Date: | Re: Rambling noise #1: generic/230 can trigger kernel debug lock detector, Michael L. Semon |
| Previous by Thread: | Re: xfs_efi_item slab corruption. (v3.9-10936-g51a26ae), Mark Tinguely |
| Next by Thread: | Re: xfs_efi_item slab corruption. (v3.9-10936-g51a26ae), Dmitry Monakhov |
| Indexes: | [Date] [Thread] [Top] [All Lists] |