diff options
author | glebius <glebius@FreeBSD.org> | 2005-06-10 12:44:21 +0000 |
---|---|---|
committer | glebius <glebius@FreeBSD.org> | 2005-06-10 12:44:21 +0000 |
commit | 8386a8457c7bb9838a5e984eac959cbfd2ab5f1a (patch) | |
tree | 417c702059fb54ec07a2586a8368ad043d94343a /sys | |
parent | 9f795bf703e2446c4dede664749a8e7d3df4e18a (diff) | |
download | FreeBSD-src-8386a8457c7bb9838a5e984eac959cbfd2ab5f1a.zip FreeBSD-src-8386a8457c7bb9838a5e984eac959cbfd2ab5f1a.tar.gz |
- Remove redundant parenthesis in M_CHECK macro.
- Do not edit pullup_len outside M_CHECK macro.
- Do not reimplement NG_FWD_NEW_DATA().
- Remove redundant check for item being not NULL.
Submitted by: ru
Diffstat (limited to 'sys')
-rw-r--r-- | sys/netgraph/ng_tcpmss.c | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/sys/netgraph/ng_tcpmss.c b/sys/netgraph/ng_tcpmss.c index eda72d3..760dad3 100644 --- a/sys/netgraph/ng_tcpmss.c +++ b/sys/netgraph/ng_tcpmss.c @@ -289,10 +289,10 @@ ng_tcpmss_rcvdata(hook_p hook, item_p item) #define M_CHECK(length) do { \ pullup_len += length; \ - if ((m)->m_pkthdr.len < (pullup_len)) \ + if ((m)->m_pkthdr.len < pullup_len) \ goto send; \ - if ((m)->m_len < (pullup_len) && \ - (((m) = m_pullup((m),(pullup_len))) == NULL)) \ + if ((m)->m_len < pullup_len && \ + (((m) = m_pullup((m), pullup_len)) == NULL)) \ ERROUT(ENOBUFS); \ } while (0) @@ -308,21 +308,19 @@ ng_tcpmss_rcvdata(hook_p hook, item_p item) iphlen = ip->ip_hl << 2; if (iphlen < sizeof(struct ip) || iphlen > pktlen ) ERROUT(EINVAL); - pullup_len += iphlen - sizeof(struct ip); /* Check if it is TCP. */ if (!(ip->ip_p == IPPROTO_TCP)) goto send; /* Check mbuf packet size and arrange for IP+TCP header */ - M_CHECK(sizeof(struct tcphdr)); + M_CHECK(iphlen - sizeof(struct ip) + sizeof(struct tcphdr)); tcp = (struct tcphdr *)((caddr_t )ip + iphlen); /* Check TCP header length. */ tcphlen = tcp->th_off << 2; if (tcphlen < sizeof(struct tcphdr) || tcphlen > pktlen - iphlen) ERROUT(EINVAL); - pullup_len += tcphlen - sizeof(struct tcphdr); /* Check SYN packet and has options. */ if (!(tcp->th_flags & TH_SYN) || tcphlen == sizeof(struct tcphdr)) @@ -331,7 +329,7 @@ ng_tcpmss_rcvdata(hook_p hook, item_p item) /* Update SYN stats. */ priv->stats.SYNPkts++; - M_CHECK(0); + M_CHECK(tcphlen - sizeof(struct tcphdr)); #undef M_CHECK @@ -342,15 +340,12 @@ ng_tcpmss_rcvdata(hook_p hook, item_p item) send: /* Deliver frame out destination hook. */ - NGI_M(item) = m; - NG_FWD_ITEM_HOOK(error, item, priv->outHook); - + NG_FWD_NEW_DATA(error, item, priv->outHook, m); + return (error); done: - /* Free mbuf if unfreed left. */ - if (item) - NG_FREE_ITEM(item); + NG_FREE_ITEM(item); NG_FREE_M(m); return (error); |