Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1637009

Re: [PATCH] platform/x86: toshiba_acpi: use memdup_user_nul

Show all headers | View raw

On Sat, May 6, 2017 at 6:40 PM, Geliang Tang <geliangtang@gmail.com> wrote:
> Use memdup_user_nul() helper instead of open-coding to simplify the
> code.
>

Pushed to testing, thanks.

> Signed-off-by: Geliang Tang <geliangtang@gmail.com>
> ---
>  drivers/platform/x86/toshiba_acpi.c | 11 +++--------
>  1 file changed, 3 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c
> index d0daf75..88f9f79 100644
> --- a/drivers/platform/x86/toshiba_acpi.c
> +++ b/drivers/platform/x86/toshiba_acpi.c
> @@ -1502,14 +1502,9 @@ static ssize_t video_proc_write(struct file *file, const char __user *buf,
>         int ret;
>         u32 video_out;
>
> -       cmd = kmalloc(count + 1, GFP_KERNEL);
> -       if (!cmd)
> -               return -ENOMEM;
> -       if (copy_from_user(cmd, buf, count)) {
> -               kfree(cmd);
> -               return -EFAULT;
> -       }
> -       cmd[count] = '\0';
> +       cmd = memdup_user_nul(buf, count);
> +       if (IS_ERR(cmd))
> +               return PTR_ERR(cmd);
>
>         buffer = cmd;
>
> --
> 2.9.3
>



-- 
With Best Regards,
Andy Shevchenko

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] drm/i915: use memdup_user_nul Geliang Tang <geliangtang@gmail.com> - 2017-05-06 17:50 +0200
  [PATCH] ima: use memdup_user_nul Geliang Tang <geliangtang@gmail.com> - 2017-05-06 17:50 +0200
    Re: [PATCH] ima: use memdup_user_nul Mimi Zohar <zohar@linux.vnet.ibm.com> - 2017-05-08 17:50 +0200
  [PATCH] platform/x86: toshiba_acpi: use memdup_user_nul Geliang Tang <geliangtang@gmail.com> - 2017-05-06 17:50 +0200
    Re: [PATCH] platform/x86: toshiba_acpi: use memdup_user_nul Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-05-07 23:40 +0200
  [PATCH] sparc: use memdup_user_nul in sun4m LED driver Geliang Tang <geliangtang@gmail.com> - 2017-05-06 17:50 +0200
    Re: [PATCH] sparc: use memdup_user_nul in sun4m LED driver David Miller <davem@davemloft.net> - 2017-05-09 21:20 +0200
  Re: [Intel-gfx] [PATCH] drm/i915: use memdup_user_nul Daniel Vetter <daniel@ffwll.ch> - 2017-05-08 09:30 +0200

csiph-web