From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1755320AbcGUAyK (ORCPT <rfc822;w@1wt.eu>);
	Wed, 20 Jul 2016 20:54:10 -0400
Received: from mail-pf0-f195.google.com ([209.85.192.195]:35026 "EHLO
	mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1755084AbcGUAyI (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 20 Jul 2016 20:54:08 -0400
Date: Wed, 20 Jul 2016 17:54:04 -0700
From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Samuel Thibault <samuel.thibault@ens-lyon.org>,
        Muhammad Falak R Wani <falakreyaz@gmail.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Jiri Slaby <jslaby@suse.com>, Pavel Machek <pavel@ucw.cz>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] tty/vt/keyboard: use memdup_user().
Message-ID: <20160721005404.GE26006@dtor-ws>
References: <1463747026-23968-1-git-send-email-falakreyaz@gmail.com>
 <20160520122705.GW10620@var.bordeaux.inria.fr>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20160520122705.GW10620@var.bordeaux.inria.fr>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, May 20, 2016 at 02:27:05PM +0200, Samuel Thibault wrote:
> Muhammad Falak R Wani, on Fri 20 May 2016 17:53:28 +0530, wrote:
> > Use memdup_user to duplicate a memory region from user-space to
> > kernel-space, instead of open coding using kmalloc & copy_from_user.
> > 
> > Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com>
> 
> Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

Applied, thank you.

> 
> > ---
> >  drivers/tty/vt/keyboard.c | 14 ++++----------
> >  1 file changed, 4 insertions(+), 10 deletions(-)
> > 
> > diff --git a/drivers/tty/vt/keyboard.c b/drivers/tty/vt/keyboard.c
> > index f973bfc..698ea43 100644
> > --- a/drivers/tty/vt/keyboard.c
> > +++ b/drivers/tty/vt/keyboard.c
> > @@ -1745,16 +1745,10 @@ int vt_do_diacrit(unsigned int cmd, void __user *udp, int perm)
> >  			return -EINVAL;
> >  
> >  		if (ct) {
> > -			buf = kmalloc(ct * sizeof(struct kbdiacruc),
> > -								GFP_KERNEL);
> > -			if (buf == NULL)
> > -				return -ENOMEM;
> > -
> > -			if (copy_from_user(buf, a->kbdiacruc,
> > -					ct * sizeof(struct kbdiacruc))) {
> > -				kfree(buf);
> > -				return -EFAULT;
> > -			}
> > +			buf = memdup_user(a->kbdiacruc,
> > +					  ct * sizeof(struct kbdiacruc));
> > +			if (IS_ERR(buf))
> > +				return PTR_ERR(buf);
> >  		} 
> >  		spin_lock_irqsave(&kbd_event_lock, flags);
> >  		if (ct)
> > -- 
> > 1.9.1
> > 
> 
> -- 
> Samuel
> (03:13:14) <j> bon
> (03:13:19) <j> il est tard :p
> (03:13:25) <g> c'est l'heure de manger
> (03:13:38) <j> hm j'ai mangé à 1h moi, j'ai des horaires raisonnables

-- 
Dmitry