diff options
| author | Jesper Juhl <juhl-lkml@dif.dk> | 2005-06-18 23:00:15 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-06-18 23:00:15 -0700 |
| commit | 93765d8a435fa021c4b7cd0521b7959239d7158a (patch) | |
| tree | 05858c1b7f3c8bf069a2b45051af9d9db8e0804b /net/ipv4/raw.c | |
| parent | 926d4b8122fb324de294a09a7d96d8af7cfc6861 (diff) | |
| download | linux-93765d8a435fa021c4b7cd0521b7959239d7158a.tar.gz linux-93765d8a435fa021c4b7cd0521b7959239d7158a.tar.bz2 linux-93765d8a435fa021c4b7cd0521b7959239d7158a.zip | |
[IPV4]: [3/4] signed vs unsigned cleanup in net/ipv4/raw.c
This patch changes the type of the local variable 'i' in
raw_probe_proto_opt() from 'int' to 'unsigned int'. The only use of 'i' in
this function is as a counter in a for() loop and subsequent index into
the msg->msg_iov[] array.
Since 'i' is compared in a loop to the unsigned variable msg->msg_iovlen
gcc -W generates this warning :
net/ipv4/raw.c:340: warning: comparison between signed and unsigned
Changing 'i' to unsigned silences this warning and is safe since the array
index can never be negative anyway, so unsigned int is the logical type to
use for 'i' and also enables a larger msg_iov[] array (but I don't know if
that will ever matter).
Signed-off-by: Jesper Juhl <juhl-lkml@dif.dk>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/raw.c')
| -rw-r--r-- | net/ipv4/raw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/raw.c b/net/ipv4/raw.c index 1fcb23d33518..085a08ea5a5e 100644 --- a/net/ipv4/raw.c +++ b/net/ipv4/raw.c @@ -332,7 +332,7 @@ static void raw_probe_proto_opt(struct flowi *fl, struct msghdr *msg) u8 __user *type = NULL; u8 __user *code = NULL; int probed = 0; - int i; + unsigned int i; if (!msg->msg_iov) return; |