From: Markus Elfring <[email protected]>
Date: Fri, 7 Apr 2017 22:20:39 +0200

Replace the specification of three data structures by pointer dereferences
as the parameter for the operator "sizeof" to make the corresponding size
determinations a bit safer according to the Linux coding style convention.

Signed-off-by: Markus Elfring <[email protected]>
---
 drivers/infiniband/ulp/isert/ib_isert.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/infiniband/ulp/isert/ib_isert.c 
b/drivers/infiniband/ulp/isert/ib_isert.c
index c56af6183082..5dafda92a642 100644
--- a/drivers/infiniband/ulp/isert/ib_isert.c
+++ b/drivers/infiniband/ulp/isert/ib_isert.c
@@ -368,7 +368,7 @@ isert_device_get(struct rdma_cm_id *cma_id)
                }
        }
 
-       device = kzalloc(sizeof(struct isert_device), GFP_KERNEL);
+       device = kzalloc(sizeof(*device), GFP_KERNEL);
        if (!device) {
                mutex_unlock(&device_list_mutex);
                return ERR_PTR(-ENOMEM);
@@ -516,7 +516,7 @@ isert_connect_request(struct rdma_cm_id *cma_id, struct 
rdma_cm_event *event)
        isert_dbg("cma_id: %p, portal: %p\n",
                 cma_id, cma_id->context);
 
-       isert_conn = kzalloc(sizeof(struct isert_conn), GFP_KERNEL);
+       isert_conn = kzalloc(sizeof(*isert_conn), GFP_KERNEL);
        if (!isert_conn)
                return -ENOMEM;
 
@@ -2306,7 +2306,7 @@ isert_setup_np(struct iscsi_np *np,
        struct rdma_cm_id *isert_lid;
        int ret;
 
-       isert_np = kzalloc(sizeof(struct isert_np), GFP_KERNEL);
+       isert_np = kzalloc(sizeof(*isert_np), GFP_KERNEL);
        if (!isert_np)
                return -ENOMEM;
 
-- 
2.12.2

Reply via email to