From: NeilBrown Signed-off-by: Neil Brown Signed-off-by: Andrew Morton --- 25-akpm/drivers/md/bitmap.c | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff -puN drivers/md/bitmap.c~md-minor-code-rearrangement-in-bitmap_init_from_disk drivers/md/bitmap.c --- 25/drivers/md/bitmap.c~md-minor-code-rearrangement-in-bitmap_init_from_disk 2005-03-23 02:09:46.000000000 -0800 +++ 25-akpm/drivers/md/bitmap.c 2005-03-23 02:09:46.000000000 -0800 @@ -781,7 +781,9 @@ static int bitmap_init_from_disk(struct "recovery\n", bmname(bitmap)); bytes = (chunks + 7) / 8; - num_pages = (bytes + PAGE_SIZE - 1) / PAGE_SIZE; + + num_pages = (bytes + sizeof(bitmap_super_t) + PAGE_SIZE - 1) / PAGE_SIZE + 1; + if (i_size_read(file->f_mapping->host) < bytes + sizeof(bitmap_super_t)) { printk(KERN_INFO "%s: bitmap file too short %lu < %lu\n", bmname(bitmap), @@ -789,18 +791,16 @@ static int bitmap_init_from_disk(struct bytes + sizeof(bitmap_super_t)); goto out; } - num_pages++; + + ret = -ENOMEM; + bitmap->filemap = kmalloc(sizeof(struct page *) * num_pages, GFP_KERNEL); - if (!bitmap->filemap) { - ret = -ENOMEM; + if (!bitmap->filemap) goto out; - } bitmap->filemap_attr = kmalloc(sizeof(long) * num_pages, GFP_KERNEL); - if (!bitmap->filemap_attr) { - ret = -ENOMEM; + if (!bitmap->filemap_attr) goto out; - } memset(bitmap->filemap_attr, 0, sizeof(long) * num_pages); _