diff options
| author | Peter Huewe <peterhuewe@gmx.de> | 2012-04-14 20:20:27 -0400 |
|---|---|---|
| committer | Miklos Szeredi <mszeredi@suse.cz> | 2012-04-25 06:25:05 -0400 |
| commit | e2690695ce5085677b84fbf2e38d2ed57cad39cd (patch) | |
| tree | 3e9f852e1590d15057bd46ec7cd640db27b04257 | |
| parent | 66f75a5d028beaf67c931435fdc3e7823125730c (diff) | |
fuse: Convert to kstrtoul_from_user
This patch replaces the code for getting an number from a
userspace buffer by a simple call to kstroul_from_user.
This makes it easier to read and less error prone.
Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
| -rw-r--r-- | fs/fuse/control.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/fs/fuse/control.c b/fs/fuse/control.c index 42593c587d48..03ff5b1eba93 100644 --- a/fs/fuse/control.c +++ b/fs/fuse/control.c | |||
| @@ -75,19 +75,13 @@ static ssize_t fuse_conn_limit_write(struct file *file, const char __user *buf, | |||
| 75 | unsigned global_limit) | 75 | unsigned global_limit) |
| 76 | { | 76 | { |
| 77 | unsigned long t; | 77 | unsigned long t; |
| 78 | char tmp[32]; | ||
| 79 | unsigned limit = (1 << 16) - 1; | 78 | unsigned limit = (1 << 16) - 1; |
| 80 | int err; | 79 | int err; |
| 81 | 80 | ||
| 82 | if (*ppos || count >= sizeof(tmp) - 1) | 81 | if (*ppos) |
| 83 | return -EINVAL; | ||
| 84 | |||
| 85 | if (copy_from_user(tmp, buf, count)) | ||
| 86 | return -EINVAL; | 82 | return -EINVAL; |
| 87 | 83 | ||
| 88 | tmp[count] = '\0'; | 84 | err = kstrtoul_from_user(buf, count, 0, &t); |
| 89 | |||
| 90 | err = strict_strtoul(tmp, 0, &t); | ||
| 91 | if (err) | 85 | if (err) |
| 92 | return err; | 86 | return err; |
| 93 | 87 | ||
