On Sat, Sep 08, 2018 at 01:59:50AM +0800, Liu Bo wrote:
> kcalloc is defined as
> kcalloc(size_t n, size_t size, gfp_t flags)
>
> Although this won't cause problems in practice, btrfsic_read_block()
> has switched n with size.
>
> This updates btrfsic_read_block() to be correct in using kcalloc.
>
> Signed-off-by: Liu Bo <bo.liu@xxxxxxxxxxxxxxxxx>
> ---
> fs/btrfs/check-integrity.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/fs/btrfs/check-integrity.c b/fs/btrfs/check-integrity.c
> index 833cf3c35b4d..651b75939ce9 100644
> --- a/fs/btrfs/check-integrity.c
> +++ b/fs/btrfs/check-integrity.c
> @@ -1594,6 +1594,7 @@ static int btrfsic_read_block(struct btrfsic_state *state,
> {
> unsigned int num_pages;
> unsigned int i;
> + u64 size;
size_t?
Besides that, Reviewed-by: Omar Sandoval <osandov@xxxxxx>
> u64 dev_bytenr;
> int ret;
>
> @@ -1608,9 +1609,8 @@ static int btrfsic_read_block(struct btrfsic_state *state,
>
> num_pages = (block_ctx->len + (u64)PAGE_SIZE - 1) >>
> PAGE_SHIFT;
> - block_ctx->mem_to_free = kcalloc(sizeof(*block_ctx->datav) +
> - sizeof(*block_ctx->pagev),
> - num_pages, GFP_NOFS);
> + size = sizeof(*block_ctx->datav) + sizeof(*block_ctx->pagev);
> + block_ctx->mem_to_free = kcalloc(num_pages, size, GFP_NOFS);
> if (!block_ctx->mem_to_free)
> return -ENOMEM;
> block_ctx->datav = block_ctx->mem_to_free;
> --
> 1.8.3.1
>
