From: Aishwarya Pant Date: Wed, 1 Mar 2017 18:24:01 +0000 (+0530) Subject: staging: bcm2835-camera: prefer allocating memory using sizeof(*ptr) instead of sizeo... X-Git-Tag: v5.15~11366^2~765 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=36b4894e4a03293f20a7e34e20336f2f05b546d1;p=platform%2Fkernel%2Flinux-starfive.git staging: bcm2835-camera: prefer allocating memory using sizeof(*ptr) instead of sizeof(struct type name) Using sizeof(*ptr) instead of sizeof(struct type name) is more robust to errors. Signed-off-by: Aishwarya Pant Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c b/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c index 84650d9..c9a68d4 100644 --- a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c +++ b/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c @@ -1908,7 +1908,7 @@ static int __init bm2835_mmal_init(void) num_cameras = MAX_BCM2835_CAMERAS; for (camera = 0; camera < num_cameras; camera++) { - dev = kzalloc(sizeof(struct bm2835_mmal_dev), GFP_KERNEL); + dev = kzalloc(sizeof(*dev), GFP_KERNEL); if (!dev) { ret = -ENOMEM; goto cleanup_gdev;