Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/netipsec Improve stupid check, style, and fix leak (m, n...



details:   https://anonhg.NetBSD.org/src/rev/1db5e23f4f0c
branches:  trunk
changeset: 320942:1db5e23f4f0c
user:      maxv <maxv%NetBSD.org@localhost>
date:      Mon Mar 05 12:42:28 2018 +0000

description:
Improve stupid check, style, and fix leak (m, not m0).

diffstat:

 sys/netipsec/ipsec_mbuf.c |  18 +++++++-----------
 1 files changed, 7 insertions(+), 11 deletions(-)

diffs (67 lines):

diff -r 41511cfe12e1 -r 1db5e23f4f0c sys/netipsec/ipsec_mbuf.c
--- a/sys/netipsec/ipsec_mbuf.c Mon Mar 05 11:50:25 2018 +0000
+++ b/sys/netipsec/ipsec_mbuf.c Mon Mar 05 12:42:28 2018 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ipsec_mbuf.c,v 1.20 2018/02/26 06:17:01 maxv Exp $     */
+/*     $NetBSD: ipsec_mbuf.c,v 1.21 2018/03/05 12:42:28 maxv Exp $     */
 
 /*
  * Copyright (c) 2002, 2003 Sam Leffler, Errno Consulting
@@ -29,7 +29,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ipsec_mbuf.c,v 1.20 2018/02/26 06:17:01 maxv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ipsec_mbuf.c,v 1.21 2018/03/05 12:42:28 maxv Exp $");
 
 /*
  * IPsec-specific mbuf routines.
@@ -317,10 +317,8 @@
        register int len, pad;
        void *retval;
 
-       if (n <= 0) {  /* No stupid arguments. */
-               IPSECLOG(LOG_DEBUG, "pad length invalid (%d)\n", n);
-               m_freem(m);
-               return NULL;
+       if (__predict_false(n > MLEN)) {
+               panic("%s: %d > MLEN", __func__, n);
        }
 
        len = m->m_pkthdr.len;
@@ -329,7 +327,7 @@
 
        while (m0->m_len < len) {
                KASSERTMSG(m0->m_next != NULL,
-                   "m0 null, len %u m_len %u", len, m0->m_len);/*XXX*/
+                   "m0 null, len %u m_len %u", len, m0->m_len);
                len -= m0->m_len;
                m0 = m0->m_next;
        }
@@ -338,7 +336,6 @@
                IPSECLOG(LOG_DEBUG,
                    "length mismatch (should be %d instead of %d)\n",
                    m->m_pkthdr.len, m->m_pkthdr.len + m0->m_len - len);
-
                m_freem(m);
                return NULL;
        }
@@ -350,7 +347,6 @@
                            "length mismatch (should be %d instead of %d)\n",
                            m->m_pkthdr.len,
                            m->m_pkthdr.len + m1->m_next->m_len);
-
                        m_freem(m);
                        return NULL;
                }
@@ -361,8 +357,8 @@
        if (pad > M_TRAILINGSPACE(m0)) {
                /* Add an mbuf to the chain. */
                MGET(m1, M_DONTWAIT, MT_DATA);
-               if (m1 == 0) {
-                       m_freem(m0);
+               if (m1 == NULL) {
+                       m_freem(m);
                        IPSECLOG(LOG_DEBUG, "unable to get extra mbuf\n");
                        return NULL;
                }



Home | Main Index | Thread Index | Old Index