[src/trunk]: src/sys/netinet make sending of keepalive messages work again:

[drochner <drochner%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/bab1f7b7e4fa
branches:  trunk
changeset: 474659:bab1f7b7e4fa
user:      drochner <drochner%NetBSD.org@localhost>
date:      Wed Jul 14 22:08:52 1999 +0000

description:
make sending of keepalive messages work again:
-remove bogus sanity check involving an uninitialized variable
-correct mbuf cluster allocation
-(non-critical) remove redundant check in cleanup after error

diffstat:

 sys/netinet/tcp_subr.c |  13 ++-----------
 1 files changed, 2 insertions(+), 11 deletions(-)

diffs (40 lines):

diff -r fd3c93acfc4b -r bab1f7b7e4fa sys/netinet/tcp_subr.c
--- a/sys/netinet/tcp_subr.c    Wed Jul 14 21:36:36 1999 +0000
+++ b/sys/netinet/tcp_subr.c    Wed Jul 14 22:08:52 1999 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: tcp_subr.c,v 1.70 1999/07/09 22:57:22 thorpej Exp $    */
+/*     $NetBSD: tcp_subr.c,v 1.71 1999/07/14 22:08:52 drochner Exp $   */
 
 /*
  * Copyright (C) 1995, 1996, 1997, and 1998 WIDE Project.
@@ -391,13 +391,6 @@
        ip6 = NULL;
 #endif
        if (m == 0) {
-               if (template
-                && template->m_len < hlen + sizeof(struct tcphdr)) {
-                       if (m)
-                               m_freem(m);
-                       return EINVAL;
-               }
-
                /* get family information from template */
                switch (mtod(template, struct ip *)->ip_v) {
                case 4:
@@ -411,15 +404,13 @@
                        break;
 #endif
                default:
-                       if (m)
-                               m_freem(m);
                        return EAFNOSUPPORT;
                }
 
                MGETHDR(m, M_DONTWAIT, MT_HEADER);
                if (m) {
                        MCLGET(m, M_DONTWAIT);
-                       if ((m->m_flags & M_EXT)) {
+                       if (!(m->m_flags & M_EXT)) {
                                m_free(m);
                                m = NULL;
                        }