diff options
| author | Dave Anderson <anderson@redhat.com> | 2009-04-13 14:39:46 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-04-13 15:04:29 -0700 |
| commit | eb2e5f452a1456c7a20f7566a24d5a8f6ef3edd5 (patch) | |
| tree | 934e58355313148ac212c9cc2e51663c535976d8 /fs/hfs/inode.c | |
| parent | 251eb40f5ccd07a905633a816fbf8f2b6b25cced (diff) | |
| download | linux-stable-eb2e5f452a1456c7a20f7566a24d5a8f6ef3edd5.tar.gz linux-stable-eb2e5f452a1456c7a20f7566a24d5a8f6ef3edd5.tar.bz2 linux-stable-eb2e5f452a1456c7a20f7566a24d5a8f6ef3edd5.zip | |
hfs: fix memory leak when unmounting
When an HFS filesystem is unmounted, it leaks a 2-page bitmap. Also,
under extreme memory pressure, it's possible that hfs_releasepage() may
use a tree pointer that has not been initialized, and if so, the release
request should just be rejected.
[akpm@linux-foundation.org: free_pages(0) is legal, remove obvious comment]
Signed-off-by: Dave Anderson <anderson@redhat.com>
Tested-by: Eugene Teo <eugeneteo@kernel.sg>
Cc: Roman Zippel <zippel@linux-m68k.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/hfs/inode.c')
| -rw-r--r-- | fs/hfs/inode.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/hfs/inode.c b/fs/hfs/inode.c index 9435dda8f1e0..a1cbff2b4d99 100644 --- a/fs/hfs/inode.c +++ b/fs/hfs/inode.c @@ -70,6 +70,10 @@ static int hfs_releasepage(struct page *page, gfp_t mask) BUG(); return 0; } + + if (!tree) + return 0; + if (tree->node_size >= PAGE_CACHE_SIZE) { nidx = page->index >> (tree->node_size_shift - PAGE_CACHE_SHIFT); spin_lock(&tree->hash_lock); |