X-Git-Url: https://git.tcpdump.org/tcpdump/blobdiff_plain/27d428c0a126e9ca6518b16f011cacc1caab1f2c..296d466cd6bbf2f7e75e15bb6a01268e88c76ed0:/print-timed.c

diff --git a/print-timed.c b/print-timed.c
index 40908dd0..6733ef1b 100644
--- a/print-timed.c
+++ b/print-timed.c
@@ -19,41 +19,31 @@
  * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
-#ifndef lint
-static const char rcsid[] _U_ =
-    "@(#) $Header: /tcpdump/master/tcpdump/print-timed.c,v 1.9 2003-11-16 09:36:40 guy Exp $";
-#endif
+/* \summary: Berkeley UNIX Time Synchronization Protocol */
 
-#ifdef HAVE_CONFIG_H
-#include "config.h"
-#endif
+/* specification: https://docs.freebsd.org/44doc/smm/12.timed/paper.pdf */
 
-#include <tcpdump-stdinc.h>
+#include <config.h>
 
-#include <stdio.h>
-#include <string.h>
+#include "netdissect-stdinc.h"
 
-#include "interface.h"
+#include "netdissect.h"
 #include "extract.h"
 
-/*
- * Time Synchronization Protocol
- */
-
 struct tsp_timeval {
-	u_int32_t	tv_sec;
-	u_int32_t	tv_usec;
+	nd_int32_t	tv_sec;
+	nd_int32_t	tv_usec;
 };
 
 struct tsp {
-	u_int8_t	tsp_type;
-	u_int8_t	tsp_vers;
-	u_int16_t	tsp_seq;
+	nd_uint8_t	tsp_type;
+	nd_uint8_t	tsp_vers;
+	nd_uint16_t	tsp_seq;
 	union {
 		struct tsp_timeval tspu_time;
-		int8_t tspu_hopcnt;
+		nd_int8_t tspu_hopcnt;
 	} tsp_u;
-	int8_t tsp_name[256];
+	nd_byte		tsp_name[256];	/* null-terminated string up to 256 */
 };
 
 #define	tsp_time	tsp_u.tspu_time
