summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYi Yang <yang.y.yi@gmail.com>2005-12-30 09:37:10 +0100
committerLinus Torvalds <torvalds@g5.osdl.org>2005-12-31 02:22:08 +0100
commit82c9df820112c6286a8e8fbe482e94b65b49062c (patch)
tree6f52b0ae3db256de9a3ac46083229d96f2d130e9
parentsysctl: don't overflow the user-supplied buffer with '\0' (diff)
downloadlinux-82c9df820112c6286a8e8fbe482e94b65b49062c.tar.xz
linux-82c9df820112c6286a8e8fbe482e94b65b49062c.zip
[PATCH] Fix false old value return of sysctl
For the sysctl syscall, if the user wants to get the old value of a sysctl entry and set a new value for it in the same syscall, the old value is always overwritten by the new value if the sysctl entry is of string type and if the user sets its strategy to sysctl_string. This issue lies in the strategy being run twice if the strategy is set to sysctl_string, the general strategy sysctl_string always returns 0 if success. Such strategy routines as sysctl_jiffies and sysctl_jiffies_ms return 1 because they do read and write for the sysctl entry. The strategy routine sysctl_string return 0 although it actually read and write the sysctl entry. According to my analysis, if a strategy routine do read and write, it should return 1, if it just does some necessary check but not read and write, it should return 0, for example sysctl_intvec. Signed-off-by: Yi Yang <yang.y.yi@gmail.com> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r--kernel/sysctl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index ad0425a8f709..e5102ea6e104 100644
--- a/kernel/sysctl.c
+++ b/kernel/sysctl.c
@@ -2221,7 +2221,7 @@ int sysctl_string(ctl_table *table, int __user *name, int nlen,
len--;
((char *) table->data)[len] = 0;
}
- return 0;
+ return 1;
}
/*