diff options
author | Julia Lawall <julia@diku.dk> | 2008-12-19 16:50:22 -0600 |
---|---|---|
committer | Eric Van Hensbergen <ericvh@gmail.com> | 2008-12-19 16:50:22 -0600 |
commit | f1d9e4586e79e1fc7b26d1c567599a904013f199 (patch) | |
tree | 394c1d5b17a023f83f44ad71ae69daec0dfc94ad /fs | |
parent | 7dd0cdc51c126915e243ee3291f7b6b5ea5e5833 (diff) | |
download | op-kernel-dev-f1d9e4586e79e1fc7b26d1c567599a904013f199.zip op-kernel-dev-f1d9e4586e79e1fc7b26d1c567599a904013f199.tar.gz |
fs/9p: change simple_strtol to simple_strtoul
Since v9ses->uid is unsigned, it would seem better to use simple_strtoul that
simple_strtol.
A simplified version of the semantic patch that makes this change is as
follows: (http://www.emn.fr/x-info/coccinelle/)
// <smpl>
@r2@
long e;
position p;
@@
e = simple_strtol@p(...)
@@
position p != r2.p;
type T;
T e;
@@
e =
- simple_strtol@p
+ simple_strtoul
(...)
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Acked-by: Eric Van Hensbergen <ericvh@gmail.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/9p/v9fs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/9p/v9fs.c b/fs/9p/v9fs.c index 24eb010..332b5ff0 100644 --- a/fs/9p/v9fs.c +++ b/fs/9p/v9fs.c @@ -160,7 +160,7 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses) v9ses->flags |= V9FS_ACCESS_ANY; else { v9ses->flags |= V9FS_ACCESS_SINGLE; - v9ses->uid = simple_strtol(s, &e, 10); + v9ses->uid = simple_strtoul(s, &e, 10); if (*e != '\0') v9ses->uid = ~0; } |