Doug / smarc-fsl-linux-kernel

Download as
Browse Code ยป

Commit f1d9e4586e79e1fc7b26d1c567599a904013f199

Authored by Julia Lawall
Committed by Eric Van Hensbergen
1 parent 7dd0cdc51c

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>

Showing 1 changed file with 1 additions and 1 deletions Side-by-side Diff

... ... @@ -160,7 +160,7 @@
160 160 v9ses->flags |= V9FS_ACCESS_ANY;
161 161 else {
162 162 v9ses->flags |= V9FS_ACCESS_SINGLE;
163   - v9ses->uid = simple_strtol(s, &e, 10);
  163 + v9ses->uid = simple_strtoul(s, &e, 10);
164 164 if (*e != '\0')
165 165 v9ses->uid = ~0;
166 166 }