From: Jing Zhang <zj.barak@xxxxxxxxx> Date: Wed Mar 24 20:38:48 2010 If ext4_mb_init_cache() returns error, there is no page_cache_release() issued. Cc: Theodore Ts'o <tytso@xxxxxxx> Cc: Andreas Dilger <adilger@xxxxxxx> Cc: Dave Kleikamp <shaggy@xxxxxxxxxxxxxxxxxx> Signed-off-by: Jing Zhang <zj.barak@xxxxxxxxx> --- --- linux-2.6.32/fs/ext4/mballoc.c 2009-12-03 11:51:22.000000000 +0800 +++ ext4_mm_leak/mballoc-9.c 2010-03-24 22:24:56.000000000 +0800 @@ -1132,8 +1132,6 @@ repeat_load_buddy: e4b->bd_buddy = page_address(page) + (poff * sb->s_blocksize); mark_page_accessed(page); - BUG_ON(e4b->bd_bitmap_page == NULL); - BUG_ON(e4b->bd_buddy_page == NULL); return 0; @@ -1142,6 +1140,8 @@ err: page_cache_release(e4b->bd_bitmap_page); if (e4b->bd_buddy_page) page_cache_release(e4b->bd_buddy_page); + if (page) + page_cache_release(page); e4b->bd_buddy = NULL; e4b->bd_bitmap = NULL; -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html