inet6: Conversion from u8 to int
authorGerrit Renker <gerrit@erg.abdn.ac.uk>
Sun, 9 Aug 2009 08:12:48 +0000 (08:12 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 13 Aug 2009 23:43:31 +0000 (16:43 -0700)
This replaces assignments of the type "int on LHS" = "u8 on RHS" with
simpler code. The LHS can express all of the unsigned right hand side
values, hence the assigned value can not be negative.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/icmp.c
net/ipv6/ip6_output.c
net/ipv6/ipv6_sockglue.c
net/ipv6/raw.c
net/ipv6/udp.c

index eab62a7a8f06e54fd7e161b8312565f8da59ed60..e2325f6a05fbd4e9a0d154ca5ddcf6386296760e 100644 (file)
@@ -323,7 +323,7 @@ void icmpv6_send(struct sk_buff *skb, u8 type, u8 code, __u32 info,
        int iif = 0;
        int addr_type = 0;
        int len;
-       int hlimit, tclass;
+       int hlimit;
        int err = 0;
 
        if ((u8 *)hdr < skb->head ||
@@ -469,10 +469,6 @@ route_done:
        if (hlimit < 0)
                hlimit = ip6_dst_hoplimit(dst);
 
-       tclass = np->tclass;
-       if (tclass < 0)
-               tclass = 0;
-
        msg.skb = skb;
        msg.offset = skb_network_offset(skb);
        msg.type = type;
@@ -488,8 +484,8 @@ route_done:
 
        err = ip6_append_data(sk, icmpv6_getfrag, &msg,
                              len + sizeof(struct icmp6hdr),
-                             sizeof(struct icmp6hdr),
-                             hlimit, tclass, NULL, &fl, (struct rt6_info*)dst,
+                             sizeof(struct icmp6hdr), hlimit,
+                             np->tclass, NULL, &fl, (struct rt6_info*)dst,
                              MSG_DONTWAIT);
        if (err) {
                ip6_flush_pending_frames(sk);
@@ -522,7 +518,6 @@ static void icmpv6_echo_reply(struct sk_buff *skb)
        struct dst_entry *dst;
        int err = 0;
        int hlimit;
-       int tclass;
 
        saddr = &ipv6_hdr(skb)->daddr;
 
@@ -562,10 +557,6 @@ static void icmpv6_echo_reply(struct sk_buff *skb)
        if (hlimit < 0)
                hlimit = ip6_dst_hoplimit(dst);
 
-       tclass = np->tclass;
-       if (tclass < 0)
-               tclass = 0;
-
        idev = in6_dev_get(skb->dev);
 
        msg.skb = skb;
@@ -573,7 +564,7 @@ static void icmpv6_echo_reply(struct sk_buff *skb)
        msg.type = ICMPV6_ECHO_REPLY;
 
        err = ip6_append_data(sk, icmpv6_getfrag, &msg, skb->len + sizeof(struct icmp6hdr),
-                               sizeof(struct icmp6hdr), hlimit, tclass, NULL, &fl,
+                               sizeof(struct icmp6hdr), hlimit, np->tclass, NULL, &fl,
                                (struct rt6_info*)dst, MSG_DONTWAIT);
 
        if (err) {
index 93beee944657f12c61482996b941dd6b9cf1f7be..6ad5aadf81ad73b53f7e2320c87fe265140df691 100644 (file)
@@ -194,7 +194,8 @@ int ip6_xmit(struct sock *sk, struct sk_buff *skb, struct flowi *fl,
        struct ipv6hdr *hdr;
        u8  proto = fl->proto;
        int seg_len = skb->len;
-       int hlimit, tclass;
+       int hlimit = -1;
+       int tclass = 0;
        u32 mtu;
 
        if (opt) {
@@ -237,19 +238,13 @@ int ip6_xmit(struct sock *sk, struct sk_buff *skb, struct flowi *fl,
        /*
         *      Fill in the IPv6 header
         */
-
-       hlimit = -1;
-       if (np)
+       if (np) {
+               tclass = np->tclass;
                hlimit = np->hop_limit;
+       }
        if (hlimit < 0)
                hlimit = ip6_dst_hoplimit(dst);
 
-       tclass = -1;
-       if (np)
-               tclass = np->tclass;
-       if (tclass < 0)
-               tclass = 0;
-
        *(__be32 *)hdr = htonl(0x60000000 | (tclass << 20)) | fl->fl6_flowlabel;
 
        hdr->payload_len = htons(seg_len);
index a7fdf9a27f15e4b16cf36cb714f58e238921f451..c390b1eafb0b52a62b1b667112ea5583077ebd83 100644 (file)
@@ -1037,8 +1037,6 @@ static int do_ipv6_getsockopt(struct sock *sk, int level, int optname,
 
        case IPV6_TCLASS:
                val = np->tclass;
-               if (val < 0)
-                       val = 0;
                break;
 
        case IPV6_RECVTCLASS:
index d6c3c1c34b2d0cf1eac94211675800fce65be843..506841030fbee0dab9fe980d8f717e407babc06f 100644 (file)
@@ -877,11 +877,8 @@ static int rawv6_sendmsg(struct kiocb *iocb, struct sock *sk,
                        hlimit = ip6_dst_hoplimit(dst);
        }
 
-       if (tclass < 0) {
+       if (tclass < 0)
                tclass = np->tclass;
-               if (tclass < 0)
-                       tclass = 0;
-       }
 
        if (msg->msg_flags&MSG_CONFIRM)
                goto do_confirm;
index d79fa6724451549659339a5282d15b4e44c48437..20d2ffc15f0d1096acf7021943ec168ddd759068 100644 (file)
@@ -946,11 +946,8 @@ do_udp_sendmsg:
                        hlimit = ip6_dst_hoplimit(dst);
        }
 
-       if (tclass < 0) {
+       if (tclass < 0)
                tclass = np->tclass;
-               if (tclass < 0)
-                       tclass = 0;
-       }
 
        if (msg->msg_flags&MSG_CONFIRM)
                goto do_confirm;