Cosmetic cleanups.
authorJeffrey Hsu <hsu@dragonflybsd.org>
Thu, 6 Jan 2005 09:14:13 +0000 (09:14 +0000)
committerJeffrey Hsu <hsu@dragonflybsd.org>
Thu, 6 Jan 2005 09:14:13 +0000 (09:14 +0000)
33 files changed:
sys/net/bpf.c
sys/net/faith/if_faith.c
sys/net/if.c
sys/net/if_arcsubr.c
sys/net/if_atmsubr.c
sys/net/if_ethersubr.c
sys/net/if_fddisubr.c
sys/net/if_iso88025subr.c
sys/net/radix.c
sys/net/raw_usrreq.c
sys/net/route.c
sys/net/route.h
sys/net/rtsock.c
sys/netinet/if_ether.c
sys/netinet/in.c
sys/netinet/in_cksum.c
sys/netinet/in_gif.c
sys/netinet/in_pcb.c
sys/netinet/ip_divert.c
sys/netinet/ip_encap.c
sys/netinet/ip_flow.c
sys/netinet/ip_icmp.c
sys/netinet/tcp_debug.c
sys/netinet/tcp_subr.c
sys/netinet/tcp_usrreq.c
sys/netinet/udp_usrreq.c
sys/netinet6/icmp6.c
sys/netinet6/in6.c
sys/netinet6/in6_gif.c
sys/netinet6/in6_ifattach.c
sys/netinet6/in6_rmx.c
sys/netinet6/nd6_nbr.c
sys/sys/socket.h

index f7e3105..54fb53c 100644 (file)
@@ -38,7 +38,7 @@
  *      @(#)bpf.c      8.2 (Berkeley) 3/28/94
  *
  * $FreeBSD: src/sys/net/bpf.c,v 1.59.2.12 2002/04/14 21:41:48 luigi Exp $
- * $DragonFly: src/sys/net/bpf.c,v 1.20 2004/12/21 02:54:14 hsu Exp $
+ * $DragonFly: src/sys/net/bpf.c,v 1.21 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "use_bpf.h"
@@ -308,7 +308,7 @@ bpfopen(dev_t dev, int flags, int fmt, struct thread *td)
        if (d != NULL)
                return(EBUSY);
        make_dev(&bpf_cdevsw, minor(dev), 0, 0, 0600, "bpf%d", lminor(dev));
-       MALLOC(d, struct bpf_d *, sizeof(*d), M_BPF, M_WAITOK | M_ZERO);
+       MALLOC(d, struct bpf_d *, sizeof *d, M_BPF, M_WAITOK | M_ZERO);
        dev->si_drv1 = d;
        d->bd_bufsize = bpf_bufsize;
        d->bd_sig = SIGIO;
@@ -690,14 +690,14 @@ bpfioctl(dev_t dev, u_long cmd, caddr_t addr, int flags, struct thread *td)
         * Get interface name.
         */
        case BIOCGETIF:
-               if (d->bd_bif == NULL)
+               if (d->bd_bif == NULL) {
                        error = EINVAL;
-               else {
+               else {
                        struct ifnet *const ifp = d->bd_bif->bif_ifp;
                        struct ifreq *const ifr = (struct ifreq *)addr;
 
                        strlcpy(ifr->ifr_name, ifp->if_xname,
-                           sizeof(ifr->ifr_name));
+                               sizeof ifr->ifr_name);
                }
                break;
 
@@ -863,7 +863,7 @@ bpf_setf(struct bpf_d *d, struct bpf_program *fp)
        if (flen > BPF_MAXINSNS)
                return(EINVAL);
 
-       size = flen * sizeof(*fp->bf_insns);
+       size = flen * sizeof *fp->bf_insns;
        fcode = (struct bpf_insn *)malloc(size, M_BPF, M_WAITOK);
        if (copyin(fp->bf_insns, fcode, size) == 0 &&
            bpf_validate(fcode, (int)flen)) {
@@ -1181,7 +1181,7 @@ bpfattach(struct ifnet *ifp, u_int dlt, u_int hdrlen)
 {
        struct bpf_if *bp;
 
-       bp = malloc(sizeof(*bp), M_BPF, M_WAITOK | M_ZERO);
+       bp = malloc(sizeof *bp, M_BPF, M_WAITOK | M_ZERO);
 
        bp->bif_ifp = ifp;
        bp->bif_dlt = dlt;
index 0ee9998..d723c2d 100644 (file)
@@ -33,7 +33,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/net/if_faith.c,v 1.3.2.6 2002/04/28 05:40:25 suz Exp $
- * $DragonFly: src/sys/net/faith/if_faith.c,v 1.9 2004/12/21 02:54:15 hsu Exp $
+ * $DragonFly: src/sys/net/faith/if_faith.c,v 1.10 2005/01/06 09:14:13 hsu Exp $
  */
 /*
  * derived from
@@ -187,7 +187,7 @@ void
 faith_clone_destroy(ifp)
        struct ifnet *ifp;
 {
-       struct faith_softc *sc = (void *) ifp;
+       struct faith_softc *sc = (struct faith_softc *) ifp;
 
        LIST_REMOVE(sc, sc_list);
        bpfdetach(ifp);
@@ -364,17 +364,17 @@ faithprefix(in6)
        if (ip6_keepfaith == 0)
                return 0;
 
-       bzero(&sin6, sizeof(sin6));
+       bzero(&sin6, sizeof sin6);
        sin6.sin6_family = AF_INET6;
        sin6.sin6_len = sizeof(struct sockaddr_in6);
        sin6.sin6_addr = *in6;
        rt = rtlookup((struct sockaddr *)&sin6, 0, 0UL);
-       if (rt && rt->rt_ifp && rt->rt_ifp->if_type == IFT_FAITH &&
+       if (rt != NULL && rt->rt_ifp && rt->rt_ifp->if_type == IFT_FAITH &&
            (rt->rt_ifp->if_flags & IFF_UP))
                ret = 1;
        else
                ret = 0;
-       if (rt)
+       if (rt != NULL)
                RTFREE(rt);
        return ret;
 }
index 3589184..c7ca41c 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)if.c        8.3 (Berkeley) 1/4/94
  * $FreeBSD: src/sys/net/if.c,v 1.185 2004/03/13 02:35:03 brooks Exp $
- * $DragonFly: src/sys/net/if.c,v 1.23 2004/12/21 02:54:14 hsu Exp $
+ * $DragonFly: src/sys/net/if.c,v 1.24 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_compat.h"
@@ -617,22 +617,19 @@ ifa_ifwithaddr(struct sockaddr *addr)
        struct ifnet *ifp;
        struct ifaddr *ifa;
 
-#define        equal(a1, a2) \
-  (bcmp((a1), (a2), ((struct sockaddr *)(a1))->sa_len) == 0)
-
        TAILQ_FOREACH(ifp, &ifnet, if_link)
            TAILQ_FOREACH(ifa, &ifp->if_addrhead, ifa_link) {
                if (ifa->ifa_addr->sa_family != addr->sa_family)
                        continue;
-               if (equal(addr, ifa->ifa_addr))
+               if (sa_equal(addr, ifa->ifa_addr))
                        return (ifa);
                if ((ifp->if_flags & IFF_BROADCAST) && ifa->ifa_broadaddr &&
-                   /* IP6 doesn't have broadcast */
+                   /* IPv6 doesn't have broadcast */
                    ifa->ifa_broadaddr->sa_len != 0 &&
-                   equal(ifa->ifa_broadaddr, addr))
+                   sa_equal(ifa->ifa_broadaddr, addr))
                        return (ifa);
        }
-       return ((struct ifaddr *)0);
+       return ((struct ifaddr *)NULL);
 }
 /*
  * Locate the point to point interface with a given destination address.
@@ -648,10 +645,11 @@ ifa_ifwithdstaddr(struct sockaddr *addr)
                TAILQ_FOREACH(ifa, &ifp->if_addrhead, ifa_link) {
                        if (ifa->ifa_addr->sa_family != addr->sa_family)
                                continue;
-                       if (ifa->ifa_dstaddr && equal(addr, ifa->ifa_dstaddr))
+                       if (ifa->ifa_dstaddr &&
+                           sa_equal(addr, ifa->ifa_dstaddr))
                                return (ifa);
        }
-       return ((struct ifaddr *)0);
+       return ((struct ifaddr *)NULL);
 }
 
 /*
@@ -696,8 +694,8 @@ next:                               continue;
                                 * The trouble is that we don't know the
                                 * netmask for the remote end.
                                 */
-                               if (ifa->ifa_dstaddr != 0 &&
-                                   equal(addr, ifa->ifa_dstaddr))
+                               if (ifa->ifa_dstaddr != NULL &&
+                                   sa_equal(addr, ifa->ifa_dstaddr))
                                        return (ifa);
                        } else {
                                /*
@@ -766,14 +764,15 @@ ifaof_ifpforaddr(struct sockaddr *addr, struct ifnet *ifp)
                        continue;
                if (ifa_maybe == 0)
                        ifa_maybe = ifa;
-               if (ifa->ifa_netmask == 0) {
-                       if (equal(addr, ifa->ifa_addr) ||
-                           (ifa->ifa_dstaddr && equal(addr, ifa->ifa_dstaddr)))
+               if (ifa->ifa_netmask == NULL) {
+                       if (sa_equal(addr, ifa->ifa_addr) ||
+                           (ifa->ifa_dstaddr != NULL &&
+                            sa_equal(addr, ifa->ifa_dstaddr)))
                                return (ifa);
                        continue;
                }
                if (ifp->if_flags & IFF_POINTOPOINT) {
-                       if (equal(addr, ifa->ifa_dstaddr))
+                       if (sa_equal(addr, ifa->ifa_dstaddr))
                                return (ifa);
                } else {
                        cp = addr->sa_data;
@@ -1488,7 +1487,7 @@ if_addmulti(
         * then don't add a new one, just add a reference
         */
        LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
-               if (equal(sa, ifma->ifma_addr)) {
+               if (sa_equal(sa, ifma->ifma_addr)) {
                        ifma->ifma_refcount++;
                        if (retifma)
                                *retifma = ifma;
@@ -1530,7 +1529,7 @@ if_addmulti(
 
        if (llsa != 0) {
                LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
-                       if (equal(ifma->ifma_addr, llsa))
+                       if (sa_equal(ifma->ifma_addr, llsa))
                                break;
                }
                if (ifma) {
@@ -1571,7 +1570,7 @@ if_delmulti(struct ifnet *ifp, struct sockaddr *sa)
        int s;
 
        LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link)
-               if (equal(sa, ifma->ifma_addr))
+               if (sa_equal(sa, ifma->ifma_addr))
                        break;
        if (ifma == 0)
                return ENOENT;
@@ -1609,7 +1608,7 @@ if_delmulti(struct ifnet *ifp, struct sockaddr *sa)
         * in that case.)
         */
        LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link)
-               if (equal(sa, ifma->ifma_addr))
+               if (sa_equal(sa, ifma->ifma_addr))
                        break;
        if (ifma == 0)
                return 0;
@@ -1702,7 +1701,7 @@ ifmaof_ifpforaddr(struct sockaddr *sa, struct ifnet *ifp)
        struct ifmultiaddr *ifma;
 
        LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link)
-               if (equal(ifma->ifma_addr, sa))
+               if (sa_equal(ifma->ifma_addr, sa))
                        break;
 
        return ifma;
index 7a7f882..0bd74af 100644 (file)
@@ -1,6 +1,6 @@
 /*     $NetBSD: if_arcsubr.c,v 1.36 2001/06/14 05:44:23 itojun Exp $   */
 /*     $FreeBSD: src/sys/net/if_arcsubr.c,v 1.1.2.5 2003/02/05 18:42:15 fjoe Exp $ */
-/*     $DragonFly: src/sys/net/Attic/if_arcsubr.c,v 1.11 2004/12/21 02:54:14 hsu Exp $ */
+/*     $DragonFly: src/sys/net/Attic/if_arcsubr.c,v 1.12 2005/01/06 09:14:13 hsu Exp $ */
 
 /*
  * Copyright (c) 1994, 1995 Ignatios Souvatzis
@@ -93,7 +93,7 @@ static int    arc_output(struct ifnet *, struct mbuf *, struct sockaddr *,
 
 #define ARC_LLADDR(ifp)        (*(u_int8_t *)IF_LLADDR(ifp))
 
-#define senderr(e) { error = (e); goto bad;}
+#define gotoerr(e) { error = (e); goto bad;}
 #define SIN(s) ((struct sockaddr_in *)s)
 #define SIPX(s)        ((struct sockaddr_ipx *)s)
 
@@ -177,13 +177,13 @@ arc_output(struct ifnet *ifp, struct mbuf *m, struct sockaddr *dst,
        default:
                printf("%s: can't handle af%d\n", ifp->if_xname,
                    dst->sa_family);
-               senderr(EAFNOSUPPORT);
+               gotoerr(EAFNOSUPPORT);
        }
 
        isphds = arc_isphds(atype);
        M_PREPEND(m, isphds ? ARC_HDRNEWLEN : ARC_HDRLEN, MB_DONTWAIT);
        if (m == NULL)
-               senderr(ENOBUFS);
+               gotoerr(ENOBUFS);
        ah = mtod(m, struct arc_header *);
        ah->arc_type = atype;
        ah->arc_dhost = adst;
@@ -209,14 +209,14 @@ arc_output(struct ifnet *ifp, struct mbuf *m, struct sockaddr *dst,
                bpf_mtap(ifp, m);
 
        if (!IF_HANDOFF(&ifp->if_snd, m, ifp)) {
-               m = 0;
-               senderr(ENOBUFS);
+               m = NULL;
+               gotoerr(ENOBUFS);
        }
 
        return (0);
 
 bad:
-       if (m)
+       if (m != NULL)
                m_freem(m);
        return (error);
 }
@@ -456,12 +456,12 @@ arc_defrag(ifp, m)
                /* if all else fails, it is out of sequence, too */
        }
 outofseq:
-       if (m1) {
+       if (m1 != NULL) {
                m_freem(m1);
                af->af_packet = NULL;
        }
 
-       if (m)
+       if (m != NULL)
                m_freem(m);
 
        log(LOG_INFO,"%s: got out of seq. packet: %s\n",
@@ -498,7 +498,7 @@ arc_input(struct ifnet *ifp, struct mbuf *m)
        int isr;
        u_int8_t atype;
 
-       if ((ifp->if_flags & IFF_UP) == 0) {
+       if (!(ifp->if_flags & IFF_UP)) {
                m_freem(m);
                return;
        }
@@ -513,9 +513,9 @@ arc_input(struct ifnet *ifp, struct mbuf *m)
 
        ah = mtod(m, struct arc_header *);
        /* does this belong to us? */
-       if ((ifp->if_flags & IFF_PROMISC) == 0
-           && ah->arc_dhost != ifp->if_broadcastaddr[0]
-           && ah->arc_dhost != ARC_LLADDR(ifp)) {
+       if (!(ifp->if_flags & IFF_PROMISC) &&
+           ah->arc_dhost != ifp->if_broadcastaddr[0] &&
+           ah->arc_dhost != ARC_LLADDR(ifp)) {
                m_freem(m);
                return;
        }
index 9460b7b..0db7d5a 100644 (file)
@@ -32,7 +32,7 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/net/if_atmsubr.c,v 1.10.2.1 2001/03/06 00:29:26 obrien Exp $
- * $DragonFly: src/sys/net/if_atmsubr.c,v 1.10 2004/07/23 07:16:30 joerg Exp $
+ * $DragonFly: src/sys/net/if_atmsubr.c,v 1.11 2005/01/06 09:14:13 hsu Exp $
  */
 
 /*
@@ -72,7 +72,7 @@
 #define ETHERTYPE_IPV6 0x86dd
 #endif
 
-#define senderr(e) { error = (e); goto bad;}
+#define gotoerr(e) { error = (e); goto bad;}
 
 /*
  * atm_output: ATM output routine
@@ -103,28 +103,28 @@ atm_output(struct ifnet *ifp, struct mbuf *m0, struct sockaddr *dst,
        u_int32_t atm_flags;
 
        if ((ifp->if_flags & (IFF_UP|IFF_RUNNING)) != (IFF_UP|IFF_RUNNING))
-               senderr(ENETDOWN);
+               gotoerr(ENETDOWN);
 
        /*
         * check route
         */
        if ((rt = rt0) != NULL) {
 
-               if ((rt->rt_flags & RTF_UP) == 0) { /* route went down! */
+               if (!(rt->rt_flags & RTF_UP)) { /* route went down! */
                        if ((rt0 = rt = RTALLOC1(dst, 0)) != NULL)
                                rt->rt_refcnt--;
                        else 
-                               senderr(EHOSTUNREACH);
+                               gotoerr(EHOSTUNREACH);
                }
 
                if (rt->rt_flags & RTF_GATEWAY) {
-                       if (rt->rt_gwroute == 0)
+                       if (rt->rt_gwroute == NULL)
                                goto lookup;
-                       if (((rt = rt->rt_gwroute)->rt_flags & RTF_UP) == 0) {
+                       if (((rt = rt->rt_gwroute)->rt_flags & RTF_UP) == NULL) {
                                rtfree(rt); rt = rt0;
                        lookup: rt->rt_gwroute = RTALLOC1(rt->rt_gateway, 0);
-                               if ((rt = rt->rt_gwroute) == 0)
-                                       senderr(EHOSTUNREACH);
+                               if ((rt = rt->rt_gwroute) == NULL)
+                                       gotoerr(EHOSTUNREACH);
                        }
                }
 
@@ -147,7 +147,7 @@ atm_output(struct ifnet *ifp, struct mbuf *m0, struct sockaddr *dst,
                        if (!atmresolve(rt, m, dst, &atmdst)) {
                                m = NULL; 
                                /* XXX: atmresolve already free'd it */
-                               senderr(EHOSTUNREACH);
+                               gotoerr(EHOSTUNREACH);
                                /* XXX: put ATMARP stuff here */
                                /* XXX: watch who frees m on failure */
                        }
@@ -171,7 +171,7 @@ atm_output(struct ifnet *ifp, struct mbuf *m0, struct sockaddr *dst,
                        printf("%s: can't handle af%d\n", ifp->if_xname, 
                            dst->sa_family);
 #endif
-                       senderr(EAFNOSUPPORT);
+                       gotoerr(EAFNOSUPPORT);
                }
 
                /*
@@ -181,8 +181,8 @@ atm_output(struct ifnet *ifp, struct mbuf *m0, struct sockaddr *dst,
                atm_flags = ATM_PH_FLAGS(&atmdst);
                if (atm_flags & ATM_PH_LLCSNAP) sz += 8; /* sizeof snap == 8 */
                M_PREPEND(m, sz, MB_DONTWAIT);
-               if (m == 0)
-                       senderr(ENOBUFS);
+               if (m == NULL)
+                       gotoerr(ENOBUFS);
                ad = mtod(m, struct atm_pseudohdr *);
                *ad = atmdst;
                if (atm_flags & ATM_PH_LLCSNAP) {
@@ -206,17 +206,17 @@ atm_output(struct ifnet *ifp, struct mbuf *m0, struct sockaddr *dst,
        if (IF_QFULL(&ifp->if_snd)) {
                IF_DROP(&ifp->if_snd);
                splx(s);
-               senderr(ENOBUFS);
+               gotoerr(ENOBUFS);
        }
        ifp->if_obytes += m->m_pkthdr.len;
        IF_ENQUEUE(&ifp->if_snd, m);
-       if ((ifp->if_flags & IFF_OACTIVE) == 0)
+       if (!(ifp->if_flags & IFF_OACTIVE))
                (*ifp->if_start)(ifp);
        splx(s);
        return (error);
 
 bad:
-       if (m)
+       if (m != NULL)
                m_freem(m);
        return (error);
 }
@@ -236,7 +236,7 @@ atm_input(ifp, ah, m, rxhand)
        int isr;
        int s;
 
-       if ((ifp->if_flags & IFF_UP) == 0) {
+       if (!(ifp->if_flags & IFF_UP)) {
                m_freem(m);
                return;
        }
@@ -262,7 +262,7 @@ atm_input(ifp, ah, m, rxhand)
                if (ATM_PH_FLAGS(ah) & ATM_PH_LLCSNAP) {
                        struct atmllc *alc;
                        if (m->m_len < sizeof(*alc) &&
-                           (m = m_pullup(m, sizeof(*alc))) == 0)
+                           (m = m_pullup(m, sizeof(*alc))) == NULL)
                                return; /* failed */
                        alc = mtod(m, struct atmllc *);
                        if (bcmp(alc, ATMLLC_HDR, 6)) {
index df184ea..4a32d94 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)if_ethersubr.c      8.1 (Berkeley) 6/10/93
  * $FreeBSD: src/sys/net/if_ethersubr.c,v 1.70.2.33 2003/04/28 15:45:53 archie Exp $
- * $DragonFly: src/sys/net/if_ethersubr.c,v 1.24 2004/12/28 08:09:59 hsu Exp $
+ * $DragonFly: src/sys/net/if_ethersubr.c,v 1.25 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_atalk.h"
@@ -338,17 +338,18 @@ ether_output(struct ifnet *ifp, struct mbuf *m, struct sockaddr *dst,
 
        /* Handle ng_ether(4) processing, if any */
        if (ng_ether_output_p != NULL) {
-               if ((error = (*ng_ether_output_p)(ifp, &m)) != 0) {
-bad:                   if (m != NULL)
-                               m_freem(m);
-                       return (error);
-               }
+               if ((error = (*ng_ether_output_p)(ifp, &m)) != 0)
+                       goto bad;
                if (m == NULL)
                        return (0);
        }
 
        /* Continue with link-layer output */
        return ether_output_frame(ifp, m);
+
+bad:
+       m_freem(m);
+       return (error);
 }
 
 /*
@@ -382,8 +383,7 @@ ether_output_frame(struct ifnet *ifp, struct mbuf *m)
                eh = mtod(m, struct ether_header *);
                m_adj(m, ETHER_HDR_LEN);
                m = bdg_forward_ptr(m, eh, ifp);
-               if (m != NULL)
-                       m_freem(m);
+               m_freem(m);
                return (0);
        }
 
@@ -576,7 +576,7 @@ ether_input(struct ifnet *ifp, struct ether_header *eh, struct mbuf *m)
                bpf_mtap(ifp, (struct mbuf *)&mh);
        }
 
-       ifp->if_ibytes += m->m_pkthdr.len + sizeof (*eh);
+       ifp->if_ibytes += m->m_pkthdr.len + (sizeof *eh);
 
        /* Handle ng_ether(4) processing, if any */
        if (ng_ether_input_p != NULL) {
@@ -613,8 +613,7 @@ ether_input(struct ifnet *ifp, struct ether_header *eh, struct mbuf *m)
                        goto recvLocal;         /* receive locally */
 
                /* If not local and not multicast, just drop it */
-               if (m != NULL)
-                       m_freem(m);
+               m_freem(m);
                return;
        }
 
@@ -678,8 +677,7 @@ ether_demux(struct ifnet *ifp, struct ether_header *eh, struct mbuf *m)
 post_stats:
        if (IPFW_LOADED && ether_ipfw != 0) {
                if (!ether_ipfw_chk(&m, NULL, &rule, eh, FALSE)) {
-                       if (m != NULL)
-                               m_freem(m);
+                       m_freem(m);
                        return;
                }
                eh = mtod(m, struct ether_header *);
@@ -771,14 +769,14 @@ post_stats:
                    l->llc_ssap == LLC_SNAP_LSAP &&
                    l->llc_control == LLC_UI) {
                        if (bcmp(&(l->llc_snap_org_code)[0], at_org_code,
-                           sizeof(at_org_code)) == 0 &&
+                                sizeof at_org_code) == 0 &&
                            ntohs(l->llc_snap_ether_type) == ETHERTYPE_AT) {
                                m_adj(m, sizeof(struct llc));
                                isr = NETISR_ATALK2;
                                break;
                        }
                        if (bcmp(&(l->llc_snap_org_code)[0], aarp_org_code,
-                           sizeof(aarp_org_code)) == 0 &&
+                                sizeof aarp_org_code) == 0 &&
                            ntohs(l->llc_snap_ether_type) == ETHERTYPE_AARP) {
                                m_adj(m, sizeof(struct llc));
                                isr = NETISR_AARP;
index 88866f7..3a6f250 100644 (file)
@@ -34,7 +34,7 @@
  *
  *     from: if_ethersubr.c,v 1.5 1994/12/13 22:31:45 wollman Exp
  * $FreeBSD: src/sys/net/if_fddisubr.c,v 1.41.2.8 2002/02/20 23:34:09 fjoe Exp $
- * $DragonFly: src/sys/net/Attic/if_fddisubr.c,v 1.13 2004/12/21 02:54:14 hsu Exp $
+ * $DragonFly: src/sys/net/Attic/if_fddisubr.c,v 1.14 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_atalk.h"
@@ -352,8 +352,7 @@ queue_it:
        return (0);
 
 bad:
-       if (m)
-               m_freem(m);
+       m_freem(m);
        return (error);
 }
 
@@ -377,7 +376,7 @@ fddi_input(struct ifnet *ifp, struct mbuf *m)
        m_adj(m, sizeof(struct fddi_header));
        m->m_pkthdr.rcvif = ifp;
 
-       if ((ifp->if_flags & IFF_UP) == 0) {
+       if (!(ifp->if_flags & IFF_UP)) {
                m_freem(m);
                return;
        }
index d83d82f..002f75a 100644 (file)
@@ -31,7 +31,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/net/if_iso88025subr.c,v 1.7.2.7 2002/06/18 00:15:31 kbyanc Exp $
- * $DragonFly: src/sys/net/Attic/if_iso88025subr.c,v 1.10 2004/12/21 02:54:14 hsu Exp $
+ * $DragonFly: src/sys/net/Attic/if_iso88025subr.c,v 1.11 2005/01/06 09:14:13 hsu Exp $
  *
  */
 
@@ -305,8 +305,7 @@ iso88025_output(struct ifnet *ifp, struct mbuf *m, struct sockaddr *dst,
        return (error);
 
 bad:
-       if (m)
-               m_freem(m);
+       m_freem(m);
        return (error);
 }
 
index 7ae4100..fe8abea 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)radix.c     8.4 (Berkeley) 11/2/94
  * $FreeBSD: src/sys/net/radix.c,v 1.20.2.3 2002/04/28 05:40:25 suz Exp $
- * $DragonFly: src/sys/net/radix.c,v 1.9 2004/12/28 08:09:59 hsu Exp $
+ * $DragonFly: src/sys/net/radix.c,v 1.10 2005/01/06 09:14:13 hsu Exp $
  */
 
 /*
@@ -1032,6 +1032,7 @@ rn_inithead(void **head, int off)
        right->rn_key = rn_ones;
 
        rnh->rnh_treetop = root;
+
        rnh->rnh_addaddr = rn_addroute;
        rnh->rnh_deladdr = rn_delete;
        rnh->rnh_matchaddr = rn_match;
index dd3717f..d150a8d 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)raw_usrreq.c        8.1 (Berkeley) 6/10/93
  * $FreeBSD: src/sys/net/raw_usrreq.c,v 1.18 1999/08/28 00:48:28 peter Exp $
- * $DragonFly: src/sys/net/raw_usrreq.c,v 1.8 2004/12/21 02:54:14 hsu Exp $
+ * $DragonFly: src/sys/net/raw_usrreq.c,v 1.9 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include <sys/param.h>
@@ -89,11 +89,9 @@ raw_input(m0, proto, src, dst)
                 * Note that if the lengths are not the same
                 * the comparison will fail at the first byte.
                 */
-#define        equal(a1, a2) \
-  (bcmp((caddr_t)(a1), (caddr_t)(a2), a1->sa_len) == 0)
-               if (rp->rcb_laddr && !equal(rp->rcb_laddr, dst))
+               if (rp->rcb_laddr && !sa_equal(rp->rcb_laddr, dst))
                        continue;
-               if (rp->rcb_faddr && !equal(rp->rcb_faddr, src))
+               if (rp->rcb_faddr && !sa_equal(rp->rcb_faddr, src))
                        continue;
                if (last) {
                        struct mbuf *n;
index 99286d7..1e76df9 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)route.c     8.3 (Berkeley) 1/9/95
  * $FreeBSD: src/sys/net/route.c,v 1.59.2.10 2003/01/17 08:04:00 ru Exp $
- * $DragonFly: src/sys/net/route.c,v 1.11 2004/12/28 08:09:59 hsu Exp $
+ * $DragonFly: src/sys/net/route.c,v 1.12 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_inet.h"
@@ -201,8 +201,6 @@ rtfree(struct rtentry *rt)
        }
 }
 
-#define        sa_equal(a1, a2) (bcmp((a1), (a2), (a1)->sa_len) == 0)
-
 /*
  * Force a routing table entry to the specified
  * destination to go through the given gateway.
@@ -210,7 +208,6 @@ rtfree(struct rtentry *rt)
  * message from the network layer.
  *
  * N.B.: must be called at splnet
- *
  */
 void
 rtredirect(
@@ -271,7 +268,7 @@ create:
                        if (rt != NULL)
                                rtfree(rt);
                        flags |=  RTF_GATEWAY | RTF_DYNAMIC;
-                       bzero(&info, sizeof(info));
+                       bzero(&info, sizeof info);
                        info.rti_info[RTAX_DST] = dst;
                        info.rti_info[RTAX_GATEWAY] = gateway;
                        info.rti_info[RTAX_NETMASK] = netmask;
@@ -312,7 +309,7 @@ out:
        else if (stat != NULL)
                (*stat)++;
 
-       bzero(&info, sizeof(info));
+       bzero(&info, sizeof info);
        info.rti_info[RTAX_DST] = dst;
        info.rti_info[RTAX_GATEWAY] = gateway;
        info.rti_info[RTAX_NETMASK] = netmask;
@@ -877,7 +874,7 @@ rt_setgate(struct rtentry *rt0, struct sockaddr *dst, struct sockaddr *gate)
                 * a gateway.  This is obviously mandatory when we
                 * get rt->rt_output().
                 *
-                * This breaks TTCP!  XXX JH
+                * This breaks TTCP for hosts outside the gateway!  XXX JH
                 */
                rt->rt_gwroute = rtlookup(gate, 1, RTF_PRCLONING);
                if (rt->rt_gwroute == rt) {
@@ -1029,7 +1026,7 @@ rtinit(struct ifaddr *ifa, int cmd, int flags)
        /*
         * Do the actual request
         */
-       bzero(&info, sizeof(info));
+       bzero(&info, sizeof info);
        info.rti_ifa = ifa;
        info.rti_flags = flags | ifa->ifa_flags;
        info.rti_info[RTAX_DST] = dst;
index d821f54..8bb70b4 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)route.h     8.4 (Berkeley) 1/9/95
  * $FreeBSD: src/sys/net/route.h,v 1.36.2.5 2002/02/01 11:48:01 ru Exp $
- * $DragonFly: src/sys/net/route.h,v 1.8 2005/01/05 18:21:20 joerg Exp $
+ * $DragonFly: src/sys/net/route.h,v 1.9 2005/01/06 09:14:13 hsu Exp $
  */
 
 #ifndef _NET_ROUTE_H_
@@ -259,6 +259,7 @@ struct rt_addrinfo {
 };
 
 #ifdef _KERNEL
+
 #define        sa_dst          rti_info[RTAX_DST]
 #define        sa_gateway      rti_info[RTAX_GATEWAY]
 #define        sa_netmask      rti_info[RTAX_NETMASK]
index 2778f91..0afdc4c 100644 (file)
  *
  *     @(#)rtsock.c    8.7 (Berkeley) 10/12/95
  * $FreeBSD: src/sys/net/rtsock.c,v 1.44.2.11 2002/12/04 14:05:41 ru Exp $
- * $DragonFly: src/sys/net/rtsock.c,v 1.18 2004/12/28 08:09:59 hsu Exp $
+ * $DragonFly: src/sys/net/rtsock.c,v 1.19 2005/01/06 09:14:13 hsu Exp $
  */
 
-
 #include <sys/param.h>
 #include <sys/systm.h>
 #include <sys/kernel.h>
@@ -439,8 +438,7 @@ route_output(struct mbuf *m, struct socket *so, ...)
                             info.sa_gateway != NULL) ||
                            info.sa_ifpaddr != NULL ||
                            (info.sa_ifaaddr != NULL &&
-                            bcmp(info.sa_ifaaddr, rt->rt_ifa->ifa_addr,
-                                 info.sa_ifaaddr->sa_len) == 0)) {
+                            sa_equal(info.sa_ifaaddr, rt->rt_ifa->ifa_addr))) {
                                if ((error = rt_getifa(&info)) != 0)
                                        gotoerr(error);
                        }
@@ -561,7 +559,7 @@ rt_xaddrs(char *cp, char *cplim, struct rt_addrinfo *rtinfo)
                /*
                 * It won't fit.
                 */
-               if ( (cp + sa->sa_len) > cplim ) {
+               if ((cp + sa->sa_len) > cplim) {
                        return (EINVAL);
                }
 
@@ -585,9 +583,7 @@ rt_xaddrs(char *cp, char *cplim, struct rt_addrinfo *rtinfo)
 }
 
 static struct mbuf *
-rt_msg1(type, rtinfo)
-       int type;
-       struct rt_addrinfo *rtinfo;
+rt_msg1(int type, struct rt_addrinfo *rtinfo)
 {
        struct rt_msghdr *rtm;
        struct mbuf *m;
@@ -732,7 +728,7 @@ again:
 void
 rt_missmsg(int type, struct rt_addrinfo *rtinfo, int flags, int error)
 {
-       struct sockaddr *sa = rtinfo->rti_info[RTAX_DST];
+       struct sockaddr *dst = rtinfo->rti_info[RTAX_DST];
        struct rt_msghdr *rtm;
        struct mbuf *m;
 
@@ -745,7 +741,7 @@ rt_missmsg(int type, struct rt_addrinfo *rtinfo, int flags, int error)
        rtm->rtm_flags = RTF_DONE | flags;
        rtm->rtm_errno = error;
        rtm->rtm_addrs = rtinfo->rti_addrs;
-       route_proto.sp_protocol = sa ? sa->sa_family : 0;
+       route_proto.sp_protocol = (dst != NULL) ? dst->sa_family : 0;
        raw_input(m, &route_proto, &route_src, &route_dst);
 }
 
@@ -754,8 +750,7 @@ rt_missmsg(int type, struct rt_addrinfo *rtinfo, int flags, int error)
  * socket indicating that the status of a network interface has changed.
  */
 void
-rt_ifmsg(ifp)
-       struct ifnet *ifp;
+rt_ifmsg(struct ifnet *ifp)
 {
        struct if_msghdr *ifm;
        struct mbuf *m;
@@ -847,10 +842,7 @@ rt_rtmsg(int cmd, struct ifaddr *ifa, int error, struct rtentry *rt)
  * copies of it.
  */
 void
-rt_newaddrmsg(cmd, ifa, error, rt)
-       int cmd, error;
-       struct ifaddr *ifa;
-       struct rtentry *rt;
+rt_newaddrmsg(int cmd, struct ifaddr *ifa, int error, struct rtentry *rt)
 {
        if (route_cb.any_count == 0)
                return;
@@ -871,9 +863,7 @@ rt_newaddrmsg(cmd, ifa, error, rt)
  * there is no route state to worry about.
  */
 void
-rt_newmaddrmsg(cmd, ifma)
-       int cmd;
-       struct ifmultiaddr *ifma;
+rt_newmaddrmsg(int cmd, struct ifmultiaddr *ifma)
 {
        struct rt_addrinfo info;
        struct mbuf *m = NULL;
index e396f15..5d15b7d 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)if_ether.c  8.1 (Berkeley) 6/10/93
  * $FreeBSD: src/sys/netinet/if_ether.c,v 1.64.2.23 2003/04/11 07:23:15 fjoe Exp $
- * $DragonFly: src/sys/netinet/if_ether.c,v 1.22 2004/12/28 08:09:59 hsu Exp $
+ * $DragonFly: src/sys/netinet/if_ether.c,v 1.23 2005/01/06 09:14:13 hsu Exp $
  */
 
 /*
@@ -184,7 +184,7 @@ arp_rtrequest(int req, struct rtentry *rt, struct rt_addrinfo *info)
                         * Case 1: This route should come from a route to iface.
                         */
                        rt_setgate(rt, rt_key(rt),
-                           (struct sockaddr *)&null_sdl);
+                                  (struct sockaddr *)&null_sdl);
                        gate = rt->rt_gateway;
                        SDL(gate)->sdl_type = rt->rt_ifp->if_type;
                        SDL(gate)->sdl_index = rt->rt_ifp->if_index;
@@ -212,7 +212,7 @@ arp_rtrequest(int req, struct rtentry *rt, struct rt_addrinfo *info)
                 * Case 2:  This route may come from cloning, or a manual route
                 * add with a LL address.
                 */
-               R_Malloc(la, struct llinfo_arp *, sizeof(*la));
+               R_Malloc(la, struct llinfo_arp *, sizeof *la);
                rt->rt_llinfo = la;
                if (la == NULL) {
                        log(LOG_DEBUG, "arp_rtrequest: malloc failed\n");
@@ -321,19 +321,19 @@ arprequest(struct ifnet *ifp, struct in_addr *sip, struct in_addr *tip,
        case IFT_ISO88025:
                ar_hrd = htons(ARPHRD_IEEE802);
 
-               m->m_len = sizeof(llcx) +
+               m->m_len = (sizeof llcx) +
                    arphdr_len2(ifp->if_addrlen, sizeof(struct in_addr));
                m->m_pkthdr.len = m->m_len;
                MH_ALIGN(m, m->m_len);
 
-               memcpy(mtod(m, caddr_t), llcx, sizeof(llcx));
+               memcpy(mtod(m, caddr_t), llcx, sizeof llcx);
                memcpy(sa.sa_data, ifp->if_broadcastaddr, ifp->if_addrlen);
                memcpy(sa.sa_data + 6, enaddr, 6);
                sa.sa_data[6] |= TR_RII;
                sa.sa_data[12] = TR_AC;
                sa.sa_data[13] = TR_LLC_FRAME;
 
-               ah = (struct arphdr *)(mtod(m, char *) + sizeof(llcx));
+               ah = (struct arphdr *)(mtod(m, char *) + sizeof llcx);
                break;
        case IFT_FDDI:
        case IFT_ETHER:
@@ -368,7 +368,7 @@ arprequest(struct ifnet *ifp, struct in_addr *sip, struct in_addr *tip,
        memcpy(ar_tpa(ah), tip, ah->ar_pln);
 
        sa.sa_family = AF_UNSPEC;
-       sa.sa_len = sizeof(sa);
+       sa.sa_len = sizeof sa;
        (*ifp->if_output)(ifp, m, &sa, (struct rtentry *)NULL);
 }
 
@@ -548,7 +548,6 @@ out2:
  * but formerly didn't normally send requests.
  */
 static int log_arp_wrong_iface = 1;
-
 SYSCTL_INT(_net_link_ether_inet, OID_AUTO, log_arp_wrong_iface, CTLFLAG_RW,
        &log_arp_wrong_iface, 0,
        "log arp packets arriving on the wrong interface");
@@ -757,7 +756,7 @@ reply:
                        sin.sin_addr = itaddr;
 
                        rt = rtlookup((struct sockaddr *)&sin, 0, 0UL);
-                       if (!rt) {
+                       if (rt == NULL) {
                                m_freem(m);
                                return;
                        }
@@ -775,8 +774,7 @@ reply:
                        memcpy(ar_sha(ah), IF_LLADDR(ifp), ah->ar_hln);
                        --rt->rt_refcnt;
 #ifdef DEBUG_PROXY
-                       printf("arp: proxying for %s\n",
-                              inet_ntoa(itaddr));
+                       printf("arp: proxying for %s\n", inet_ntoa(itaddr));
 #endif
                } else {
                        rt = la->la_rt;
@@ -800,9 +798,9 @@ reply:
        case IFT_ISO88025:
                /* Re-arrange the source/dest address */
                memcpy(th->iso88025_dhost, th->iso88025_shost,
-                   sizeof(th->iso88025_dhost));
+                   sizeof th->iso88025_dhost);
                memcpy(th->iso88025_shost, IF_LLADDR(ifp),
-                   sizeof(th->iso88025_shost));
+                   sizeof th->iso88025_shost);
                /* Set the source routing bit if neccesary */
                if (th->iso88025_dhost[0] & TR_RII) {
                        th->iso88025_dhost[0] &= ~TR_RII;
@@ -811,9 +809,9 @@ reply:
                }
                /* Copy the addresses, ac and fc into sa_data */
                memcpy(sa.sa_data, th->iso88025_dhost,
-                   sizeof(th->iso88025_dhost) * 2);
-               sa.sa_data[(sizeof(th->iso88025_dhost) * 2)] = TR_AC;
-               sa.sa_data[(sizeof(th->iso88025_dhost) * 2) + 1] = TR_LLC_FRAME;
+                   (sizeof th->iso88025_dhost) * 2);
+               sa.sa_data[(sizeof th->iso88025_dhost) * 2] = TR_AC;
+               sa.sa_data[(sizeof th->iso88025_dhost) * 2 + 1] = TR_LLC_FRAME;
                break;
        case IFT_ETHER:
        case IFT_FDDI:
@@ -823,12 +821,12 @@ reply:
         */
        default:
                eh = (struct ether_header *)sa.sa_data;
-               memcpy(eh->ether_dhost, ar_tha(ah), sizeof(eh->ether_dhost));
+               memcpy(eh->ether_dhost, ar_tha(ah), sizeof eh->ether_dhost);
                eh->ether_type = htons(ETHERTYPE_ARP);
                break;
        }
        sa.sa_family = AF_UNSPEC;
-       sa.sa_len = sizeof(sa);
+       sa.sa_len = sizeof sa;
        (*ifp->if_output)(ifp, m, &sa, (struct rtentry *)0);
        return;
 }
@@ -861,6 +859,7 @@ arptfree(struct llinfo_arp *la)
        }
        rtrequest(RTM_DELETE, rt_key(rt), NULL, rt_mask(rt), 0, NULL);
 }
+
 /*
  * Lookup or enter a new address in arptab.
  */
index 9f02c92..66669ab 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)in.c        8.4 (Berkeley) 1/9/95
  * $FreeBSD: src/sys/netinet/in.c,v 1.44.2.14 2002/11/08 00:45:50 suz Exp $
- * $DragonFly: src/sys/netinet/in.c,v 1.13 2004/12/21 02:54:15 hsu Exp $
+ * $DragonFly: src/sys/netinet/in.c,v 1.14 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_bootp.h"
@@ -148,12 +148,12 @@ in_mask2len(mask)
        u_char *p;
 
        p = (u_char *)mask;
-       for (x = 0; x < sizeof(*mask); x++) {
+       for (x = 0; x < sizeof *mask; x++) {
                if (p[x] != 0xff)
                        break;
        }
        y = 0;
-       if (x < sizeof(*mask)) {
+       if (x < sizeof *mask) {
                for (y = 0; y < 8; y++) {
                        if ((p[x] & (0x80 >> y)) == 0)
                                break;
@@ -171,7 +171,7 @@ in_len2mask(mask, len)
        u_char *p;
 
        p = (u_char *)mask;
-       bzero(mask, sizeof(*mask));
+       bzero(mask, sizeof *mask);
        for (i = 0; i < len / 8; i++)
                p[i] = 0xff;
        if (len % 8)
@@ -279,7 +279,7 @@ in_control(so, cmd, data, ifp, td)
                                malloc(sizeof *ia, M_IFADDR, M_WAITOK);
                        if (ia == (struct in_ifaddr *)NULL)
                                return (ENOBUFS);
-                       bzero((caddr_t)ia, sizeof *ia);
+                       bzero(ia, sizeof *ia);
                        /*
                         * Protect from ipintr() traversing address list
                         * while we're modifying it.
@@ -296,7 +296,7 @@ in_control(so, cmd, data, ifp, td)
                        ia->ia_sockmask.sin_len = 8;
                        ia->ia_sockmask.sin_family = AF_INET;
                        if (ifp->if_flags & IFF_BROADCAST) {
-                               ia->ia_broadaddr.sin_len = sizeof(ia->ia_addr);
+                               ia->ia_broadaddr.sin_len = sizeof ia->ia_addr;
                                ia->ia_broadaddr.sin_family = AF_INET;
                        }
                        ia->ia_ifp = ifp;
@@ -537,9 +537,8 @@ in_lifaddr_ioctl(so, cmd, data, ifp, td)
                        return EINVAL;
 
                /* copy args to in_aliasreq, perform ioctl(SIOCAIFADDR_IN6). */
-               bzero(&ifra, sizeof(ifra));
-               bcopy(iflr->iflr_name, ifra.ifra_name,
-                       sizeof(ifra.ifra_name));
+               bzero(&ifra, sizeof ifra);
+               bcopy(iflr->iflr_name, ifra.ifra_name, sizeof ifra.ifra_name);
 
                bcopy(&iflr->addr, &ifra.ifra_addr, iflr->addr.ss_len);
 
@@ -562,7 +561,7 @@ in_lifaddr_ioctl(so, cmd, data, ifp, td)
                struct sockaddr_in *sin;
                int cmp;
 
-               bzero(&mask, sizeof(mask));
+               bzero(&mask, sizeof mask);
                if (iflr->flags & IFLR_PREFIX) {
                        /* lookup a prefix rather than address. */
                        in_len2mask(&mask, iflr->prefixlen);
@@ -612,7 +611,7 @@ in_lifaddr_ioctl(so, cmd, data, ifp, td)
                                bcopy(&ia->ia_dstaddr, &iflr->dstaddr,
                                        ia->ia_dstaddr.sin_len);
                        } else
-                               bzero(&iflr->dstaddr, sizeof(iflr->dstaddr));
+                               bzero(&iflr->dstaddr, sizeof iflr->dstaddr);
 
                        iflr->prefixlen =
                                in_mask2len(&ia->ia_sockmask.sin_addr);
@@ -624,9 +623,9 @@ in_lifaddr_ioctl(so, cmd, data, ifp, td)
                        struct in_aliasreq ifra;
 
                        /* fill in_aliasreq and do ioctl(SIOCDIFADDR_IN6) */
-                       bzero(&ifra, sizeof(ifra));
+                       bzero(&ifra, sizeof ifra);
                        bcopy(iflr->iflr_name, ifra.ifra_name,
-                               sizeof(ifra.ifra_name));
+                               sizeof ifra.ifra_name);
 
                        bcopy(&ia->ia_addr, &ifra.ifra_addr,
                                ia->ia_addr.sin_len);
@@ -856,7 +855,7 @@ in_addmulti(ap, ifp)
 
        /* XXX - if_addmulti uses M_WAITOK.  Can this really be called
           at interrupt time?  If so, need to fix if_addmulti. XXX */
-       inm = malloc(sizeof(*inm), M_IPMADDR, M_WAITOK | M_ZERO);
+       inm = malloc(sizeof *inm, M_IPMADDR, M_WAITOK | M_ZERO);
        inm->inm_addr = *ap;
        inm->inm_ifp = ifp;
        inm->inm_ifma = ifma;
index 837f56f..f7e44cf 100644 (file)
@@ -31,7 +31,7 @@
  * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  *
- * $DragonFly: src/sys/netinet/in_cksum.c,v 1.8 2004/12/21 02:54:15 hsu Exp $
+ * $DragonFly: src/sys/netinet/in_cksum.c,v 1.9 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include <sys/param.h>
@@ -89,7 +89,7 @@ in_cksum_range(struct mbuf *m, int nxt, int offset, int bytes)
                panic("in_cksum_range: offset too short");
        if (m->m_len < sizeof(struct ip))
                panic("in_cksum_range: bad mbuf chain");
-       bzero(&ipov, sizeof(ipov));
+       bzero(&ipov, sizeof ipov);
        ipov.ih_len = htons(bytes);
        ipov.ih_pr = nxt;
        ipov.ih_src = mtod(m, struct ip *)->ip_src;
index 07b1985..8a6bd09 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * $FreeBSD: src/sys/netinet/in_gif.c,v 1.5.2.11 2003/01/23 21:06:45 sam Exp $
- * $DragonFly: src/sys/netinet/in_gif.c,v 1.11 2004/12/21 02:54:15 hsu Exp $
+ * $DragonFly: src/sys/netinet/in_gif.c,v 1.12 2005/01/06 09:14:13 hsu Exp $
  * $KAME: in_gif.c,v 1.54 2001/05/14 14:02:16 itojun Exp $
  */
 /*
@@ -87,10 +87,7 @@ SYSCTL_INT(_net_inet_ip, IPCTL_GIF_TTL, gifttl, CTLFLAG_RW,
        &ip_gif_ttl,    0, "");
 
 int
-in_gif_output(ifp, family, m)
-       struct ifnet    *ifp;
-       int             family;
-       struct mbuf     *m;
+in_gif_output(struct ifnet *ifp, int family, struct mbuf *m)
 {
        struct gif_softc *sc = (struct gif_softc*)ifp;
        struct sockaddr_in *dst = (struct sockaddr_in *)&sc->gif_ro.ro_dst;
@@ -114,8 +111,8 @@ in_gif_output(ifp, family, m)
                struct ip *ip;
 
                proto = IPPROTO_IPV4;
-               if (m->m_len < sizeof(*ip)) {
-                       m = m_pullup(m, sizeof(*ip));
+               if (m->m_len < sizeof *ip) {
+                       m = m_pullup(m, sizeof *ip);
                        if (!m)
                                return ENOBUFS;
                }
@@ -129,8 +126,8 @@ in_gif_output(ifp, family, m)
            {
                struct ip6_hdr *ip6;
                proto = IPPROTO_IPV6;
-               if (m->m_len < sizeof(*ip6)) {
-                       m = m_pullup(m, sizeof(*ip6));
+               if (m->m_len < sizeof *ip6) {
+                       m = m_pullup(m, sizeof *ip6);
                        if (!m)
                                return ENOBUFS;
                }
@@ -148,7 +145,7 @@ in_gif_output(ifp, family, m)
                return EAFNOSUPPORT;
        }
 
-       bzero(&iphdr, sizeof(iphdr));
+       bzero(&iphdr, sizeof iphdr);
        iphdr.ip_src = sin_src->sin_addr;
        /* bidirectional configured tunnel mode */
        if (sin_dst->sin_addr.s_addr != INADDR_ANY)
@@ -204,8 +201,8 @@ in_gif_output(ifp, family, m)
                        return ENETUNREACH;     /* XXX */
                }
 #if 0
-               ifp->if_mtu = sc->gif_ro.ro_rt->rt_ifp->if_mtu
-                       - sizeof(struct ip);
+               ifp->if_mtu = sc->gif_ro.ro_rt->rt_ifp->if_mtu -
+                   sizeof(struct ip);
 #endif
        }
 
@@ -247,8 +244,8 @@ in_gif_input(struct mbuf *m, ...)
            {
                struct ip *ip;
                af = AF_INET;
-               if (m->m_len < sizeof(*ip)) {
-                       m = m_pullup(m, sizeof(*ip));
+               if (m->m_len < sizeof *ip) {
+                       m = m_pullup(m, sizeof *ip);
                        if (!m)
                                return;
                }
@@ -266,8 +263,8 @@ in_gif_input(struct mbuf *m, ...)
                struct ip6_hdr *ip6;
                u_int8_t itos;
                af = AF_INET6;
-               if (m->m_len < sizeof(*ip6)) {
-                       m = m_pullup(m, sizeof(*ip6));
+               if (m->m_len < sizeof *ip6) {
+                       m = m_pullup(m, sizeof *ip6);
                        if (!m)
                                return;
                }
@@ -295,10 +292,7 @@ in_gif_input(struct mbuf *m, ...)
  * validate outer address.
  */
 static int
-gif_validate4(ip, sc, ifp)
-       const struct ip *ip;
-       struct gif_softc *sc;
-       struct ifnet *ifp;
+gif_validate4(const struct ip *ip, struct gif_softc *sc, struct ifnet *ifp)
 {
        struct sockaddr_in *src, *dst;
        struct in_ifaddr *ia4;
@@ -333,7 +327,7 @@ gif_validate4(ip, sc, ifp)
                struct sockaddr_in sin;
                struct rtentry *rt;
 
-               bzero(&sin, sizeof(sin));
+               bzero(&sin, sizeof sin);
                sin.sin_family = AF_INET;
                sin.sin_len = sizeof(struct sockaddr_in);
                sin.sin_addr = ip->ip_src;
@@ -359,11 +353,7 @@ gif_validate4(ip, sc, ifp)
  * matched the physical addr family.  see gif_encapcheck().
  */
 int
-gif_encapcheck4(m, off, proto, arg)
-       const struct mbuf *m;
-       int off;
-       int proto;
-       void *arg;
+gif_encapcheck4(const struct mbuf *m, int off, int proto, void *arg)
 {
        struct ip ip;
        struct gif_softc *sc;
@@ -373,15 +363,14 @@ gif_encapcheck4(m, off, proto, arg)
        sc = (struct gif_softc *)arg;
 
        /* LINTED const cast */
-       m_copydata(__DECONST(struct mbuf *, m), 0, sizeof(ip), (caddr_t)&ip);
+       m_copydata(__DECONST(struct mbuf *, m), 0, sizeof ip, (caddr_t)&ip);
        ifp = ((m->m_flags & M_PKTHDR) != 0) ? m->m_pkthdr.rcvif : NULL;
 
        return gif_validate4(&ip, sc, ifp);
 }
 
 int
-in_gif_attach(sc)
-       struct gif_softc *sc;
+in_gif_attach(struct gif_softc *sc)
 {
        sc->encap_cookie4 = encap_attach_func(AF_INET, -1, gif_encapcheck,
            &in_gif_protosw, sc);
@@ -391,8 +380,7 @@ in_gif_attach(sc)
 }
 
 int
-in_gif_detach(sc)
-       struct gif_softc *sc;
+in_gif_detach(struct gif_softc *sc)
 {
        int error;
 
index ac4992c..fe32ec1 100644 (file)
@@ -82,7 +82,7 @@
  *
  *     @(#)in_pcb.c    8.4 (Berkeley) 5/24/95
  * $FreeBSD: src/sys/netinet/in_pcb.c,v 1.59.2.27 2004/01/02 04:06:42 ambrisko Exp $
- * $DragonFly: src/sys/netinet/in_pcb.c,v 1.31 2004/12/30 05:14:30 hsu Exp $
+ * $DragonFly: src/sys/netinet/in_pcb.c,v 1.32 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_ipsec.h"
@@ -724,8 +724,7 @@ in_pcbnotifyall(head, faddr, errno, notify)
         * the structure and skip it.
         */
        s = splnet();
-       for (inp = LIST_FIRST(head); inp != NULL; inp = ninp) {
-               ninp = LIST_NEXT(inp, inp_list);
+       LIST_FOREACH_MUTABLE(inp, head, inp_list, ninp) {
                if (inp->inp_flags & INP_PLACEMARKER)
                        continue;
 #ifdef INET6
@@ -735,7 +734,7 @@ in_pcbnotifyall(head, faddr, errno, notify)
                if (inp->inp_faddr.s_addr != faddr.s_addr ||
                    inp->inp_socket == NULL)
                        continue;
-               (*notify)(inp, errno);
+               (*notify)(inp, errno);          /* can remove inp from list! */
        }
        splx(s);
 }
@@ -1214,8 +1213,8 @@ in_pcblist_global(SYSCTL_HANDLER_ARGS)
        }
        LIST_REMOVE(marker, inp_list);
        if (error == 0 && i < n) {
-               bzero(&xi, sizeof(xi));
-               xi.xi_len = sizeof(xi);
+               bzero(&xi, sizeof xi);
+               xi.xi_len = sizeof xi;
                while (i < n) {
                        error = SYSCTL_OUT(req, &xi, sizeof xi);
                        ++i;
index 6ca9a6a..ccf5eb3 100644 (file)
@@ -31,7 +31,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/netinet/ip_divert.c,v 1.42.2.6 2003/01/23 21:06:45 sam Exp $
- * $DragonFly: src/sys/netinet/ip_divert.c,v 1.17 2004/12/28 08:09:59 hsu Exp $
+ * $DragonFly: src/sys/netinet/ip_divert.c,v 1.18 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_inet.h"
@@ -211,7 +211,7 @@ divert_packet(struct mbuf *m, int incoming, int port, int rule)
                 * (see div_output for the other half of this.)
                 */
                snprintf(divsrc.sin_zero, sizeof divsrc.sin_zero,
-                           m->m_pkthdr.rcvif->if_xname);
+                        m->m_pkthdr.rcvif->if_xname);
        }
 
        /* Put packet on socket queue, if any */
@@ -224,8 +224,8 @@ divert_packet(struct mbuf *m, int incoming, int port, int rule)
                        sa = inp->inp_socket;
        }
        if (sa) {
-               if (sbappendaddr(&sa->so_rcv, (struct sockaddr *)&divsrc,
-                               m, (struct mbuf *)NULL) == 0)
+               if (sbappendaddr(&sa->so_rcv, (struct sockaddr *)&divsrc, m,
+                                (struct mbuf *)NULL) == 0)
                        m_freem(m);
                else
                        sorwakeup(sa);
index 2cec801..d14beef 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet/ip_encap.c,v 1.1.2.5 2003/01/23 21:06:45 sam Exp $    */
-/*     $DragonFly: src/sys/netinet/ip_encap.c,v 1.9 2004/11/30 19:21:26 joerg Exp $    */
+/*     $DragonFly: src/sys/netinet/ip_encap.c,v 1.10 2005/01/06 09:14:13 hsu Exp $     */
 /*     $KAME: ip_encap.c,v 1.41 2001/03/15 08:35:08 itojun Exp $       */
 
 /*
@@ -146,11 +146,11 @@ encap4_input(struct mbuf *m, ...)
 
        ip = mtod(m, struct ip *);
 
-       bzero(&s, sizeof(s));
+       bzero(&s, sizeof s);
        s.sin_family = AF_INET;
        s.sin_len = sizeof(struct sockaddr_in);
        s.sin_addr = ip->ip_src;
-       bzero(&d, sizeof(d));
+       bzero(&d, sizeof d);
        d.sin_family = AF_INET;
        d.sin_len = sizeof(struct sockaddr_in);
        d.sin_addr = ip->ip_dst;
@@ -237,11 +237,11 @@ encap6_input(mp, offp, proto)
 
        ip6 = mtod(m, struct ip6_hdr *);
 
-       bzero(&s, sizeof(s));
+       bzero(&s, sizeof s);
        s.sin6_family = AF_INET6;
        s.sin6_len = sizeof(struct sockaddr_in6);
        s.sin6_addr = ip6->ip6_src;
-       bzero(&d, sizeof(d));
+       bzero(&d, sizeof d);
        d.sin6_family = AF_INET6;
        d.sin6_len = sizeof(struct sockaddr_in6);
        d.sin6_addr = ip6->ip6_dst;
@@ -318,7 +318,7 @@ encap_attach(af, proto, sp, sm, dp, dm, psw, arg)
 
        s = splnet();
        /* sanity check on args */
-       if (sp->sa_len > sizeof(ep->src) || dp->sa_len > sizeof(ep->dst)) {
+       if (sp->sa_len > sizeof ep->src || dp->sa_len > sizeof ep->dst) {
                error = EINVAL;
                goto fail;
        }
@@ -350,7 +350,7 @@ encap_attach(af, proto, sp, sm, dp, dm, psw, arg)
                goto fail;
        }
 
-       ep = malloc(sizeof(*ep), M_NETADDR, M_INTWAIT | M_ZERO | M_NULLOK);
+       ep = malloc(sizeof *ep, M_NETADDR, M_INTWAIT | M_ZERO | M_NULLOK);
        if (ep == NULL) {
                error = ENOBUFS;
                goto fail;
@@ -395,7 +395,7 @@ encap_attach_func(af, proto, func, psw, arg)
                goto fail;
        }
 
-       ep = malloc(sizeof(*ep), M_NETADDR, M_INTWAIT | M_ZERO | M_NULLOK);
+       ep = malloc(sizeof *ep, M_NETADDR, M_INTWAIT | M_ZERO | M_NULLOK);
        if (ep == NULL) {
                error = ENOBUFS;
                goto fail;
@@ -449,7 +449,7 @@ mask_match(ep, sp, dp)
        u_int8_t *r;
        int matchlen;
 
-       if (sp->sa_len > sizeof(s) || dp->sa_len > sizeof(d))
+       if (sp->sa_len > sizeof s || dp->sa_len > sizeof d)
                return 0;
        if (sp->sa_family != ep->af || dp->sa_family != ep->af)
                return 0;
@@ -496,9 +496,9 @@ encap_fillarg(m, ep)
 {
        struct m_tag *tag;
 
-       tag = m_tag_get(PACKET_TAG_ENCAP, sizeof (void*), MB_DONTWAIT);
-       if (tag) {
-               *(void**)(tag+1) = ep->arg;
+       tag = m_tag_get(PACKET_TAG_ENCAP, sizeof(void *), MB_DONTWAIT);
+       if (tag != NULL) {
+               *(void **)(tag + 1) = ep->arg;
                m_tag_prepend(m, tag);
        }
 }
@@ -511,8 +511,8 @@ encap_getarg(m)
        struct m_tag *tag;
 
        tag = m_tag_find(m, PACKET_TAG_ENCAP, NULL);
-       if (tag) {
-               p = *(void**)(tag+1);
+       if (tag != NULL) {
+               p = (void *)(tag + 1);
                m_tag_delete(m, tag);
        }
        return p;
index 7b5ad02..ad6c15c 100644 (file)
@@ -34,7 +34,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/netinet/ip_flow.c,v 1.9.2.2 2001/11/04 17:35:31 luigi Exp $
- * $DragonFly: src/sys/netinet/ip_flow.c,v 1.5 2004/12/21 02:54:15 hsu Exp $
+ * $DragonFly: src/sys/netinet/ip_flow.c,v 1.6 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include <sys/param.h>
@@ -299,13 +299,13 @@ ipflow_create(const struct route *ro, struct mbuf *m)
                if (ipflow_inuse == IPFLOW_MAX) {
                        ipf = ipflow_reap();
                } else {
-                       ipf = malloc(sizeof(*ipf), M_IPFLOW,
+                       ipf = malloc(sizeof *ipf, M_IPFLOW,
                                     M_INTWAIT | M_NULLOK);
                        if (ipf == NULL)
                                return;
                        ipflow_inuse++;
                }
-               bzero((caddr_t) ipf, sizeof(*ipf));
+               bzero(ipf, sizeof *ipf);
        } else {
                s = splimp();
                LIST_REMOVE(ipf, ipf_next);
index 47f5b4f..eb13888 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)ip_icmp.c   8.2 (Berkeley) 1/4/94
  * $FreeBSD: src/sys/netinet/ip_icmp.c,v 1.39.2.19 2003/01/24 05:11:34 sam Exp $
- * $DragonFly: src/sys/netinet/ip_icmp.c,v 1.17 2004/12/29 03:26:42 hsu Exp $
+ * $DragonFly: src/sys/netinet/ip_icmp.c,v 1.18 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_ipsec.h"
@@ -424,7 +424,7 @@ icmp_input(struct mbuf *m, ...)
 
                        rt = rtlookup((struct sockaddr *)&icmpsrc, 0,
                                      RTF_CLONING | RTF_PRCLONING);
-                       if (rt && (rt->rt_flags & RTF_HOST) &&
+                       if (rt != NULL && (rt->rt_flags & RTF_HOST) &&
                            !(rt->rt_rmx.rmx_locks & RTV_MTU)) {
                                mtu = ntohs(icp->icmp_nextmtu);
                                if (!mtu)
@@ -442,7 +442,7 @@ icmp_input(struct mbuf *m, ...)
                                        rt->rt_rmx.rmx_mtu = mtu;
                                }
                        }
-                       if (rt)
+                       if (rt != NULL)
                                --rt->rt_refcnt;
                }
 
index 29b14b9..735acdc 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)tcp_debug.c 8.1 (Berkeley) 6/10/93
  * $FreeBSD: src/sys/netinet/tcp_debug.c,v 1.16.2.1 2000/07/15 07:14:31 kris Exp $
- * $DragonFly: src/sys/netinet/tcp_debug.c,v 1.2 2003/06/17 04:28:51 dillon Exp $
+ * $DragonFly: src/sys/netinet/tcp_debug.c,v 1.3 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_inet.h"
@@ -112,54 +112,48 @@ tcp_trace(act, ostate, tp, ipgen, th, req)
        if (tp)
                td->td_cb = *tp;
        else
-               bzero((caddr_t)&td->td_cb, sizeof (*tp));
+               bzero(&td->td_cb, sizeof *tp);
        if (ipgen) {
                switch (td->td_family) {
                case AF_INET:
-                       bcopy((caddr_t)ipgen, (caddr_t)&td->td_ti.ti_i,
-                             sizeof(td->td_ti.ti_i));
-                       bzero((caddr_t)td->td_ip6buf, sizeof(td->td_ip6buf));
+                       bcopy(ipgen, &td->td_ti.ti_i, sizeof(td->td_ti.ti_i));
+                       bzero(td->td_ip6buf, sizeof td->td_ip6buf);
                        break;
 #ifdef INET6
                case AF_INET6:
-                       bcopy((caddr_t)ipgen, (caddr_t)td->td_ip6buf,
-                             sizeof(td->td_ip6buf));
-                       bzero((caddr_t)&td->td_ti.ti_i,
-                             sizeof(td->td_ti.ti_i));
+                       bcopy(ipgen, td->td_ip6buf, sizeof td->td_ip6buf);
+                       bzero(&td->td_ti.ti_i, sizeof td->td_ti.ti_i);
                        break;
 #endif
                default:
-                       bzero((caddr_t)td->td_ip6buf, sizeof(td->td_ip6buf));
-                       bzero((caddr_t)&td->td_ti.ti_i,
-                             sizeof(td->td_ti.ti_i));
+                       bzero(td->td_ip6buf, sizeof td->td_ip6buf);
+                       bzero(&td->td_ti.ti_i, sizeof td->td_ti.ti_i);
                        break;
                }
        } else {
-               bzero((caddr_t)&td->td_ti.ti_i, sizeof(td->td_ti.ti_i));
-               bzero((caddr_t)td->td_ip6buf, sizeof(td->td_ip6buf));
+               bzero(&td->td_ti.ti_i, sizeof td->td_ti.ti_i);
+               bzero(td->td_ip6buf, sizeof td->td_ip6buf);
        }
        if (th) {
                switch (td->td_family) {
                case AF_INET:
                        td->td_ti.ti_t = *th;
-                       bzero((caddr_t)&td->td_ti6.th, sizeof(td->td_ti6.th));
+                       bzero(&td->td_ti6.th, sizeof td->td_ti6.th);
                        break;
 #ifdef INET6
                case AF_INET6:
                        td->td_ti6.th = *th;
-                       bzero((caddr_t)&td->td_ti.ti_t,
-                             sizeof(td->td_ti.ti_t));
+                       bzero(&td->td_ti.ti_t, sizeoftd->td_ti.ti_t);
                        break;
 #endif
                default:
-                       bzero((caddr_t)&td->td_ti.ti_t,
-                             sizeof(td->td_ti.ti_t));
-                       bzero((caddr_t)&td->td_ti6.th, sizeof(td->td_ti6.th));
+                       bzero(&td->td_ti.ti_t, sizeof td->td_ti.ti_t);
+                       bzero(&td->td_ti6.th, sizeof td->td_ti6.th);
                        break;
                }
        } else {
-               bzero((caddr_t)&td->td_ti.ti_t, sizeof(td->td_ti.ti_t));
-               bzero((caddr_t)&td->td_ti6.th, sizeof(td->td_ti6.th));
+               bzero(&td->td_ti.ti_t, sizeof td->td_ti.ti_t);
+               bzero(&td->td_ti6.th, sizeof td->td_ti6.th);
        }
        td->td_req = req;
 #ifdef TCPDEBUG
@@ -190,7 +184,7 @@ tcp_trace(act, ostate, tp, ipgen, th, req)
                        len = ntohs((u_short)len);
                }
                if (act == TA_OUTPUT)
-                       len -= sizeof (struct tcphdr);
+                       len -= sizeof(struct tcphdr);
                if (len)
                        printf("[%x..%x)", seq, seq+len);
                else
index f9ae62c..00740ed 100644 (file)
@@ -82,7 +82,7 @@
  *
  *     @(#)tcp_subr.c  8.2 (Berkeley) 5/24/95
  * $FreeBSD: src/sys/netinet/tcp_subr.c,v 1.73.2.31 2003/01/24 05:11:34 sam Exp $
- * $DragonFly: src/sys/netinet/tcp_subr.c,v 1.43 2004/12/21 02:54:15 hsu Exp $
+ * $DragonFly: src/sys/netinet/tcp_subr.c,v 1.44 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_compat.h"
@@ -1178,8 +1178,8 @@ tcp_pcblist(SYSCTL_HANDLER_ARGS)
                }
                LIST_REMOVE(marker, inp_list);
                if (error == 0 && i < n) {
-                       bzero(&xt, sizeof(xt));
-                       xt.xt_len = sizeof(xt);
+                       bzero(&xt, sizeof xt);
+                       xt.xt_len = sizeof xt;
                        while (i < n) {
                                error = SYSCTL_OUT(req, &xt, sizeof xt);
                                if (error)
@@ -1317,7 +1317,7 @@ tcp_ctlinput(int cmd, struct sockaddr *sa, void *vip)
                notify = in_rtchange;
        } else if (cmd == PRC_HOSTDEAD)
                ip = NULL;
-       else if ((unsigned)cmd > PRC_NCMDS || inetctlerrmap[cmd] == 0)
+       else if ((unsigned)cmd >= PRC_NCMDS || inetctlerrmap[cmd] == 0)
                return;
        if (ip != NULL) {
                s = splnet();
index 412f367..be3a519 100644 (file)
@@ -82,7 +82,7 @@
  *
  *     From: @(#)tcp_usrreq.c  8.2 (Berkeley) 1/3/94
  * $FreeBSD: src/sys/netinet/tcp_usrreq.c,v 1.51.2.17 2002/10/11 11:46:44 ume Exp $
- * $DragonFly: src/sys/netinet/tcp_usrreq.c,v 1.31 2004/12/21 02:54:15 hsu Exp $
+ * $DragonFly: src/sys/netinet/tcp_usrreq.c,v 1.32 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_ipsec.h"
@@ -881,7 +881,7 @@ tcp_connect_oncpu(struct tcpcb *tp, struct sockaddr_in *sin,
         */
        if ((taop = tcp_gettaocache(&tp->t_inpcb->inp_inc)) == NULL) {
                taop = &tao_noncached;
-               bzero(taop, sizeof(*taop));
+               bzero(taop, sizeof *taop);
        }
 
        tp->cc_send = CC_INC(tcp_ccgen);
@@ -1043,7 +1043,7 @@ tcp6_connect(struct tcpcb *tp, struct sockaddr *nam, struct thread *td)
         */
        if ((taop = tcp_gettaocache(&tp->t_inpcb->inp_inc)) == NULL) {
                taop = &tao_noncached;
-               bzero(taop, sizeof(*taop));
+               bzero(taop, sizeof *taop);
        }
 
        tp->cc_send = CC_INC(tcp_ccgen);
index b2547a0..9b987b6 100644 (file)
@@ -82,7 +82,7 @@
  *
  *     @(#)udp_usrreq.c        8.6 (Berkeley) 5/23/95
  * $FreeBSD: src/sys/netinet/udp_usrreq.c,v 1.64.2.18 2003/01/24 05:11:34 sam Exp $
- * $DragonFly: src/sys/netinet/udp_usrreq.c,v 1.31 2004/12/29 03:26:42 hsu Exp $
+ * $DragonFly: src/sys/netinet/udp_usrreq.c,v 1.32 2005/01/06 09:14:13 hsu Exp $
  */
 
 #include "opt_ipsec.h"
@@ -215,8 +215,7 @@ udp_init()
  *
  * Returns 0 if the packet is acceptable, -1 if it is not.
  */
-static __inline
-int
+static __inline int
 check_multicast_membership(struct ip *ip, struct inpcb *inp, struct mbuf *m)
 {
        int mshipno;
@@ -224,18 +223,20 @@ check_multicast_membership(struct ip *ip, struct inpcb *inp, struct mbuf *m)
 
        if (strict_mcast_mship == 0 ||
            !IN_MULTICAST(ntohl(ip->ip_dst.s_addr))) {
-               return(0);
+               return (0);
        }
        mopt = inp->inp_moptions;
        if (mopt == NULL)
-               return(-1);
+               return (-1);
        for (mshipno = 0; mshipno <= mopt->imo_num_memberships; ++mshipno) {
-               if (ip->ip_dst.s_addr == mopt->imo_membership[mshipno]->inm_addr.s_addr &&
-                   m->m_pkthdr.rcvif == mopt->imo_membership[mshipno]->inm_ifp) {
-                       return(0);
+               struct in_multi *maddr = mopt->imo_membership[mshipno];
+
+               if (ip->ip_dst.s_addr == maddr->inm_addr.s_addr &&
+                   m->m_pkthdr.rcvif == maddr->inm_ifp) {
+                       return (0);
                }
        }
-       return(-1);
+       return (-1);
 }
 
 void
index 54598f8..7ec4653 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/icmp6.c,v 1.6.2.13 2003/05/06 06:46:58 suz Exp $     */
-/*     $DragonFly: src/sys/netinet6/icmp6.c,v 1.14 2004/12/21 02:54:47 hsu Exp $       */
+/*     $DragonFly: src/sys/netinet6/icmp6.c,v 1.15 2005/01/06 09:14:13 hsu Exp $       */
 /*     $KAME: icmp6.c,v 1.211 2001/04/04 05:56:20 itojun Exp $ */
 
 /*
@@ -1142,7 +1142,7 @@ icmp6_mtudisc_update(struct ip6ctlparam *ip6cp, int validated)
        /* sin6.sin6_scope_id = XXX: should be set if DST is a scoped addr */
        rt = rtlookup((struct sockaddr *)&sin6, 0, RTF_CLONING | RTF_PRCLONING);
 
-       if (rt && (rt->rt_flags & RTF_HOST) &&
+       if (rt != NULL && (rt->rt_flags & RTF_HOST) &&
            !(rt->rt_rmx.rmx_locks & RTV_MTU)) {
                if (mtu < IPV6_MMTU) {                          /* XXX */
                        rt->rt_rmx.rmx_locks |= RTV_MTU;
@@ -2255,7 +2255,7 @@ icmp6_redirect_input(struct mbuf *m, int off)
        sin6.sin6_len = sizeof(struct sockaddr_in6);
        bcopy(&reddst6, &sin6.sin6_addr, sizeof reddst6);
        rt = rtlookup((struct sockaddr *)&sin6, 0, 0UL);
-       if (rt) {
+       if (rt != NULL) {
                if (rt->rt_gateway == NULL ||
                    rt->rt_gateway->sa_family != AF_INET6) {
                        nd6log((LOG_ERR,
index 7a756c8..a76654f 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/in6.c,v 1.7.2.9 2002/04/28 05:40:26 suz Exp $        */
-/*     $DragonFly: src/sys/netinet6/in6.c,v 1.10 2004/12/21 02:54:47 hsu Exp $ */
+/*     $DragonFly: src/sys/netinet6/in6.c,v 1.11 2005/01/06 09:14:13 hsu Exp $ */
 /*     $KAME: in6.c,v 1.259 2002/01/21 11:37:50 keiichi Exp $  */
 
 /*
@@ -227,7 +227,7 @@ in6_ifaddloop(struct ifaddr *ifa)
        if (rt == NULL || !(rt->rt_flags & RTF_HOST) ||
            !(rt->rt_ifp->if_flags & IFF_LOOPBACK))
                in6_ifloop_request(RTM_ADD, ifa);
-       if (rt)
+       if (rt != NULL)
                rt->rt_refcnt--;
 }
 
index 365b8ac..69ccc16 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/in6_gif.c,v 1.2.2.7 2003/01/23 21:06:47 sam Exp $    */
-/*     $DragonFly: src/sys/netinet6/in6_gif.c,v 1.9 2004/12/21 02:54:47 hsu Exp $      */
+/*     $DragonFly: src/sys/netinet6/in6_gif.c,v 1.10 2005/01/06 09:14:13 hsu Exp $     */
 /*     $KAME: in6_gif.c,v 1.49 2001/05/14 14:02:17 itojun Exp $        */
 
 /*
@@ -328,13 +328,13 @@ gif_validate6(const struct ip6_hdr *ip6, struct gif_softc *sc,
 #endif
 
                rt = rtlookup((struct sockaddr *)&sin6, 0, 0UL);
-               if (!rt || rt->rt_ifp != ifp) {
+               if (rt == NULL || rt->rt_ifp != ifp) {
 #if 0
                        log(LOG_WARNING, "%s: packet from %s dropped "
                            "due to ingress filter\n", if_name(&sc->gif_if),
                            ip6_sprintf(&sin6.sin6_addr));
 #endif
-                       if (rt)
+                       if (rt != NULL)
                                --rt->rt_refcnt;
                        return 0;
                }
index 48b4e25..4fe2900 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/in6_ifattach.c,v 1.2.2.6 2002/04/28 05:40:26 suz Exp $       */
-/*     $DragonFly: src/sys/netinet6/in6_ifattach.c,v 1.9 2004/12/21 02:54:47 hsu Exp $ */
+/*     $DragonFly: src/sys/netinet6/in6_ifattach.c,v 1.10 2005/01/06 09:14:13 hsu Exp $        */
 /*     $KAME: in6_ifattach.c,v 1.118 2001/05/24 07:44:00 itojun Exp $  */
 
 /*
@@ -955,7 +955,7 @@ in6_ifdetach(struct ifnet *ifp)
        sin6.sin6_addr = in6addr_linklocal_allnodes;
        sin6.sin6_addr.s6_addr16[1] = htons(ifp->if_index);
        rt = rtlookup((struct sockaddr *)&sin6, 0, 0UL);
-       if (rt && rt->rt_ifp == ifp) {
+       if (rt != NULL && rt->rt_ifp == ifp) {
                --rt->rt_refcnt;
                rtrequest(RTM_DELETE, (struct sockaddr *)rt_key(rt),
                        rt->rt_gateway, rt_mask(rt), rt->rt_flags, 0);
index 24fb8b7..04517bb 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/in6_rmx.c,v 1.1.2.4 2004/10/06 02:35:17 suz Exp $    */
-/*     $DragonFly: src/sys/netinet6/in6_rmx.c,v 1.9 2005/01/01 09:19:40 hsu Exp $      */
+/*     $DragonFly: src/sys/netinet6/in6_rmx.c,v 1.10 2005/01/06 09:14:13 hsu Exp $     */
 /*     $KAME: in6_rmx.c,v 1.11 2001/07/26 06:53:16 jinmei Exp $        */
 
 /*
@@ -168,15 +168,13 @@ in6_addroute(char *key, char *mask, struct radix_node_head *head,
                if (rt2 != NULL) {
                        --rt2->rt_refcnt;
                        if (rt2->rt_flags & RTF_LLINFO &&
-                               rt2->rt_flags & RTF_HOST &&
-                               rt2->rt_gateway &&
-                               rt2->rt_gateway->sa_family == AF_LINK) {
-                               rtrequest(RTM_DELETE,
-                                         (struct sockaddr *)rt_key(rt2),
-                                         rt2->rt_gateway,
-                                         rt_mask(rt2), rt2->rt_flags, 0);
-                               ret = rn_addroute(key, mask, head,
-                                       treenodes);
+                           rt2->rt_flags & RTF_HOST &&
+                           rt2->rt_gateway &&
+                           rt2->rt_gateway->sa_family == AF_LINK) {
+                               rtrequest(RTM_DELETE, rt_key(rt2),
+                                         rt2->rt_gateway, rt_mask(rt2),
+                                         rt2->rt_flags, NULL);
+                               ret = rn_addroute(key, mask, head, treenodes);
                        }
                }
        } else if (ret == NULL && rt->rt_flags & RTF_CLONING) {
@@ -263,8 +261,7 @@ in6_clsroute(struct radix_node *rn, struct radix_node_head *head)
        if ((rt->rt_flags & (RTF_LLINFO | RTF_HOST)) != RTF_HOST)
                return;
 
-       if ((rt->rt_flags & (RTF_WASCLONED | RTPRF_OURS))
-          != RTF_WASCLONED)
+       if ((rt->rt_flags & (RTF_WASCLONED | RTPRF_OURS)) != RTF_WASCLONED)
                return;
 
        /*
@@ -314,10 +311,8 @@ in6_rtqkill(struct radix_node *rn, void *rock)
                        if (rt->rt_refcnt > 0)
                                panic("rtqkill route really not free");
 
-                       err = rtrequest(RTM_DELETE,
-                                       (struct sockaddr *)rt_key(rt),
-                                       rt->rt_gateway, rt_mask(rt),
-                                       rt->rt_flags, 0);
+                       err = rtrequest(RTM_DELETE, rt_key(rt), rt->rt_gateway,
+                                       rt_mask(rt), rt->rt_flags, NULL);
                        if (err) {
                                log(LOG_WARNING, "in6_rtqkill: error %d", err);
                        } else {
index 0b1cd38..3fce1d2 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/nd6_nbr.c,v 1.4.2.6 2003/01/23 21:06:47 sam Exp $    */
-/*     $DragonFly: src/sys/netinet6/nd6_nbr.c,v 1.7 2004/12/21 02:54:47 hsu Exp $      */
+/*     $DragonFly: src/sys/netinet6/nd6_nbr.c,v 1.8 2005/01/06 09:14:13 hsu Exp $      */
 /*     $KAME: nd6_nbr.c,v 1.86 2002/01/21 02:33:04 jinmei Exp $        */
 
 /*
@@ -213,7 +213,7 @@ nd6_ns_input(struct mbuf *m, int off, int icmp6len)
                tsin6.sin6_addr = taddr6;
 
                rt = rtlookup((struct sockaddr *)&tsin6, 0, 0);
-               if (rt && (rt->rt_flags & RTF_ANNOUNCE) &&
+               if (rt != NULL && (rt->rt_flags & RTF_ANNOUNCE) &&
                    rt->rt_gateway->sa_family == AF_LINK) {
                        /*
                         * proxy NDP for single entry
@@ -225,10 +225,10 @@ nd6_ns_input(struct mbuf *m, int off, int icmp6len)
                                proxydl = SDL(rt->rt_gateway);
                        }
                }
-               if (rt)
+               if (rt != NULL)
                        --rt->rt_refcnt;
        }
-       if (!ifa) {
+       if (ifa == NULL) {
                /*
                 * We've got an NS packet, and we don't have that adddress
                 * assigned for us.  We MUST silently ignore it.
index 761344f..c727942 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)socket.h    8.4 (Berkeley) 2/21/94
  * $FreeBSD: src/sys/sys/socket.h,v 1.39.2.7 2001/07/03 11:02:01 ume Exp $
- * $DragonFly: src/sys/sys/socket.h,v 1.8 2003/11/15 19:28:42 asmodai Exp $
+ * $DragonFly: src/sys/sys/socket.h,v 1.9 2005/01/06 09:14:13 hsu Exp $
  */
 
 #ifndef _SYS_SOCKET_H_
@@ -172,6 +172,16 @@ struct sockaddr {
 };
 #define        SOCK_MAXADDRLEN 255             /* longest possible addresses */
 
+#ifdef _KERNEL
+#include <sys/libkern.h>               /* for bcmp() */
+
+static __inline boolean_t
+sa_equal(struct sockaddr *a1, struct sockaddr *a2)
+{
+       return (bcmp(a1, a2, a1->sa_len) == 0);
+}
+#endif
+
 /*
  * Structure used by kernel to pass protocol
  * information in raw sockets.