Re: [Stable-review] [patch 000/108] 2.6.30-stable review

[Julien BLACHE]

"Luis R. Rodriguez" <mcgrof@xxxxxxxxx> wrote:

Hi,

>> Actually I was under the impression that the proper request had been
>> made for inclusion into -stable, but it looks like it's not been done
>> after all.
>>
>> So, here goes.
>
> Better inline.

Crap, I got tricked by the inline thing again. Sorry, properly inlined
this time.

JB.

-- 
Julien BLACHE                                   <http://www.jblache.org> 
<jb@xxxxxxxxxxx>                                  GPG KeyID 0xF5D65169



From: Felix Blyakher <felixb@xxxxxxx>
Date: Thu, 11 Jun 2009 22:07:28 +0000 (-0500)
Subject: xfs: fix freeing memory in xfs_getbmap()
X-Git-Url: http://oss.sgi.com/cgi-bin/gitweb.cgi?p=xfs%2Fxfs.git;a=commitdiff_plain;h=7747a0b0af5976ba3828796b4f7a7adc3bb76dbd

xfs: fix freeing memory in xfs_getbmap()

Regression from commit 28e211700a81b0a934b6c7a4b8e7dda843634d2f.
Need to free temporary buffer allocated in xfs_getbmap().

Signed-off-by: Felix Blyakher <felixb@xxxxxxx>
Signed-off-by: Hedi Berriche <hedi@xxxxxxx>
Reported-by: Justin Piszcz <jpiszcz@xxxxxxxxxxxxxxx>
Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
---

diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c
index 4b0f6ef..7928b99 100644
--- a/fs/xfs/xfs_bmap.c
+++ b/fs/xfs/xfs_bmap.c
@@ -6086,6 +6086,7 @@ xfs_getbmap(
 			break;
 	}
 
+	kmem_free(out);
 	return error;
 }
 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/