diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2016-08-25 07:23:06 -0400 |
---|---|---|
committer | David Vrabel <david.vrabel@citrix.com> | 2016-08-26 05:44:22 -0400 |
commit | 4f0fbdf22e739c94ad4c18c790be014dddaedd28 (patch) | |
tree | 80c5fad3b81aad9a2d2a7f37c383766ef653517c | |
parent | 0252937a87e1d46a8261da85cbd99dffe612a2d3 (diff) |
xen/grant-table: Use kmalloc_array() in arch_gnttab_valloc()
* A multiplication for the size determination of a memory allocation
indicated that an array data structure should be processed.
Thus reuse the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle software.
* Replace the specification of a data type 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>
Reviewed-by: Juergen Gross <jgross@suse.com>
Signed-off-by: David Vrabel <david.vrabel@citrix.com>
-rw-r--r-- | arch/x86/xen/grant-table.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/xen/grant-table.c b/arch/x86/xen/grant-table.c index de4144c24f1c..809b6c812654 100644 --- a/arch/x86/xen/grant-table.c +++ b/arch/x86/xen/grant-table.c | |||
@@ -89,7 +89,7 @@ void arch_gnttab_unmap(void *shared, unsigned long nr_gframes) | |||
89 | 89 | ||
90 | static int arch_gnttab_valloc(struct gnttab_vm_area *area, unsigned nr_frames) | 90 | static int arch_gnttab_valloc(struct gnttab_vm_area *area, unsigned nr_frames) |
91 | { | 91 | { |
92 | area->ptes = kmalloc(sizeof(pte_t *) * nr_frames, GFP_KERNEL); | 92 | area->ptes = kmalloc_array(nr_frames, sizeof(*area->ptes), GFP_KERNEL); |
93 | if (area->ptes == NULL) | 93 | if (area->ptes == NULL) |
94 | return -ENOMEM; | 94 | return -ENOMEM; |
95 | 95 | ||