Kernel Police:
authorHiten Pandya <hmp@dragonflybsd.org>
Tue, 29 Jul 2003 12:51:30 +0000 (12:51 +0000)
committerHiten Pandya <hmp@dragonflybsd.org>
Tue, 29 Jul 2003 12:51:30 +0000 (12:51 +0000)
- Fix Mbuf/Malloc flag misuse.
- Remove redundant memset() redifiniton (ng_l2tp.c)

15 files changed:
sys/net/bridge/bridge.c
sys/net/ip6fw/ip6_fw.c
sys/net/ipfw/ip_fw.c
sys/net/oldbridge/bridge.c
sys/net/radix.c
sys/net/radix.h
sys/net/raw_cb.c
sys/netgraph/l2tp/ng_l2tp.c
sys/netgraph/ppp/ng_ppp.c
sys/netgraph/pptpgre/ng_pptpgre.c
sys/netinet/ip_encap.c
sys/netinet6/esp_core.c
sys/netinet6/ip6_input.c
sys/netinet6/ipcomp_output.c
sys/netproto/key/keysock.c

index 2bebafa..6398fc2 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/net/bridge.c,v 1.16.2.25 2003/01/23 21:06:44 sam Exp $
- * $DragonFly: src/sys/net/bridge/Attic/bridge.c,v 1.2 2003/06/17 04:28:47 dillon Exp $
+ * $DragonFly: src/sys/net/bridge/Attic/bridge.c,v 1.3 2003/07/29 12:51:29 hmp Exp $
  */
 
 /*
@@ -243,7 +243,7 @@ add_cluster(u_int16_t cluster_id, struct arpcom *ac)
            goto found;
 
     /* Not found, need to reallocate */
-    c = malloc((1+n_clusters) * sizeof (*c), M_IFADDR, M_DONTWAIT | M_ZERO);
+    c = malloc((1+n_clusters) * sizeof (*c), M_IFADDR, M_NOWAIT | M_ZERO);
     if (c == NULL) {/* malloc failure */
        printf("-- bridge: cannot add new cluster\n");
        return NULL;
index 151b687..12b34f5 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/ip6_fw.c,v 1.2.2.9 2002/04/28 05:40:27 suz Exp $     */
-/*     $DragonFly: src/sys/net/ip6fw/ip6_fw.c,v 1.2 2003/06/17 04:28:52 dillon Exp $   */
+/*     $DragonFly: src/sys/net/ip6fw/ip6_fw.c,v 1.3 2003/07/29 12:51:30 hmp Exp $      */
 /*     $KAME: ip6_fw.c,v 1.21 2001/01/24 01:25:32 itojun Exp $ */
 
 /*
@@ -844,8 +844,8 @@ add_entry6(struct ip6_fw_head *chainptr, struct ip6_fw *frwl)
        u_short nbr = 0;
        int s;
 
-       fwc = malloc(sizeof *fwc, M_IP6FW, M_DONTWAIT);
-       ftmp = malloc(sizeof *ftmp, M_IP6FW, M_DONTWAIT);
+       fwc = malloc(sizeof *fwc, M_IP6FW, M_NOWAIT);
+       ftmp = malloc(sizeof *ftmp, M_IP6FW, M_NOWAIT);
        if (!fwc || !ftmp) {
                dprintf(("%s malloc said no\n", err_prefix));
                if (fwc)  free(fwc, M_IP6FW);
index b4f1466..56f4dba 100644 (file)
@@ -14,7 +14,7 @@
  * This software is provided ``AS IS'' without any warranties of any kind.
  *
  * $FreeBSD: src/sys/netinet/ip_fw.c,v 1.131.2.39 2003/01/20 02:23:07 iedowse Exp $
- * $DragonFly: src/sys/net/ipfw/Attic/ip_fw.c,v 1.3 2003/07/26 21:00:04 rob Exp $
+ * $DragonFly: src/sys/net/ipfw/Attic/ip_fw.c,v 1.4 2003/07/29 12:51:30 hmp Exp $
  */
 
 #define        DEB(x)
@@ -854,7 +854,7 @@ add_dyn_rule(struct ipfw_flow_id *id, u_int8_t dyn_type, struct ip_fw *rule)
     }
     i = hash_packet(id);
 