@@ -87,80 +77,75 @@ struct tsp {
 #define	TSP_SETDATE		22	/* New from date command */
 #define	TSP_SETDATEREQ		23	/* New remote for above */
 #define	TSP_LOOP		24	/* loop detection packet */
-
-#define	TSPTYPENUMBER		25
-
-static const char tstr[] = "[|timed]";
-
-static const char *tsptype[TSPTYPENUMBER] =
-  { "ANY", "ADJTIME", "ACK", "MASTERREQ", "MASTERACK", "SETTIME", "MASTERUP",
-  "SLAVEUP", "ELECTION", "ACCEPT", "REFUSE", "CONFLICT", "RESOLVE", "QUIT",
-  "DATE", "DATEREQ", "DATEACK", "TRACEON", "TRACEOFF", "MSITE", "MSITEREQ",
-  "TEST", "SETDATE", "SETDATEREQ", "LOOP" };
+static const struct tok tsptype_str[] = {
+	{ TSP_ANY,        "TSP_ANY"        },
+	{ TSP_ADJTIME,    "TSP_ADJTIME"    },
+	{ TSP_ACK,        "TSP_ACK"        },
+	{ TSP_MASTERREQ,  "TSP_MASTERREQ"  },
+	{ TSP_MASTERACK,  "TSP_MASTERACK"  },
+	{ TSP_SETTIME,    "TSP_SETTIME"    },
+	{ TSP_MASTERUP,   "TSP_MASTERUP"   },
+	{ TSP_SLAVEUP,    "TSP_SLAVEUP"    },
+	{ TSP_ELECTION,   "TSP_ELECTION"   },
+	{ TSP_ACCEPT,     "TSP_ACCEPT"     },
+	{ TSP_REFUSE,     "TSP_REFUSE"     },
+	{ TSP_CONFLICT,   "TSP_CONFLICT"   },
+	{ TSP_RESOLVE,    "TSP_RESOLVE"    },
+	{ TSP_QUIT,       "TSP_QUIT"       },
+	{ TSP_DATE,       "TSP_DATE"       },
+	{ TSP_DATEREQ,    "TSP_DATEREQ"    },
+	{ TSP_DATEACK,    "TSP_DATEACK"    },
+	{ TSP_TRACEON,    "TSP_TRACEON"    },
+	{ TSP_TRACEOFF,   "TSP_TRACEOFF"   },
+	{ TSP_MSITE,      "TSP_MSITE"      },
+	{ TSP_MSITEREQ,   "TSP_MSITEREQ"   },
+	{ TSP_TEST,       "TSP_TEST"       },
+	{ TSP_SETDATE,    "TSP_SETDATE"    },
+	{ TSP_SETDATEREQ, "TSP_SETDATEREQ" },
+	{ TSP_LOOP,       "TSP_LOOP"       },
+	{ 0, NULL }
+};
 
 void
-timed_print(register const u_char *bp)
+timed_print(netdissect_options *ndo,
+            const u_char *bp)
 {
-#define endof(x) ((u_char *)&(x) + sizeof (x))
-	struct tsp *tsp = (struct tsp *)bp;
-	long sec, usec;
-	const u_char *end;
-
-	if (endof(tsp->tsp_type) > snapend) {
-		printf("%s", tstr);
-		return;
-	}
-	if (tsp->tsp_type < TSPTYPENUMBER)
-		printf("TSP_%s", tsptype[tsp->tsp_type]);
-	else
-		printf("(tsp_type %#x)", tsp->tsp_type);
-
-	if (endof(tsp->tsp_vers) > snapend) {
-		printf(" %s", tstr);
-		return;
-	}
-	printf(" vers %d", tsp->tsp_vers);
-
-	if (endof(tsp->tsp_seq) > snapend) {
-		printf(" %s", tstr);
-		return;
-	}
-	printf(" seq %d", tsp->tsp_seq);
-
-	if (tsp->tsp_type == TSP_LOOP) {
-		if (endof(tsp->tsp_hopcnt) > snapend) {
-			printf(" %s", tstr);
-			return;
-		}
-		printf(" hopcnt %d", tsp->tsp_hopcnt);
-	} else if (tsp->tsp_type == TSP_SETTIME ||
-	  tsp->tsp_type == TSP_ADJTIME ||
-	  tsp->tsp_type == TSP_SETDATE ||
-	  tsp->tsp_type == TSP_SETDATEREQ) {
-		if (endof(tsp->tsp_time) > snapend) {
-			printf(" %s", tstr);
-			return;
-		}
-		sec = EXTRACT_32BITS(&tsp->tsp_time.tv_sec);
-		usec = EXTRACT_32BITS(&tsp->tsp_time.tv_usec);
+	const struct tsp *tsp = (const struct tsp *)bp;
+	uint8_t tsp_type;
+	int sec, usec;
+
+	ndo->ndo_protocol = "timed";
+	tsp_type = GET_U_1(tsp->tsp_type);
+	ND_PRINT("%s", tok2str(tsptype_str, "(tsp_type %#x)", tsp_type));
+
+	ND_PRINT(" vers %u", GET_U_1(tsp->tsp_vers));
+
+	ND_PRINT(" seq %u", GET_BE_U_2(tsp->tsp_seq));
+
+	switch (tsp_type) {
+	case TSP_LOOP:
+		ND_PRINT(" hopcnt %u", GET_U_1(tsp->tsp_hopcnt));
+		break;
+	case TSP_SETTIME:
+	case TSP_ADJTIME:
+	case TSP_SETDATE:
+	case TSP_SETDATEREQ:
+		sec = GET_BE_S_4(tsp->tsp_time.tv_sec);
+		usec = GET_BE_S_4(tsp->tsp_time.tv_usec);
+		/* XXX The comparison below is always false? */
 		if (usec < 0)
-			/* corrupt, skip the rest of the packet */
+			/* invalid, skip the rest of the packet */
 			return;
-		fputs(" time ", stdout);
+		ND_PRINT(" time ");
 		if (sec < 0 && usec != 0) {
 			sec++;
 			if (sec == 0)
-				fputc('-', stdout);
+				ND_PRINT("-");
 			usec = 1000000 - usec;
 		}
-		printf("%ld.%06ld", sec, usec);
-	}
-
-	end = memchr(tsp->tsp_name, '\0', snapend - (u_char *)tsp->tsp_name);
-	if (end == NULL)
-		printf(" %s", tstr);
-	else {
-		fputs(" name ", stdout);
-		fwrite(tsp->tsp_name, end - (u_char *)tsp->tsp_name, 1, stdout);
+		ND_PRINT("%d.%06d", sec, usec);
+		break;
 	}
+	ND_PRINT(" name ");
+	nd_printjnp(ndo, tsp->tsp_name, sizeof(tsp->tsp_name));
 }