From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751331AbaEWIVa (ORCPT ); Fri, 23 May 2014 04:21:30 -0400 Received: from mail-wi0-f178.google.com ([209.85.212.178]:56752 "EHLO mail-wi0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750906AbaEWIV2 (ORCPT ); Fri, 23 May 2014 04:21:28 -0400 From: Djordje Zekovic To: gregkh@linuxfoundation.org Cc: linux-kernel@vger.kernel.org, Djordje Zekovic Subject: [PATCH] staging: usbip: use kcalloc instead of kzalloc for array allocations Date: Fri, 23 May 2014 10:17:38 +0000 Message-Id: <1400840258-4919-1-git-send-email-dj@zeko.me> X-Mailer: git-send-email 1.8.5.2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. Signed-off-by: Djordje Zekovic --- drivers/staging/usbip/stub_tx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/usbip/stub_tx.c b/drivers/staging/usbip/stub_tx.c index 1622563..4994090 100644 --- a/drivers/staging/usbip/stub_tx.c +++ b/drivers/staging/usbip/stub_tx.c @@ -179,7 +179,7 @@ static int stub_send_ret_submit(struct stub_device *sdev) else iovnum = 2; - iov = kzalloc(iovnum * sizeof(struct kvec), GFP_KERNEL); + iov = kcalloc(iovnum * sizeof(struct kvec), GFP_KERNEL); if (!iov) { usbip_event_add(&sdev->ud, SDEV_EVENT_ERROR_MALLOC); -- 1.8.5.2