From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> Date: Tue, 27 Sep 2016 13:10:05 +0200 Move the assignment for the local variable "offset" behind the source code for memory allocations by this function. Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> --- drivers/md/bitmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/md/bitmap.c b/drivers/md/bitmap.c index 8cfb02c..78512c6 100644 --- a/drivers/md/bitmap.c +++ b/drivers/md/bitmap.c @@ -759,7 +759,6 @@ static int bitmap_storage_alloc(struct bitmap_storage *store, bytes += sizeof(bitmap_super_t); num_pages = DIV_ROUND_UP(bytes, PAGE_SIZE); - offset = slot_number * num_pages; store->filemap = kmalloc_array(num_pages, sizeof(*store->filemap), GFP_KERNEL); @@ -772,6 +771,7 @@ static int bitmap_storage_alloc(struct bitmap_storage *store, return -ENOMEM; } + offset = slot_number * num_pages; pnum = 0; if (store->sb_page) { store->filemap[0] = store->sb_page; -- 2.10.0 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html