orangefs: Use kcalloc() in orangefs_prepare_cdm_array()
authorMarkus Elfring <elfring@users.sourceforge.net>
Thu, 17 Aug 2017 19:18:01 +0000 (21:18 +0200)
committerMike Marshall <hubcap@omnibond.com>
Thu, 14 Sep 2017 18:58:30 +0000 (14:58 -0400)
* A multiplication for the size determination of a memory allocation
  indicated that an array data structure should be processed.
  Thus use the corresponding function "kcalloc".

  This issue was detected by using the Coccinelle software.

* Replace the specification of a data structure by a pointer dereference
  to make the corresponding size determination a bit safer according to
  the Linux coding style convention.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
fs/orangefs/orangefs-debugfs.c

index 93fe8f8..5f59917 100644 (file)
@@ -571,9 +571,7 @@ static int orangefs_prepare_cdm_array(char *debug_array_string)
                goto out;
        }
 
-       cdm_array =
-               kzalloc(cdm_element_count * sizeof(struct client_debug_mask),
-                       GFP_KERNEL);
+       cdm_array = kcalloc(cdm_element_count, sizeof(*cdm_array), GFP_KERNEL);
        if (!cdm_array) {
                rc = -ENOMEM;
                goto out;