Messages in this thread | | | Subject | Re: [PATCH 1/2] sparc: Use kmalloc_array() in three functions | From | Paolo Bonzini <> | Date | Mon, 29 Aug 2016 11:47:38 +0200 |
| |
On 25/08/2016 10:22, SF Markus Elfring wrote: > --- a/arch/sparc/kernel/led.c > +++ b/arch/sparc/kernel/led.c > @@ -69,7 +69,7 @@ static ssize_t led_proc_write(struct file *file, const char __user *buffer, > if (count > LED_MAX_LENGTH) > count = LED_MAX_LENGTH; > > - buf = kmalloc(sizeof(char) * (count + 1), GFP_KERNEL); > + buf = kmalloc_array(count + 1, sizeof(*buf), GFP_KERNEL); > if (!buf) > return -ENOMEM; >
Here it's probably best to just remove sizeof(char) completely, as it's 1 by definition.
Paolo
| |