-    r = malloc(sizeof *r, M_IPFW, M_DONTWAIT | M_ZERO);
+    r = malloc(sizeof *r, M_IPFW, M_NOWAIT | M_ZERO);
     if (r == NULL) {
        printf ("sorry cannot allocate state\n");
        return NULL ;
@@ -1628,7 +1628,7 @@ add_entry(struct ip_fw_head *head, struct ip_fw *rule)
        u_short nbr = 0;
        int s;
 
-       ftmp = malloc(sizeof *ftmp, M_IPFW, M_DONTWAIT | M_ZERO);
+       ftmp = malloc(sizeof *ftmp, M_IPFW, M_NOWAIT | M_ZERO);
        if (!ftmp)
                return (ENOSPC);
        bcopy(rule, ftmp, sizeof(*ftmp));
index 99ec7c6..dcf8f62 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/net/bridge.c,v 1.16.2.25 2003/01/23 21:06:44 sam Exp $
- * $DragonFly: src/sys/net/oldbridge/Attic/bridge.c,v 1.2 2003/06/17 04:28:47 dillon Exp $
+ * $DragonFly: src/sys/net/oldbridge/Attic/bridge.c,v 1.3 2003/07/29 12:51:29 hmp Exp $
  */
 
 /*
@@ -243,7 +243,7 @@ add_cluster(u_int16_t cluster_id, struct arpcom *ac)
            goto found;
 
     /* Not found, need to reallocate */
-    c = malloc((1+n_clusters) * sizeof (*c), M_IFADDR, M_DONTWAIT | M_ZERO);
+    c = malloc((1+n_clusters) * sizeof (*c), M_IFADDR, M_NOWAIT | M_ZERO);
     if (c == NULL) {/* malloc failure */
        printf("-- bridge: cannot add new cluster\n");
        return NULL;
index d6993a4..9adbca7 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.3 2003/07/26 20:19:33 rob Exp $
+ * $DragonFly: src/sys/net/radix.c,v 1.4 2003/07/29 12:51:29 hmp Exp $
  */
 
 /*
@@ -43,7 +43,6 @@
 #ifdef _KERNEL
 #include <sys/systm.h>
 #include <sys/malloc.h>
-#define        M_DONTWAIT M_NOWAIT
 #include <sys/domain.h>
 #else
 #include <stdlib.h>
index e6cf12d..ac9b3cc 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)radix.h     8.2 (Berkeley) 10/31/94
  * $FreeBSD: src/sys/net/radix.h,v 1.16.2.1 2000/05/03 19:17:11 wollman Exp $
- * $DragonFly: src/sys/net/radix.h,v 1.2 2003/06/17 04:28:48 dillon Exp $
+ * $DragonFly: src/sys/net/radix.h,v 1.3 2003/07/29 12:51:29 hmp Exp $
  */
 
 #ifndef _RADIX_H_
@@ -151,7 +151,7 @@ struct radix_node_head {
 #define Bcmp(a, b, n) bcmp(((caddr_t)(a)), ((caddr_t)(b)), (unsigned)(n))
 #define Bcopy(a, b, n) bcopy(((caddr_t)(a)), ((caddr_t)(b)), (unsigned)(n))
 #define Bzero(p, n) bzero((caddr_t)(p), (unsigned)(n));
-#define R_Malloc(p, t, n) (p = (t) malloc((unsigned long)(n), M_RTABLE, M_DONTWAIT))
+#define R_Malloc(p, t, n) (p = (t) malloc((unsigned long)(n), M_RTABLE, M_NOWAIT))
 #define Free(p) free((caddr_t)p, M_RTABLE);
 #endif /* _KERNEL */
 
index 60d61b8..f01b1f8 100644 (file)
@@ -32,7 +32,7 @@
  *
  *     @(#)raw_cb.c    8.1 (Berkeley) 6/10/93
  * $FreeBSD: src/sys/net/raw_cb.c,v 1.16 1999/08/28 00:48:27 peter Exp $
- * $DragonFly: src/sys/net/raw_cb.c,v 1.3 2003/07/26 20:19:33 rob Exp $
+ * $DragonFly: src/sys/net/raw_cb.c,v 1.4 2003/07/29 12:51:29 hmp Exp $
  */
 
 #include <sys/param.h>
@@ -139,7 +139,7 @@ raw_bind(so, nam)
        if (ifnet == 0)
                return (EADDRNOTAVAIL);
        rp = sotorawcb(so);
-       nam = m_copym(nam, 0, M_COPYALL, M_WAITOK);
+       nam = m_copym(nam, 0, M_COPYALL, M_TRYWAIT);
        rp->rcb_laddr = mtod(nam, struct sockaddr *);
        return (0);
 }
index 4e8397f..7442691 100644 (file)
@@ -38,7 +38,7 @@
  * Author: Archie Cobbs <archie@freebsd.org>
  *
  * $FreeBSD: src/sys/netgraph/ng_l2tp.c,v 1.1.2.1 2002/08/20 23:48:15 archie Exp $
- * $DragonFly: src/sys/netgraph/l2tp/ng_l2tp.c,v 1.2 2003/06/17 04:28:50 dillon Exp $
+ * $DragonFly: src/sys/netgraph/l2tp/ng_l2tp.c,v 1.3 2003/07/29 12:51:29 hmp Exp $
  */
 
 /*
@@ -56,6 +56,7 @@
 #include <sys/mbuf.h>
 #include <sys/malloc.h>
 #include <sys/errno.h>
+#include <sys/libkern.h>
 
 #include <netgraph/ng_message.h>
 #include <netgraph/netgraph.h>
@@ -295,9 +296,8 @@ NETGRAPH_INIT(l2tp, &ng_l2tp_typestruct);
 #define L2TP_SEQ_CHECK(x)      do { } while (0)
 #endif
 
-/* mem*() macros */
+/* memmove macro */
 #define memmove(d, s, l)       ovbcopy(s, d, l)
-#define memset(d, z, l)                bzero(d, l)             /* XXX */
 
 /* Whether to use m_copypacket() or m_dup() */
 #define L2TP_COPY_MBUF         m_copypacket
@@ -937,7 +937,7 @@ ng_l2tp_recv_ctrl(node_p node, struct mbuf *m, meta_p meta)
        }
 
        /* Copy packet */
-       if ((m = L2TP_COPY_MBUF(seq->xwin[i], M_NOWAIT)) == NULL) {
+       if ((m = L2TP_COPY_MBUF(seq->xwin[i], M_DONTWAIT)) == NULL) {
                priv->stats.memoryFailures++;
                return (ENOBUFS);
        }
@@ -1204,7 +1204,7 @@ ng_l2tp_seq_recv_nr(priv_p priv, u_int16_t nr)
         */
        while ((i = L2TP_SEQ_DIFF(seq->ns, seq->rack)) < seq->cwnd
            && seq->xwin[i] != NULL) {
-               if ((m = L2TP_COPY_MBUF(seq->xwin[i], M_NOWAIT)) == NULL)
+               if ((m = L2TP_COPY_MBUF(seq->xwin[i], M_DONTWAIT)) == NULL)
                        priv->stats.memoryFailures++;
                else
                        ng_l2tp_xmit_ctrl(priv, m, seq->ns);
@@ -1350,7 +1350,7 @@ ng_l2tp_seq_rack_timeout(void *arg)
        seq->acks = 0;
 
        /* Retransmit oldest unack'd packet */
-       if ((m = L2TP_COPY_MBUF(seq->xwin[0], M_NOWAIT)) == NULL)
+       if ((m = L2TP_COPY_MBUF(seq->xwin[0], M_DONTWAIT)) == NULL)
                priv->stats.memoryFailures++;
        else
                ng_l2tp_xmit_ctrl(priv, m, seq->rack);
index 5d1331e..43171ce 100644 (file)
@@ -37,7 +37,7 @@
  * Author: Archie Cobbs <archie@freebsd.org>
  *
  * $FreeBSD: src/sys/netgraph/ng_ppp.c,v 1.15.2.10 2003/03/10 17:55:48 archie Exp $
- * $DragonFly: src/sys/netgraph/ppp/ng_ppp.c,v 1.2 2003/06/17 04:28:51 dillon Exp $
+ * $DragonFly: src/sys/netgraph/ppp/ng_ppp.c,v 1.3 2003/07/29 12:51:29 hmp Exp $
  * $Whistle: ng_ppp.c,v 1.24 1999/11/01 09:24:52 julian Exp $
  */
 
@@ -1546,7 +1546,7 @@ deliver:
                        /* Split off next fragment as "m2" */
                        m2 = m;
                        if (!lastFragment) {
-                               struct mbuf *n = m_split(m, len, M_NOWAIT);
+                               struct mbuf *n = m_split(m, len, M_DONTWAIT);
 
                                if (n == NULL) {
                                        NG_FREE_DATA(m, meta);
@@ -1864,7 +1864,7 @@ ng_ppp_addproto(struct mbuf *m, int proto, int compOK)
 static struct mbuf *
 ng_ppp_prepend(struct mbuf *m, const void *buf, int len)
 {
-       M_PREPEND(m, len, M_NOWAIT);
+       M_PREPEND(m, len, M_DONTWAIT);
        if (m == NULL || (m->m_len < len && (m = m_pullup(m, len)) == NULL))
                return (NULL);
        bcopy(buf, mtod(m, u_char *), len);
index 427e6d7..574476d 100644 (file)
@@ -37,7 +37,7 @@
  * Author: Archie Cobbs <archie@freebsd.org>
  *
  * $FreeBSD: src/sys/netgraph/ng_pptpgre.c,v 1.2.2.13 2002/10/10 18:27:54 archie Exp $
- * $DragonFly: src/sys/netgraph/pptpgre/ng_pptpgre.c,v 1.2 2003/06/17 04:28:51 dillon Exp $
+ * $DragonFly: src/sys/netgraph/pptpgre/ng_pptpgre.c,v 1.3 2003/07/29 12:51:29 hmp Exp $
  * $Whistle: ng_pptpgre.c,v 1.7 1999/12/08 00:10:06 archie Exp $
  */
 
@@ -538,7 +538,7 @@ ng_pptpgre_xmit(node_p node, struct mbuf *m, meta_p meta)
                m->m_len = m->m_pkthdr.len = grelen;
                m->m_pkthdr.rcvif = NULL;
        } else {
-               M_PREPEND(m, grelen, M_NOWAIT);
+               M_PREPEND(m, grelen, M_DONTWAIT);
                if (m == NULL || (m->m_len < grelen
                    && (m = m_pullup(m, grelen)) == NULL)) {
                        priv->stats.memoryFailures++;
index f293927..745b2b5 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.2 2003/06/17 04:28:51 dillon Exp $   */
+/*     $DragonFly: src/sys/netinet/ip_encap.c,v 1.3 2003/07/29 12:51:30 hmp Exp $      */
 /*     $KAME: ip_encap.c,v 1.41 2001/03/15 08:35:08 itojun Exp $       */
 
 /*
@@ -492,7 +492,7 @@ encap_fillarg(m, ep)
 {
        struct m_tag *tag;
 
-       tag = m_tag_get(PACKET_TAG_ENCAP, sizeof (void*), M_NOWAIT);
+       tag = m_tag_get(PACKET_TAG_ENCAP, sizeof (void*), M_DONTWAIT);
        if (tag) {
                *(void**)(tag+1) = ep->arg;
                m_tag_prepend(m, tag);
index 1416e72..2ab45d5 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/esp_core.c,v 1.1.2.4 2002/03/26 10:12:29 ume Exp $   */
-/*     $DragonFly: src/sys/netinet6/esp_core.c,v 1.2 2003/06/17 04:28:51 dillon Exp $  */
+/*     $DragonFly: src/sys/netinet6/esp_core.c,v 1.3 2003/07/29 12:51:30 hmp Exp $     */
 /*     $KAME: esp_core.c,v 1.50 2000/11/02 12:27:38 itojun Exp $       */
 
 /*
@@ -220,7 +220,7 @@ esp_schedule(algo, sav)
        sav->schedlen = (*algo->schedlen)(algo);
        if (sav->schedlen < 0)
                return EINVAL;
-       sav->sched = malloc(sav->schedlen, M_SECA, M_DONTWAIT);
+       sav->sched = malloc(sav->schedlen, M_SECA, M_NOWAIT);
        if (!sav->sched) {
                sav->schedlen = 0;
                return ENOBUFS;
index 2d67536..ef79ae4 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/ip6_input.c,v 1.11.2.15 2003/01/24 05:11:35 sam Exp $        */
-/*     $DragonFly: src/sys/netinet6/ip6_input.c,v 1.4 2003/06/25 03:56:04 dillon Exp $ */
+/*     $DragonFly: src/sys/netinet6/ip6_input.c,v 1.5 2003/07/29 12:51:30 hmp Exp $    */
 /*     $KAME: ip6_input.c,v 1.259 2002/01/21 04:58:09 jinmei Exp $     */
 
 /*
@@ -1609,7 +1609,7 @@ ip6_addaux(m)
        if (!tag) {
                tag = m_tag_get(PACKET_TAG_IPV6_INPUT,
                                sizeof (struct ip6aux),
-                               M_NOWAIT);
+                               M_DONTWAIT);
                if (tag)
                        m_tag_prepend(m, tag);
        }
index d2849c8..7a900de 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netinet6/ipcomp_output.c,v 1.1.2.4 2003/04/29 08:33:50 suz Exp $      */
-/*     $DragonFly: src/sys/netinet6/ipcomp_output.c,v 1.2 2003/06/17 04:28:52 dillon Exp $     */
+/*     $DragonFly: src/sys/netinet6/ipcomp_output.c,v 1.3 2003/07/29 12:51:30 hmp Exp $        */
 /*     $KAME: ipcomp_output.c,v 1.25 2002/06/09 14:44:00 itojun Exp $  */
 
 /*
@@ -172,12 +172,12 @@ ipcomp_output(m, nexthdrp, md, isr, af)
         * compromise two m_copym().  we will be going through every byte of
         * the payload during compression process anyways.
         */
-       mcopy = m_copym(m, 0, M_COPYALL, M_NOWAIT);
+       mcopy = m_copym(m, 0, M_COPYALL, M_DONTWAIT);
        if (mcopy == NULL) {
                error = ENOBUFS;
                return 0;
        }
-       md0 = m_copym(md, 0, M_COPYALL, M_NOWAIT);
+       md0 = m_copym(md, 0, M_COPYALL, M_DONTWAIT);
        if (md0 == NULL) {
                m_freem(mcopy);
                error = ENOBUFS;
index 0ede95e..792b892 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/netkey/keysock.c,v 1.1.2.4 2003/01/11 19:10:59 ume Exp $      */
-/*     $DragonFly: src/sys/netproto/key/keysock.c,v 1.3 2003/07/21 07:57:51 dillon Exp $       */
+/*     $DragonFly: src/sys/netproto/key/keysock.c,v 1.4 2003/07/29 12:51:30 hmp Exp $  */
 /*     $KAME: keysock.c,v 1.25 2001/08/13 20:07:41 itojun Exp $        */
 
 /*
@@ -150,7 +150,7 @@ key_sendup0(rp, m, promisc)
        if (promisc) {
                struct sadb_msg *pmsg;
 
-               M_PREPEND(m, sizeof(struct sadb_msg), M_NOWAIT);
+               M_PREPEND(m, sizeof(struct sadb_msg), M_DONTWAIT);
                if (m && m->m_len < sizeof(struct sadb_msg))
                        m = m_pullup(m, sizeof(struct sadb_msg));
                if (!m) {