diff options
| author | Steven Rostedt <rostedt@goodmis.org> | 2009-09-18 13:05:42 -0700 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2009-09-24 07:47:22 -0400 |
| commit | 14be27460e4722d7135de3c46d043b4fc4382247 (patch) | |
| tree | d7026e0e43c1f048f2a0f31a54beb93b91b5fb89 /fs/inode.c | |
| parent | 94a8d5caba74211ec76dac80fc6e2d5c391530df (diff) | |
| download | lwn-14be27460e4722d7135de3c46d043b4fc4382247.tar.gz lwn-14be27460e4722d7135de3c46d043b4fc4382247.zip | |
libfs: make simple_read_from_buffer conventional
Impact: have simple_read_from_buffer conform to standards
It was brought to my attention by Andrew Morton, Theodore Tso, and H.
Peter Anvin that a read from userspace should only return -EFAULT if
nothing was actually read.
Looking at the simple_read_from_buffer I noticed that this function does
not conform to that rule. This patch fixes that function.
[akpm@linux-foundation.org: simplification suggested by hpa]
[hpa@zytor.com: fix count==0 handling]
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Theodore Ts'o <tytso@mit.edu>
Cc: Ingo Molnar <mingo@elte.hu>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/inode.c')
0 files changed, 0 insertions, 0 deletions