diff options
author | andre <andre@FreeBSD.org> | 2013-07-16 16:37:08 +0000 |
---|---|---|
committer | andre <andre@FreeBSD.org> | 2013-07-16 16:37:08 +0000 |
commit | 67c418ca84443f20c6221715f55d28d2b7c5d043 (patch) | |
tree | bf7494563200bc7c81c9387da09c4c50c61c1100 /sys/netinet | |
parent | 306e14bae389ddad05452ba4aaab074dd0002e44 (diff) | |
download | FreeBSD-src-67c418ca84443f20c6221715f55d28d2b7c5d043.zip FreeBSD-src-67c418ca84443f20c6221715f55d28d2b7c5d043.tar.gz |
Free the non-fatal "timestamp missing" debug string manually as it is
not covered by the catch-all free for the error cases.
Found by: Coverity
Diffstat (limited to 'sys/netinet')
-rw-r--r-- | sys/netinet/tcp_syncache.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sys/netinet/tcp_syncache.c b/sys/netinet/tcp_syncache.c index 71dfa9d..cd7b424 100644 --- a/sys/netinet/tcp_syncache.c +++ b/sys/netinet/tcp_syncache.c @@ -1041,9 +1041,12 @@ syncache_expand(struct in_conninfo *inc, struct tcpopt *to, struct tcphdr *th, * reports of non-compliants stacks. */ if ((sc->sc_flags & SCF_TIMESTAMP) && !(to->to_flags & TOF_TS)) { - if ((s = tcp_log_addrs(inc, th, NULL, NULL))) + if ((s = tcp_log_addrs(inc, th, NULL, NULL))) { log(LOG_DEBUG, "%s; %s: Timestamp missing, " "no action\n", s, __func__); + free(s, M_TCPLOG); + s = NULL; + } } /* |