Commit 5418c692 authored by Jesper Juhl's avatar Jesper Juhl Committed by David S. Miller

[IPV4]: [1/4] signed vs unsigned cleanup in net/ipv4/raw.c

This patch silences these two gcc -W warnings in net/ipv4/raw.c :

net/ipv4/raw.c:517: warning: signed and unsigned type in conditional expression
net/ipv4/raw.c:613: warning: signed and unsigned type in conditional expression

It doesn't change the behaviour of the code, simply writes the conditional 
expression with plain 'if()' syntax instead of '? :' , but since this 
breaks it into sepperate statements gcc no longer complains about having 
both a signed and unsigned value in the same conditional expression.
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 94df109a
...@@ -514,7 +514,10 @@ static int raw_sendmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *msg, ...@@ -514,7 +514,10 @@ static int raw_sendmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *msg,
kfree(ipc.opt); kfree(ipc.opt);
ip_rt_put(rt); ip_rt_put(rt);
out: return err < 0 ? err : len; out:
if (err < 0)
return err;
return len;
do_confirm: do_confirm:
dst_confirm(&rt->u.dst); dst_confirm(&rt->u.dst);
...@@ -610,7 +613,10 @@ static int raw_recvmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *msg, ...@@ -610,7 +613,10 @@ static int raw_recvmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *msg,
copied = skb->len; copied = skb->len;
done: done:
skb_free_datagram(sk, skb); skb_free_datagram(sk, skb);
out: return err ? err : copied; out:
if (err)
return err;
return copied;
} }
static int raw_init(struct sock *sk) static int raw_init(struct sock *sk)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment