diff options
author | Chen Gang <gang.chen@asianux.com> | 2013-11-12 18:11:21 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-11-12 22:09:33 -0500 |
commit | 7833819d2e5b2a0189f6e9d445ba03f4d7ba2656 (patch) | |
tree | e6fc6a60153b0f669e3c57c203666982f00fed0e /kernel | |
parent | 1310a5a99d900ee30b9f171146406bde0c6c2bd4 (diff) |
kernel/sysctl.c: check return value after call proc_put_char() in __do_proc_doulongvec_minmax()
Need to check the return value of proc_put_char(), as was done in
__do_proc_doulongvec_minmax().
Signed-off-by: Chen Gang <gang.chen@asianux.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/sysctl.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/kernel/sysctl.c b/kernel/sysctl.c index 36547dddcdb8..d37d9dd8f463 100644 --- a/kernel/sysctl.c +++ b/kernel/sysctl.c | |||
@@ -2222,8 +2222,11 @@ static int __do_proc_doulongvec_minmax(void *data, struct ctl_table *table, int | |||
2222 | *i = val; | 2222 | *i = val; |
2223 | } else { | 2223 | } else { |
2224 | val = convdiv * (*i) / convmul; | 2224 | val = convdiv * (*i) / convmul; |
2225 | if (!first) | 2225 | if (!first) { |
2226 | err = proc_put_char(&buffer, &left, '\t'); | 2226 | err = proc_put_char(&buffer, &left, '\t'); |
2227 | if (err) | ||
2228 | break; | ||
2229 | } | ||
2227 | err = proc_put_long(&buffer, &left, val, false); | 2230 | err = proc_put_long(&buffer, &left, val, false); |
2228 | if (err) | 2231 | if (err) |
2229 | break; | 2232 | break; |