aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/media/video
diff options
context:
space:
mode:
authorThomas Meyer <thomas@m3y3r.de>2011-11-29 15:08:00 -0500
committerMauro Carvalho Chehab <mchehab@redhat.com>2011-12-12 11:51:11 -0500
commite79c70e6e59e764fbef2a68a7d7303c3286ea7a8 (patch)
tree25a5039da05d2375b2dea11040e35681b5ef1ae8 /drivers/media/video
parent56a8b9f181b5e9b201ab02ad0b63ccdc17d4b191 (diff)
[media] pvrusb2: Use kcalloc instead of kzalloc to allocate array
The advantage of kcalloc is, that will prevent integer overflows which could result from the multiplication of number of elements and size and it is also a bit nicer to read. The semantic patch that makes this change is available in https://lkml.org/lkml/2011/11/25/107 Signed-off-by: Thomas Meyer <thomas@m3y3r.de> Acked-By: Mike Isely <isely@pobox.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video')
-rw-r--r--drivers/media/video/pvrusb2/pvrusb2-hdw.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/media/video/pvrusb2/pvrusb2-hdw.c b/drivers/media/video/pvrusb2/pvrusb2-hdw.c
index 122b45760f0..ebc2c7e3923 100644
--- a/drivers/media/video/pvrusb2/pvrusb2-hdw.c
+++ b/drivers/media/video/pvrusb2/pvrusb2-hdw.c
@@ -2546,8 +2546,9 @@ struct pvr2_hdw *pvr2_hdw_create(struct usb_interface *intf,
2546 } 2546 }
2547 2547
2548 /* Define and configure additional controls from cx2341x module. */ 2548 /* Define and configure additional controls from cx2341x module. */
2549 hdw->mpeg_ctrl_info = kzalloc( 2549 hdw->mpeg_ctrl_info = kcalloc(MPEGDEF_COUNT,
2550 sizeof(*(hdw->mpeg_ctrl_info)) * MPEGDEF_COUNT, GFP_KERNEL); 2550 sizeof(*(hdw->mpeg_ctrl_info)),
2551 GFP_KERNEL);
2551 if (!hdw->mpeg_ctrl_info) goto fail; 2552 if (!hdw->mpeg_ctrl_info) goto fail;
2552 for (idx = 0; idx < MPEGDEF_COUNT; idx++) { 2553 for (idx = 0; idx < MPEGDEF_COUNT; idx++) {
2553 cptr = hdw->controls + idx + CTRLDEF_COUNT; 2554 cptr = hdw->controls + idx + CTRLDEF_COUNT;