From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752518AbdH2LGL (ORCPT ); Tue, 29 Aug 2017 07:06:11 -0400 Received: from mout.web.de ([212.227.15.4]:50443 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751873AbdH2LGI (ORCPT ); Tue, 29 Aug 2017 07:06:08 -0400 Subject: [PATCH 2/2] [media] imon: Improve a size determination in two functions From: SF Markus Elfring To: linux-media@vger.kernel.org, Andi Shyti , Arnd Bergmann , Geliang Tang , Hans Verkuil , Mauro Carvalho Chehab , Sakari Ailus , Sean Young , Wolfram Sang Cc: LKML , kernel-janitors@vger.kernel.org References: <09417655-9241-72f4-6484-a3c8b3eae87a@users.sourceforge.net> Message-ID: <8a174347-580c-520c-436b-7d9406fe94cd@users.sourceforge.net> Date: Tue, 29 Aug 2017 13:05:51 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <09417655-9241-72f4-6484-a3c8b3eae87a@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 8bit X-Provags-ID: V03:K0:Ng2y8tUqhtF2PckCiP+A7SrqYEXaJgsbLxrmKTx/NrqllwIRenZ z6RrGLW6WafH73AFlja2eGZPlq7sSskNnUfMrJYHI5joTm5Khi/k9dCI16ef7EexEuaSA5h 7b8tQDXHAD7bZKWac6UBABCyZj2KW/+pdQFPzBpS+M8wp2oC2xFfiJpsMUp0aHQBMCEMAAh z09JwfOUgREbmeDV5Y7wg== X-UI-Out-Filterresults: notjunk:1;V01:K0:wZNmjAES9xY=:zumdYPefsX7pFmqWp19WCP uDlSp//usZSs56Nea0rMveiXQ7SMKjWI/s6r7ftI2koj7il7xDuMvtcqtipRY0RCLZ/oljGnl mGtKXw3Cj1IcoqRvZ+NpEM7OJi4vlU5d6+Gw/K5HvCuaG7HIk2kVWGiTxuO46lNVGbPA9ZtW9 WYFiAPL/8MI2z1c1IDuQiBrdLbbswW/ckTJNiHsz1AXw2foFVWmfqeea0RcDeSRTydY6D2oKZ kczxgcBQHCZ2/MW1t1TFbFCPC5F3axgUl958oK7gkbc7MZYXPqp+jqCkj/zCGv3yVSRm33BF5 lHmCwlRfpQRiS18LqmcT3q3Ao0gj5LeqBPTrIzoVINYnwYA++HazvAjz+OAbt/6j1s9hcZiI3 2o7zeSOxNWdPyvhm7bi5WQ4okzWKk33AlJXGuQ5FzSbbqZ27I3copThRF+X9H282mMN0zfQdB qb24iZr7TLxT6rmWq3ojQM5jZybfUNOSobvUf+79NegkoM5YLgWNOiwKGuFpSNFziAl7Aioue ts+/W+lqjOjzA6rTh6LtwFDw5IWgG4hNjytNX8ijT/YPTibGyjVZDHfZDjegriHLKeNPDr4w+ qaeOEPTBq+/ZvcQjF9cCorpOPcbMJEJKXsls9vIbqVEeHtm5vhAJYC7DtCp9vp4Mq+yXkPEAy x8+p9+liSQAOl5lDBlazpmdj6xeSrynywPXz9F8sKzG4wtE45YlR9/EVdjUoYv1CWos/1KLxb iyYR7ttjqqFy21c0KH4JIZOo4Fvt4keKepGxaLrDV5/y0xItJdvgzXQMyIUBEw+itvF9iEGCk 2xXLl7AEa5AZLUUdukqB6k2Y8MDjTXMNiJnZ/U9x7muNb//ekA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Tue, 29 Aug 2017 12:45:59 +0200 Replace the specification of data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/rc/imon.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/media/rc/imon.c b/drivers/media/rc/imon.c index e6978f1b7f2c..27aab02b75b5 100644 --- a/drivers/media/rc/imon.c +++ b/drivers/media/rc/imon.c @@ -602,8 +602,7 @@ static int send_packet(struct imon_context *ictx) ictx->tx_urb->actual_length = 0; } else { /* fill request into kmalloc'ed space: */ - control_req = kmalloc(sizeof(struct usb_ctrlrequest), - GFP_KERNEL); + control_req = kmalloc(sizeof(*control_req), GFP_KERNEL); if (control_req == NULL) return -ENOMEM; @@ -2309,7 +2308,7 @@ static struct imon_context *imon_init_intf0(struct usb_interface *intf, struct usb_host_interface *iface_desc; int ret = -ENOMEM; - ictx = kzalloc(sizeof(struct imon_context), GFP_KERNEL); + ictx = kzalloc(sizeof(*ictx), GFP_KERNEL); if (!ictx) goto exit; -- 2.14.1