Re: [patch] staging: lustre: lnet: memory corruption in selftest

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Nov 24, 2016, at 04:10, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> 
> We want sizeof(struct lstcon_node) but instead we're getting the sizeof
> a pointer.
> 
> Fixes: 8d78f0f2ba76 ("staging: lustre: lnet: cleanup some of the > 80 line issues")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Reviewed-by: Andreas Dilger <andreas.dilger@xxxxxxxxx>

> 
> diff --git a/drivers/staging/lustre/lnet/selftest/console.c b/drivers/staging/lustre/lnet/selftest/console.c
> index f2d11fa..f750b9c 100644
> --- a/drivers/staging/lustre/lnet/selftest/console.c
> +++ b/drivers/staging/lustre/lnet/selftest/console.c
> @@ -86,7 +86,7 @@ lstcon_node_find(lnet_process_id_t id, struct lstcon_node **ndpp, int create)
> 	if (!create)
> 		return -ENOENT;
> 
> -	LIBCFS_ALLOC(*ndpp, sizeof(*ndpp) + sizeof(*ndl));
> +	LIBCFS_ALLOC(*ndpp, sizeof(**ndpp) + sizeof(*ndl));
> 	if (!*ndpp)
> 		return -ENOMEM;
> 

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux