diff options
| author | Li Zefan <lizf@cn.fujitsu.com> | 2009-03-31 15:23:16 -0700 | 
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-04-01 08:59:13 -0700 | 
| commit | 610a77e04a8d9fe8764dc484e2182fa251ce1cc2 (patch) | |
| tree | ec55b9b3698321c3e85cd649fa927554027c75d0 /arch/avr32/mm/fault.c | |
| parent | e2f17d9459aeccf4e013e31cbd741d6b1858eec4 (diff) | |
| download | olio-linux-3.10-610a77e04a8d9fe8764dc484e2182fa251ce1cc2.tar.xz olio-linux-3.10-610a77e04a8d9fe8764dc484e2182fa251ce1cc2.zip  | |
memdup_user(): introduce
I notice there are many places doing copy_from_user() which follows
kmalloc():
        dst = kmalloc(len, GFP_KERNEL);
        if (!dst)
                return -ENOMEM;
        if (copy_from_user(dst, src, len)) {
		kfree(dst);
		return -EFAULT
	}
memdup_user() is a wrapper of the above code.  With this new function, we
don't have to write 'len' twice, which can lead to typos/mistakes.  It
also produces smaller code and kernel text.
A quick grep shows 250+ places where memdup_user() *may* be used.  I'll
prepare a patchset to do this conversion.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Cc: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Americo Wang <xiyou.wangcong@gmail.com>
Cc: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/avr32/mm/fault.c')
0 files changed, 0 insertions, 0 deletions