X-Git-Url: https://git.tcpdump.org/tcpdump/blobdiff_plain/6e0c21b1c0d3141e5e50bc32c9d7cf63eb68a8e6..c39d40a767a1ae36171e5bcbf6f157ff3e80fb6c:/print-igrp.c

diff --git a/print-igrp.c b/print-igrp.c
index a2d703c1..0efc78af 100644
--- a/print-igrp.c
+++ b/print-igrp.c
@@ -24,10 +24,10 @@
 /* \summary: Interior Gateway Routing Protocol (IGRP) printer */
 
 #ifdef HAVE_CONFIG_H
-#include "config.h"
+#include <config.h>
 #endif
 
-#include <netdissect-stdinc.h>
+#include "netdissect-stdinc.h"
 
 #include "netdissect.h"
 #include "extract.h"
@@ -63,36 +63,23 @@ struct igrprte {
 	nd_uint8_t igr_hct;	/* hop count */
 };
 
-#define IGRP_RTE_SIZE	14	/* don't believe sizeof ! */
+#define IGRP_RTE_SIZE	14	/* sizeof() is accurate now */
 
 static void
-igrp_entry_print(netdissect_options *ndo, const struct igrprte *igr,
-    int is_interior, int is_exterior)
+igrp_entry_print(netdissect_options *ndo, const struct igrprte *igr)
 {
 	u_int delay, bandwidth;
 	u_int metric, mtu;
 
-	if (is_interior)
-		ND_PRINT((ndo, " *.%u.%u.%u", igr->igr_net[0],
-		    igr->igr_net[1], igr->igr_net[2]));
-	else if (is_exterior)
-		ND_PRINT((ndo, " X%u.%u.%u.0", igr->igr_net[0],
-		    igr->igr_net[1], igr->igr_net[2]));
-	else
-		ND_PRINT((ndo, " %u.%u.%u.0", igr->igr_net[0],
-		    igr->igr_net[1], igr->igr_net[2]));
-
-	delay = EXTRACT_BE_U_3(igr->igr_dly);
-	bandwidth = EXTRACT_BE_U_3(igr->igr_bw);
-	metric = bandwidth + delay;
-	if (metric > 0xffffff)
-		metric = 0xffffff;
-	mtu = EXTRACT_BE_U_2(igr->igr_mtu);
-
-	ND_PRINT((ndo, " d=%u b=%u r=%u l=%u M=%u mtu=%u in %u hops",
+	delay = GET_BE_U_3(igr->igr_dly);
+	bandwidth = GET_BE_U_3(igr->igr_bw);
+	metric = ND_MIN(bandwidth + delay, 0xffffff);
+	mtu = GET_BE_U_2(igr->igr_mtu);
+
+	ND_PRINT(" d=%u b=%u r=%u l=%u M=%u mtu=%u in %u hops",
 	    10 * delay, bandwidth == 0 ? 0 : 10000000 / bandwidth,
-	    EXTRACT_U_1(igr->igr_rel), EXTRACT_U_1(igr->igr_ld), metric,
-	    mtu, EXTRACT_U_1(igr->igr_hct)));
+	    GET_U_1(igr->igr_rel), GET_U_1(igr->igr_ld), metric,
+	    mtu, GET_U_1(igr->igr_hct));
 }
 
 static const struct tok op2str[] = {
@@ -107,49 +94,56 @@ igrp_print(netdissect_options *ndo, const u_char *bp, u_int length)
 	const struct igrphdr *hdr;
 	const u_char *cp;
 	u_int nint, nsys, next;
+	uint16_t cksum;
 
+	ndo->ndo_protocol = "igrp";
 	hdr = (const struct igrphdr *)bp;
 	cp = (const u_char *)(hdr + 1);
-	ND_PRINT((ndo, "igrp:"));
+	ND_PRINT("igrp:");
 
 	/* Header */
-	ND_TCHECK(*hdr);
-	nint = EXTRACT_BE_U_2(hdr->ig_ni);
-	nsys = EXTRACT_BE_U_2(hdr->ig_ns);
-	next = EXTRACT_BE_U_2(hdr->ig_nx);
-
-	ND_PRINT((ndo, " %s V%u edit=%u AS=%u (%u/%u/%u)",
-	    tok2str(op2str, "op-#%u", IGRP_OP(EXTRACT_U_1(hdr->ig_vop))),
-	    IGRP_V(EXTRACT_U_1(hdr->ig_vop)),
-	    EXTRACT_U_1(hdr->ig_ed),
-	    EXTRACT_BE_U_2(hdr->ig_as),
+	nint = GET_BE_U_2(hdr->ig_ni);
+	nsys = GET_BE_U_2(hdr->ig_ns);
+	next = GET_BE_U_2(hdr->ig_nx);
+
+	ND_PRINT(" %s V%u edit=%u AS=%u (%u/%u/%u)",
+	    tok2str(op2str, "op-#%u", IGRP_OP(GET_U_1(hdr->ig_vop))),
+	    IGRP_V(GET_U_1(hdr->ig_vop)),
+	    GET_U_1(hdr->ig_ed),
+	    GET_BE_U_2(hdr->ig_as),
 	    nint,
 	    nsys,
-	    next));
+	    next);
+	cksum = GET_BE_U_2(hdr->ig_sum);
+	if (ndo->ndo_vflag)
+		ND_PRINT(" checksum=0x%04x", cksum);
 
 	length -= sizeof(*hdr);
 	while (length >= IGRP_RTE_SIZE) {
+		const struct igrprte *igr = (const struct igrprte *)cp;
+		uint8_t net0 = GET_U_1(&igr->igr_net[0]);
+		uint8_t net1 = GET_U_1(&igr->igr_net[1]);
+		uint8_t net2 = GET_U_1(&igr->igr_net[2]);
+
 		if (nint > 0) {
-			ND_TCHECK_LEN(cp, IGRP_RTE_SIZE);
-			igrp_entry_print(ndo, (const struct igrprte *)cp, 1, 0);
+			ND_PRINT(" *.%u.%u.%u", net0, net1, net2);
+			igrp_entry_print(ndo, igr);
 			--nint;
 		} else if (nsys > 0) {
-			ND_TCHECK_LEN(cp, IGRP_RTE_SIZE);
-			igrp_entry_print(ndo, (const struct igrprte *)cp, 0, 0);
+			ND_PRINT(" %u.%u.%u.0", net0, net1, net2);
+			igrp_entry_print(ndo, igr);
 			--nsys;
 		} else if (next > 0) {
-			ND_TCHECK_LEN(cp, IGRP_RTE_SIZE);
-			igrp_entry_print(ndo, (const struct igrprte *)cp, 0, 1);
+			ND_PRINT(" X%u.%u.%u.0", net0, net1, net2);
+			igrp_entry_print(ndo, igr);
 			--next;
 		} else {
-			ND_PRINT((ndo, " [extra bytes %u]", length));
+			ND_PRINT(" [extra bytes %u]", length);
 			break;
 		}
 		cp += IGRP_RTE_SIZE;
 		length -= IGRP_RTE_SIZE;
 	}
-	if (nint == 0 && nsys == 0 && next == 0)
-		return;
-trunc:
-	ND_PRINT((ndo, " [|igrp]"));
+	if (nint || nsys || next || length)
+		nd_print_invalid(ndo);
 }