--- zzzz-none-000/linux-2.6.39.4/net/ipv6/ndisc.c 2011-08-03 19:43:28.000000000 +0000 +++ puma6-arm-6490-729/linux-2.6.39.4/net/ipv6/ndisc.c 2021-11-10 13:23:11.000000000 +0000 @@ -215,7 +215,8 @@ memcpy(opt+2, data, data_len); data_len += 2; opt += data_len; - if ((space -= data_len) > 0) + space -= data_len; + if (space > 0) memset(opt, 0, space); return opt + space; } @@ -378,7 +379,7 @@ rcu_read_lock(); in6_dev = in6_dev_get(dev); - if (in6_dev == NULL) { + if (!in6_dev) { rcu_read_unlock(); return -EINVAL; } @@ -425,7 +426,7 @@ struct in6_addr maddr; struct net_device *dev = n->dev; - if (dev == NULL || __in6_dev_get(dev) == NULL) + if (!dev || !__in6_dev_get(dev)) return -EINVAL; addrconf_addr_solict_mult(addr, &maddr); ipv6_dev_mc_inc(dev, &maddr); @@ -438,7 +439,7 @@ struct in6_addr maddr; struct net_device *dev = n->dev; - if (dev == NULL || __in6_dev_get(dev) == NULL) + if (!dev || !__in6_dev_get(dev)) return; addrconf_addr_solict_mult(addr, &maddr); ipv6_dev_mc_dec(dev, &maddr); @@ -620,7 +621,7 @@ .icmp6_type = NDISC_NEIGHBOUR_SOLICITATION, }; - if (saddr == NULL) { + if (!saddr) { if (ipv6_get_lladdr(dev, &addr_buf, (IFA_F_TENTATIVE|IFA_F_OPTIMISTIC))) return; @@ -691,7 +692,8 @@ if (skb && ipv6_chk_addr(dev_net(dev), &ipv6_hdr(skb)->saddr, dev, 1)) saddr = &ipv6_hdr(skb)->saddr; - if ((probes -= neigh->parms->ucast_probes) < 0) { + probes -= neigh->parms->ucast_probes; + if (probes < 0) { if (!(neigh->nud_state & NUD_VALID)) { ND_PRINTK1(KERN_DEBUG "%s(): trying to ucast probe in NUD_INVALID: %pI6\n", __func__, target); @@ -1078,13 +1080,13 @@ size_t msg_size = base_size + nla_total_size(sizeof(struct in6_addr)); skb = nlmsg_new(msg_size, GFP_ATOMIC); - if (skb == NULL) { + if (!skb) { err = -ENOBUFS; goto errout; } nlh = nlmsg_put(skb, 0, 0, RTM_NEWNDUSEROPT, base_size, 0); - if (nlh == NULL) { + if (!nlh) { goto nla_put_failure; } @@ -1162,7 +1164,7 @@ */ in6_dev = in6_dev_get(skb->dev); - if (in6_dev == NULL) { + if (!in6_dev) { ND_PRINTK0(KERN_ERR "ICMPv6 RA: can't find inet6 device for %s.\n", skb->dev->name); @@ -1228,12 +1230,12 @@ rt = NULL; } - if (rt == NULL && lifetime) { + if (!rt && lifetime) { ND_PRINTK3(KERN_DEBUG "ICMPv6 RA: adding default router.\n"); rt = rt6_add_dflt_router(&ipv6_hdr(skb)->saddr, skb->dev, pref); - if (rt == NULL) { + if (!rt) { ND_PRINTK0(KERN_ERR "ICMPv6 RA: %s() failed to add default route.\n", __func__); @@ -1242,7 +1244,7 @@ } neigh = rt->rt6i_nexthop; - if (neigh == NULL) { + if (!neigh) { ND_PRINTK0(KERN_ERR "ICMPv6 RA: %s() got default router without neighbour.\n", __func__); @@ -1541,7 +1543,7 @@ &saddr_buf, &ipv6_hdr(skb)->saddr, dev->ifindex); dst = ip6_route_output(net, NULL, &fl6); - if (dst == NULL) + if (!dst) return; dst = xfrm_lookup(net, dst, flowi6_to_flowi(&fl6), NULL, 0); @@ -1580,7 +1582,7 @@ (MAX_HEADER + sizeof(struct ipv6hdr) + len + LL_ALLOCATED_SPACE(dev)), 1, &err); - if (buff == NULL) { + if (!buff) { ND_PRINTK0(KERN_ERR "ICMPv6 Redirect: %s() failed to allocate an skb, err=%d.\n", __func__, err);