]> bbs.cooldavid.org Git - net-next-2.6.git/commit - net/ipv4/raw.c
[IPV4]: [3/4] signed vs unsigned cleanup in net/ipv4/raw.c
authorJesper Juhl <juhl-lkml@dif.dk>
Sun, 19 Jun 2005 06:00:15 +0000 (23:00 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sun, 19 Jun 2005 06:00:15 +0000 (23:00 -0700)
commit93765d8a435fa021c4b7cd0521b7959239d7158a
tree05858c1b7f3c8bf069a2b45051af9d9db8e0804b
parent926d4b8122fb324de294a09a7d96d8af7cfc6861
[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>
net/ipv4/raw.c