diff options
| author |   Jesper Juhl <juhl-lkml@dif.dk> | 2005-06-18 23:00:00 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2005-06-18 23:00:00 -0700 |
| commit | 926d4b8122fb324de294a09a7d96d8af7cfc6861 (patch) | |
| tree | 1274e11e3e617936a1b5b6bda9e0978d678687fa /net | |
| parent | [IPV4]: [1/4] signed vs unsigned cleanup in net/ipv4/raw.c (diff) | |
| download | linux-dev-926d4b8122fb324de294a09a7d96d8af7cfc6861.tar.xz linux-dev-926d4b8122fb324de294a09a7d96d8af7cfc6861.zip | |
[IPV4]: [2/4] signed vs unsigned cleanup in net/ipv4/raw.c
This patch gets rid of the following gcc -W warning in net/ipv4/raw.c :
net/ipv4/raw.c:387: warning: comparison of unsigned expression < 0 is always false
Since 'len' is of type size_t it is unsigned and can thus never be <0, and
since this is obvious from the function declaration just a few lines above
I think it's ok to remove the pointless check for len<0.
Signed-off-by: Jesper Juhl <juhl-lkml@dif.dk>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -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 8d17dd3542df..1fcb23d33518 100644 --- a/net/ipv4/raw.c +++ b/net/ipv4/raw.c @@ -384,7 +384,7 @@ static int raw_sendmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *msg, int err; err = -EMSGSIZE; - if (len < 0 || len > 0xFFFF) + if (len > 0xFFFF) goto out; /* |