From ddc865fb0ddee9ed75467a164f27472c2234910a Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Wed, 26 Dec 2007 21:49:33 +0000 Subject: [PATCH] netstat: in error message, report filename and line number where we see problematic data text data bss dec hex filename 778456 832 7344 786632 c00c8 busybox_old 778394 832 7344 786570 c008a busybox_unstripped --- networking/netstat.c | 45 +++++++++++++++++++++++---------------------- 1 file changed, 23 insertions(+), 22 deletions(-) diff --git a/networking/netstat.c b/networking/netstat.c index 1c78f9d..29c2384 100644 --- a/networking/netstat.c +++ b/networking/netstat.c @@ -148,7 +148,7 @@ static char *ip_port_str(struct sockaddr *addr, int port, const char *proto, int return host_port; } -static void tcp_do_one(int lnr, char *line) +static int tcp_do_one(int lnr, char *line) { char local_addr[64], rem_addr[64]; char more[512]; @@ -161,7 +161,7 @@ static void tcp_do_one(int lnr, char *line) unsigned long rxq, txq, time_len, retr, inode; if (lnr == 0) - return; + return 0; more[0] = '\0'; num = sscanf(line, @@ -171,8 +171,7 @@ static void tcp_do_one(int lnr, char *line) &txq, &rxq, &timer_run, &time_len, &retr, &uid, &timeout, &inode, more); if (num < 10) { - bb_error_msg("warning, got bogus tcp line"); - return; + return 1; /* error */ } if (strlen(local_addr) > 8) { @@ -199,9 +198,10 @@ static void tcp_do_one(int lnr, char *line) free(l); free(r); } + return 0; } -static void udp_do_one(int lnr, char *line) +static int udp_do_one(int lnr, char *line) { char local_addr[64], rem_addr[64]; const char *state_str; @@ -215,7 +215,7 @@ static void udp_do_one(int lnr, char *line) unsigned long rxq, txq, time_len, retr, inode; if (lnr == 0) - return; + return 0; more[0] = '\0'; num = sscanf(line, @@ -236,8 +236,7 @@ static void udp_do_one(int lnr, char *line) } if (num < 10) { - bb_error_msg("warning, got bogus udp line"); - return; + return 1; /* error */ } switch (state) { case TCP_ESTABLISHED: @@ -281,9 +280,10 @@ static void udp_do_one(int lnr, char *line) free(r); } } + return 0; } -static void raw_do_one(int lnr, char *line) +static int raw_do_one(int lnr, char *line) { char local_addr[64], rem_addr[64]; char more[512]; @@ -296,7 +296,7 @@ static void raw_do_one(int lnr, char *line) unsigned long rxq, txq, time_len, retr, inode; if (lnr == 0) - return; + return 0; more[0] = '\0'; num = sscanf(line, @@ -316,8 +316,7 @@ static void raw_do_one(int lnr, char *line) } if (num < 10) { - bb_error_msg("warning, got bogus raw line"); - return; + return 1; /* error */ } { @@ -337,9 +336,10 @@ static void raw_do_one(int lnr, char *line) free(r); } } + return 0; } -static void unix_do_one(int nr, char *line) +static int unix_do_one(int nr, char *line) { unsigned long refcnt, proto, unix_flags; unsigned long inode; @@ -352,7 +352,7 @@ static void unix_do_one(int nr, char *line) /* TODO: currently we stop at first NUL byte. Is it a problem? */ if (nr == 0) - return; /* skip header */ + return 0; /* skip header */ *strchrnul(line, '\n') = '\0'; @@ -360,22 +360,21 @@ static void unix_do_one(int nr, char *line) * Other users report long lines filled by NUL bytes. * (those ^@ are NUL bytes too). We see them as empty lines. */ if (!line[0]) - return; + return 0; path_ofs = 0; /* paranoia */ num = sscanf(line, "%p: %lX %lX %lX %X %X %lu %n", &d, &refcnt, &proto, &unix_flags, &type, &state, &inode, &path_ofs); if (num < 7) { - bb_error_msg("got bogus unix line '%s'", line); - return; + return 1; /* error */ } if ((flags & (NETSTAT_LISTENING|NETSTAT_CONNECTED)) != (NETSTAT_LISTENING|NETSTAT_CONNECTED)) { if ((state == SS_UNCONNECTED) && (unix_flags & SO_ACCEPTCON)) { if (!(flags & NETSTAT_LISTENING)) - return; + return 0; } else { if (!(flags & NETSTAT_CONNECTED)) - return; + return 0; } } @@ -447,6 +446,7 @@ static void unix_do_one(int nr, char *line) printf("%-5s %-6ld %-11s %-10s %-13s %6lu %s\n", ss_proto, refcnt, ss_flags, ss_type, ss_state, inode, line + path_ofs); + return 0; } #define _PATH_PROCNET_UDP "/proc/net/udp" @@ -457,7 +457,7 @@ static void unix_do_one(int nr, char *line) #define _PATH_PROCNET_RAW6 "/proc/net/raw6" #define _PATH_PROCNET_UNIX "/proc/net/unix" -static void do_info(const char *file, const char *name, void (*proc)(int, char *)) +static void do_info(const char *file, const char *name, int (*proc)(int, char *)) { int lnr; FILE *procinfo; @@ -468,14 +468,15 @@ static void do_info(const char *file, const char *name, void (*proc)(int, char * if (errno != ENOENT) { bb_simple_perror_msg(file); } else { - bb_error_msg("no support for '%s' on this system", name); + bb_error_msg("no kernel support for %s", name); } return; } lnr = 0; /* Why? because xmalloc_fgets_str doesn't stop on NULs */ while ((buffer = xmalloc_fgets_str(procinfo, "\n")) != NULL) { - (proc)(lnr++, buffer); + if (proc(lnr++, buffer)) + bb_error_msg("%s: bogus data on line %d", file, lnr); free(buffer); } fclose(procinfo); -- 2.7.4