From: Markus Elfring <[email protected]>
Date: Mon, 6 Feb 2017 19:19:22 +0100

Replace the specification of a data structure 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.

Signed-off-by: Markus Elfring <[email protected]>
---
 drivers/hid/hid-core.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c
index 51f3547ca44f..404eccd0d670 100644
--- a/drivers/hid/hid-core.c
+++ b/drivers/hid/hid-core.c
@@ -138,11 +138,9 @@ static int open_collection(struct hid_parser *parser, 
unsigned type)
                        return -ENOMEM;
 
                memcpy(collection, parser->device->collection,
-                       sizeof(struct hid_collection) *
-                       parser->device->collection_size);
+                      sizeof(*collection) * parser->device->collection_size);
                memset(collection + parser->device->collection_size, 0,
-                       sizeof(struct hid_collection) *
-                       parser->device->collection_size);
+                      sizeof(*collection) * parser->device->collection_size);
                kfree(parser->device->collection);
                parser->device->collection = collection;
                parser->device->collection_size *= 2;
-- 
2.11.1

Reply via email to