From owner-svn-src-head@freebsd.org Thu Feb 15 21:06:57 2018 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 9B0C1F1A592; Thu, 15 Feb 2018 21:06:57 +0000 (UTC) (envelope-from cem@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4BBC4705A8; Thu, 15 Feb 2018 21:06:57 +0000 (UTC) (envelope-from cem@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 424F024042; Thu, 15 Feb 2018 21:06:57 +0000 (UTC) (envelope-from cem@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id w1FL6v1a092372; Thu, 15 Feb 2018 21:06:57 GMT (envelope-from cem@FreeBSD.org) Received: (from cem@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id w1FL6uNa092369; Thu, 15 Feb 2018 21:06:56 GMT (envelope-from cem@FreeBSD.org) Message-Id: <201802152106.w1FL6uNa092369@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: cem set sender to cem@FreeBSD.org using -f From: Conrad Meyer Date: Thu, 15 Feb 2018 21:06:56 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r329341 - head/usr.sbin/trpt X-SVN-Group: head X-SVN-Commit-Author: cem X-SVN-Commit-Paths: head/usr.sbin/trpt X-SVN-Commit-Revision: 329341 X-SVN-Commit-Repository: base MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 15 Feb 2018 21:06:57 -0000 Author: cem Date: Thu Feb 15 21:06:56 2018 New Revision: 329341 URL: https://svnweb.freebsd.org/changeset/base/329341 Log: trpt(8): Sprinkle style(9) and adddress warnings Update function definitions to post-ISO style; remove deprecated "register" keyword. Correct indentation. Sprinkle 'static' on file-local variables. Appease warnings at WARNS=6, for both Clang and GCC. Sponsored by: Dell EMC Isilon Modified: head/usr.sbin/trpt/Makefile head/usr.sbin/trpt/trpt.c Modified: head/usr.sbin/trpt/Makefile ============================================================================== --- head/usr.sbin/trpt/Makefile Thu Feb 15 20:42:38 2018 (r329340) +++ head/usr.sbin/trpt/Makefile Thu Feb 15 21:06:56 2018 (r329341) @@ -8,10 +8,12 @@ MAN= trpt.8 BINGRP= kmem BINMODE= 2555 -WARNS?= 4 - .if ${MK_INET6_SUPPORT} != "no" CFLAGS+= -DINET6 .endif .include + +# Several included system headers tickle this warning in ways that are +# difficult to work around in this program. +CFLAGS+= -Wno-missing-variable-declarations Modified: head/usr.sbin/trpt/trpt.c ============================================================================== --- head/usr.sbin/trpt/trpt.c Thu Feb 15 20:42:38 2018 (r329340) +++ head/usr.sbin/trpt/trpt.c Thu Feb 15 21:06:56 2018 (r329341) @@ -79,12 +79,13 @@ __FBSDID("$FreeBSD$"); #include #include #include +#include #include #include #include #include -struct nlist nl[3]; +static struct nlist nl[3]; #define N_TCP_DEBUG 0 #define N_TCP_DEBX 1 @@ -92,10 +93,10 @@ static caddr_t tcp_pcbs[TCP_NDEBUG]; static n_time ntime; static int aflag, kflag, memf, follow, sflag; -void dotrace(caddr_t); -void klseek(int, off_t, int); -int numeric(const void *, const void *); -void tcp_trace(short, short, struct tcpcb *, int, void *, struct tcphdr *, int); +static void dotrace(caddr_t); +static void klseek(int, off_t, int); +static int numeric(const void *, const void *); +static void tcp_trace(short, short, struct tcpcb *, int, void *, struct tcphdr *, int); static void usage(void); int @@ -150,8 +151,7 @@ main(int argc, char **argv) */ if (setgid(getgid()) != 0) err(1, "setgid"); - } - else + } else syst = getbootfile(); if (nlist(syst, nl) < 0 || !nl[0].n_value) @@ -178,8 +178,8 @@ main(int argc, char **argv) */ if (!npcbs) { for (i = 0; i < TCP_NDEBUG; i++) { - register struct tcp_debug *td = &tcp_debug[i]; - register int j; + struct tcp_debug *td = &tcp_debug[i]; + int j; if (td->td_tcb == 0) continue; @@ -201,31 +201,31 @@ main(int argc, char **argv) fputs(", ", stdout); } putchar('\n'); - } - else for (i = 0; i < npcbs; i++) { - printf("\n%p:\n", tcp_pcbs[i]); - dotrace(tcp_pcbs[i]); - } + } else + for (i = 0; i < npcbs; i++) { + printf("\n%p:\n", tcp_pcbs[i]); + dotrace(tcp_pcbs[i]); + } exit(0); } static void -usage() +usage(void) { (void)fprintf(stderr, - "usage: trpt [-afjs] [-p hex-address] [system [core]]\n"); + "usage: trpt [-afjs] [-p hex-address] [system [core]]\n"); exit(1); } -void -dotrace(tcpcb) - register caddr_t tcpcb; +static void +dotrace(caddr_t tcpcb) { - register struct tcp_debug *td; - register int i; + struct tcp_debug *td; + int i; int prev_debx = tcp_debx, family; -again: if (--tcp_debx < 0) +again: + if (--tcp_debx < 0) tcp_debx = TCP_NDEBUG - 1; for (i = prev_debx % TCP_NDEBUG; i < TCP_NDEBUG; i++) { td = &tcp_debug[i]; @@ -237,17 +237,17 @@ again: if (--tcp_debx < 0) #else family = AF_INET; #endif - switch(family) { + switch (family) { case AF_INET: - tcp_trace(td->td_act, td->td_ostate, - &td->td_cb, td->td_family, &td->td_ti.ti_i, - &td->td_ti.ti_t, td->td_req); + tcp_trace(td->td_act, td->td_ostate, &td->td_cb, + td->td_family, &td->td_ti.ti_i, &td->td_ti.ti_t, + td->td_req); break; #ifdef INET6 case AF_INET6: - tcp_trace(td->td_act, td->td_ostate, - &td->td_cb, td->td_family, &td->td_ti6.ip6, - &td->td_ti6.th, td->td_req); + tcp_trace(td->td_act, td->td_ostate, &td->td_cb, + td->td_family, &td->td_ti6.ip6, &td->td_ti6.th, + td->td_req); break; #endif } @@ -264,22 +264,23 @@ again: if (--tcp_debx < 0) #else family = AF_INET; #endif - switch(family) { + switch (family) { case AF_INET: - tcp_trace(td->td_act, td->td_ostate, - &td->td_cb, td->td_family, &td->td_ti.ti_i, - &td->td_ti.ti_t, td->td_req); + tcp_trace(td->td_act, td->td_ostate, &td->td_cb, + td->td_family, &td->td_ti.ti_i, &td->td_ti.ti_t, + td->td_req); break; #ifdef INET6 case AF_INET6: - tcp_trace(td->td_act, td->td_ostate, - &td->td_cb, td->td_family, &td->td_ti6.ip6, - &td->td_ti6.th, td->td_req); + tcp_trace(td->td_act, td->td_ostate, &td->td_cb, + td->td_family, &td->td_ti6.ip6, &td->td_ti6.th, + td->td_req); break; #endif } } -done: if (follow) { +done: + if (follow) { prev_debx = tcp_debx + 1; if (prev_debx >= TCP_NDEBUG) prev_debx = 0; @@ -302,7 +303,7 @@ done: if (follow) { * Tcp debug routines */ /*ARGSUSED*/ -void +static void tcp_trace(short act, short ostate, struct tcpcb *tp, int family __unused, void *ip, struct tcphdr *th, int req) { @@ -310,21 +311,26 @@ tcp_trace(short act, short ostate, struct tcpcb *tp, i int flags, len, win, timer; struct ip *ip4; #ifdef INET6 - int isipv6, nopkt = 1; + bool isipv6, nopkt = true; struct ip6_hdr *ip6; char ntop_buf[INET6_ADDRSTRLEN]; #endif #ifdef INET6 + /* Appease GCC -Wmaybe-uninitialized */ + ip4 = NULL; + ip6 = NULL; + isipv6 = false; + switch (family) { case AF_INET: - nopkt = 0; - isipv6 = 0; + nopkt = false; + isipv6 = false; ip4 = (struct ip *)ip; break; case AF_INET6: - nopkt = 0; - isipv6 = 1; + nopkt = false; + isipv6 = true; ip6 = (struct ip6_hdr *)ip; case 0: default: @@ -333,43 +339,39 @@ tcp_trace(short act, short ostate, struct tcpcb *tp, i #else ip4 = (struct ip *)ip; #endif - printf("%03ld %s:%s ", (long)((ntime/10) % 1000), tcpstates[ostate], + printf("%03ld %s:%s ", (long)((ntime / 10) % 1000), tcpstates[ostate], tanames[act]); switch (act) { case TA_INPUT: case TA_OUTPUT: case TA_DROP: #ifdef INET6 - if (nopkt != 0) + if (nopkt) break; #endif if (aflag) { printf("(src=%s,%u, ", #ifdef INET6 - isipv6 - ? inet_ntop(AF_INET6, &ip6->ip6_src, ntop_buf, - sizeof(ntop_buf)) : + isipv6 ? inet_ntop(AF_INET6, &ip6->ip6_src, + ntop_buf, sizeof(ntop_buf)) : #endif - inet_ntoa(ip4->ip_src), - ntohs(th->th_sport)); + inet_ntoa(ip4->ip_src), ntohs(th->th_sport)); printf("dst=%s,%u)", #ifdef INET6 - isipv6 - ? inet_ntop(AF_INET6, &ip6->ip6_dst, ntop_buf, - sizeof(ntop_buf)) : + isipv6 ? inet_ntop(AF_INET6, &ip6->ip6_dst, + ntop_buf, sizeof(ntop_buf)) : #endif - inet_ntoa(ip4->ip_dst), - ntohs(th->th_dport)); + inet_ntoa(ip4->ip_dst), ntohs(th->th_dport)); } seq = th->th_seq; ack = th->th_ack; len = #ifdef INET6 - isipv6 ? ip6->ip6_plen : + isipv6 ? ip6->ip6_plen : #endif - ip4->ip_len; + ip4->ip_len; win = th->th_win; if (act == TA_OUTPUT) { seq = ntohl(seq); @@ -421,23 +423,21 @@ tcp_trace(short act, short ostate, struct tcpcb *tp, i (u_long)tp->snd_una, (u_long)tp->snd_nxt, (u_long)tp->snd_max); printf("\tsnd_wl1 %lx snd_wl2 %lx snd_wnd %lx\n", - (u_long)tp->snd_wl1, - (u_long)tp->snd_wl2, (u_long)tp->snd_wnd); + (u_long)tp->snd_wl1, (u_long)tp->snd_wl2, + (u_long)tp->snd_wnd); } } -int -numeric(v1, v2) - const void *v1, *v2; +static int +numeric(const void *v1, const void *v2) { const caddr_t *c1 = v1, *c2 = v2; - return(*c1 - *c2); + + return (*c1 - *c2); } -void -klseek(fd, base, off) - int fd, off; - off_t base; +static void +klseek(int fd, off_t base, int off) { (void)lseek(fd, base, off); }