diff options
| author | Vincent Li <macli@brc.ubc.ca> | 2009-09-22 16:45:38 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-09-23 07:39:40 -0700 |
| commit | cba8aafe1e07dfc8bae5ba78be8e02883bd34d31 (patch) | |
| tree | 473586963e635921613db5e4ae8ac339ae497a18 /lib/dec_and_lock.c | |
| parent | fb92a4b068be96799da3748c11cbd69760e44d7b (diff) | |
| download | blackbird-op-linux-cba8aafe1e07dfc8bae5ba78be8e02883bd34d31.tar.gz blackbird-op-linux-cba8aafe1e07dfc8bae5ba78be8e02883bd34d31.zip | |
fs/proc/base.c: fix proc_fault_inject_write() input sanity check
Remove obfuscated zero-length input check and return -EINVAL instead of
-EIO error to make the error message clear to user. Add whitespace
stripping. No functionality changes.
The old code:
echo 1 > /proc/pid/make-it-fail (ok)
echo 1foo > /proc/pid/make-it-fail (-bash: echo: write error: Input/output error)
The new code:
echo 1 > /proc/pid/make-it-fail (ok)
echo 1foo > /proc/pid/make-it-fail (-bash: echo: write error: Invalid argument)
This patch is conservative in changes to not breaking existing
scripts/applications.
Signed-off-by: Vincent Li <macli@brc.ubc.ca>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'lib/dec_and_lock.c')
0 files changed, 0 insertions, 0 deletions
