At work all the log(LOG_DEBUG, ...) statements in the TCP code in 7.x are
spamming the heck out of our dmesg so I am #if 0'ing all of them out and
while doing so ran into this case. Specifically, it doesn't actually bump
the stat counter unless it succeeds in allocating memory to log the debug
message. All the other places in the syncache and tcp input code always bump
the stats, so this patch fixes it to do that.
Index: tcp_syncache.c
===================================================================
RCS file: /usr/cvs/src/sys/netinet/tcp_syncache.c,v
retrieving revision 1.143
diff -u -r1.143 tcp_syncache.c
--- tcp_syncache.c 19 Apr 2008 03:39:17 -0000 1.143
+++ tcp_syncache.c 8 May 2008 20:22:21 -0000
@@ -567,10 +567,11 @@
"connection attempt aborted by remote endpoint\n",
s, __func__);
tcpstat.tcps_sc_reset++;
- } else if ((s = tcp_log_addrs(inc, th, NULL, NULL))) {
- log(LOG_DEBUG, "%s; %s: RST with invalid SEQ %u != IRS %u "
- "(+WND %u), segment ignored\n",
- s, __func__, th->th_seq, sc->sc_irs, sc->sc_wnd);
+ } else {
+ if ((s = tcp_log_addrs(inc, th, NULL, NULL)))
+ log(LOG_DEBUG, "%s; %s: RST with invalid SEQ %u != "
+ "IRS %u (+WND %u), segment ignored\n",
+ s, __func__, th->th_seq, sc->sc_irs, sc->sc_wnd);
tcpstat.tcps_badrst++;
}
--
John Baldwin
_______________________________________________
freebsd-net@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "[EMAIL PROTECTED]"
