diff options
| author | Dominique Martinet <asmadeus@codewreck.org> | 2021-11-02 14:23:37 +0100 |
|---|---|---|
| committer | Dominique Martinet <asmadeus@codewreck.org> | 2021-11-03 09:45:04 +0100 |
| commit | 10c69a0d08bb2ad5bf17ef85a27362783ffb893a (patch) | |
| tree | 97c21241e2752457fd8116138cc2342d9343fffd /fs/9p/v9fs.c | |
| parent | 9p: fix file headers (diff) | |
| download | linux-10c69a0d08bb2ad5bf17ef85a27362783ffb893a.tar.xz linux-10c69a0d08bb2ad5bf17ef85a27362783ffb893a.zip | |
9p v9fs_parse_options: replace simple_strtoul with kstrtouint
This is also a checkpatch change, but this one might have more implications
so keeping this separate
Link: http://lkml.kernel.org/r/20211102134608.1588018-4-dominique.martinet@atmark-techno.com
Signed-off-by: Dominique Martinet <asmadeus@codewreck.org>
Diffstat (limited to 'fs/9p/v9fs.c')
| -rw-r--r-- | fs/9p/v9fs.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/fs/9p/v9fs.c b/fs/9p/v9fs.c index 0973d7a3536b..457d9ff584ec 100644 --- a/fs/9p/v9fs.c +++ b/fs/9p/v9fs.c @@ -164,7 +164,7 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses, char *opts) substring_t args[MAX_OPT_ARGS]; char *p; int option = 0; - char *s, *e; + char *s; int ret = 0; /* setup defaults */ @@ -319,12 +319,13 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses, char *opts) v9ses->flags |= V9FS_ACCESS_CLIENT; } else { uid_t uid; + v9ses->flags |= V9FS_ACCESS_SINGLE; - uid = simple_strtoul(s, &e, 10); - if (*e != '\0') { - ret = -EINVAL; - pr_info("Unknown access argument %s\n", - s); + r = kstrtouint(s, 10, &uid); + if (r) { + ret = r; + pr_info("Unknown access argument %s: %d\n", + s, r); kfree(s); continue; } |