* [PATCH] xen/grant-table: Use kmalloc_array() in arch_gnttab_valloc()
@ 2016-08-25 11:30 SF Markus Elfring
2016-08-25 12:03 ` Juergen Gross
2016-08-26 9:48 ` [Xen-devel] " David Vrabel
0 siblings, 2 replies; 3+ messages in thread
From: SF Markus Elfring @ 2016-08-25 11:30 UTC (permalink / raw)
To: xen-devel, x86, Boris Ostrovsky, David Vrabel, H. Peter Anvin,
Ingo Molnar, Jürgen Groß,
Thomas Gleixner
Cc: LKML, kernel-janitors, Julia Lawall, Paolo Bonzini
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Thu, 25 Aug 2016 13:23:06 +0200
* 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>
---
arch/x86/xen/grant-table.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/xen/grant-table.c b/arch/x86/xen/grant-table.c
index de4144c..809b6c8 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)
static int arch_gnttab_valloc(struct gnttab_vm_area *area, unsigned nr_frames)
{
- area->ptes = kmalloc(sizeof(pte_t *) * nr_frames, GFP_KERNEL);
+ area->ptes = kmalloc_array(nr_frames, sizeof(*area->ptes), GFP_KERNEL);
if (area->ptes == NULL)
return -ENOMEM;
--
2.9.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] xen/grant-table: Use kmalloc_array() in arch_gnttab_valloc()
2016-08-25 11:30 [PATCH] xen/grant-table: Use kmalloc_array() in arch_gnttab_valloc() SF Markus Elfring
@ 2016-08-25 12:03 ` Juergen Gross
2016-08-26 9:48 ` [Xen-devel] " David Vrabel
1 sibling, 0 replies; 3+ messages in thread
From: Juergen Gross @ 2016-08-25 12:03 UTC (permalink / raw)
To: SF Markus Elfring, xen-devel, x86, Boris Ostrovsky, David Vrabel,
H. Peter Anvin, Ingo Molnar, Thomas Gleixner
Cc: LKML, kernel-janitors, Julia Lawall, Paolo Bonzini
On 25/08/16 13:30, SF Markus Elfring wrote:
> From: Markus Elfring <elfring@users.sourceforge.net>
> Date: Thu, 25 Aug 2016 13:23:06 +0200
>
> * 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>
Juergen
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Xen-devel] [PATCH] xen/grant-table: Use kmalloc_array() in arch_gnttab_valloc()
2016-08-25 11:30 [PATCH] xen/grant-table: Use kmalloc_array() in arch_gnttab_valloc() SF Markus Elfring
2016-08-25 12:03 ` Juergen Gross
@ 2016-08-26 9:48 ` David Vrabel
1 sibling, 0 replies; 3+ messages in thread
From: David Vrabel @ 2016-08-26 9:48 UTC (permalink / raw)
To: SF Markus Elfring, xen-devel, x86, Boris Ostrovsky, David Vrabel,
H. Peter Anvin, Ingo Molnar, Jürgen Groß,
Thomas Gleixner
Cc: Julia Lawall, kernel-janitors, LKML, Paolo Bonzini
On 25/08/16 12:30, SF Markus Elfring wrote:
> From: Markus Elfring <elfring@users.sourceforge.net>
> Date: Thu, 25 Aug 2016 13:23:06 +0200
>
> * 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.
Applied to for-linus-4.9, thanks.
David
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-08-26 9:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-25 11:30 [PATCH] xen/grant-table: Use kmalloc_array() in arch_gnttab_valloc() SF Markus Elfring
2016-08-25 12:03 ` Juergen Gross
2016-08-26 9:48 ` [Xen-devel] " David Vrabel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).