kernel: Simplify various redundant conditions.
authorSascha Wildner <saw@online.de>
Wed, 11 Oct 2017 14:55:23 +0000 (16:55 +0200)
committerSascha Wildner <saw@online.de>
Wed, 11 Oct 2017 14:55:23 +0000 (16:55 +0200)
Found-by: cppcheck
One was reported by dcb in <https://bugs.dragonflybsd.org/issues/3078>.

sys/dev/netif/wi/if_wi.c
sys/kern/kern_sensors.c
sys/netinet6/nd6_nbr.c
sys/vfs/nfs/nfs_syscalls.c

index a03eb4f..e0e4ad8 100644 (file)
@@ -1087,7 +1087,7 @@ wi_raw_xmit(struct ieee80211_node *ni, struct mbuf *m0,
        if (params && (params->ibp_flags & IEEE80211_BPF_NOACK))
                frmhdr.wi_tx_ctl |= htole16(WI_TXCNTL_ALTRTRY);
        if ((wh->i_fc[1] & IEEE80211_FC1_PROTECTED) &&
-           (!params || (params && (params->ibp_flags & IEEE80211_BPF_CRYPTO)))) {
+           (!params || (params->ibp_flags & IEEE80211_BPF_CRYPTO))) {
                k = ieee80211_crypto_encap(ni, m0);
                if (k == NULL) {
                        rc = ENOMEM;
index 9fd5f4e..86a1108 100644 (file)
@@ -126,8 +126,8 @@ sensor_attach(struct ksensordev *sensdev, struct ksensor *sens)
        } else {
                for (v = SLIST_FIRST(sh);
                    (nv = SLIST_NEXT(v, list)) != NULL; v = nv)
-                       if (v->type == sens->type && (v->type != nv->type || 
-                           (v->type == nv->type && nv->numt - v->numt > 1)))
+                       if (v->type == sens->type &&
+                           (v->type != nv->type || nv->numt - v->numt > 1))
                                break;
                /* sensors of the same type go after each other */
                if (v->type == sens->type)
index 0373aa7..1778709 100644 (file)
@@ -287,9 +287,7 @@ nd6_ns_input(struct mbuf *m, int off, int icmp6len)
                        rtifp = rtifp->if_bridge;
 
                if (rt != NULL &&
-                   (cmpifp != rtifp ||
-                    (cmpifp == rtifp && (m->m_flags & M_MCAST) == 0))
-               ) {
+                   (cmpifp != rtifp || (m->m_flags & M_MCAST) == 0)) {
                        ifa = (struct ifaddr *)in6ifa_ifpforlinklocal(cmpifp,
                                IN6_IFF_NOTREADY|IN6_IFF_ANYCAST);
                        nd6log((LOG_INFO,
@@ -784,7 +782,7 @@ nd6_na_input(struct mbuf *m, int off, int icmp6len)
                        }
                        goto freeit;
                } else if (is_override                             /* (2a) */
-                       || (!is_override && (lladdr && !llchange)) /* (2b) */
+                       || (lladdr && !llchange)                   /* (2b) */
                        || !lladdr) {                              /* (2c) */
                        /*
                         * Update link-local address, if any.
index 3ffd97c..3afddd9 100644 (file)
@@ -1130,8 +1130,7 @@ nfs_getnickauth(struct nfsmount *nmp, struct ucred *cred, char **auth_str,
        verfp = (u_int32_t *)verf_str;
        *verfp++ = txdr_unsigned(RPCAKN_NICKNAME);
        if (time_second != nuidp->nu_timestamp.tv_sec ||
-           (time_second == nuidp->nu_timestamp.tv_sec &&
-            time_second > nuidp->nu_timestamp.tv_usec))        /* XXX */
+           time_second > nuidp->nu_timestamp.tv_usec)          /* XXX */
                getmicrotime(&nuidp->nu_timestamp);
        else
                nuidp->nu_timestamp.tv_usec++;