X-Git-Url: https://git.tcpdump.org/tcpdump/blobdiff_plain/fb2479d733dbe8a991d11909b319341f7db62ab1..18905f7312f9117e63cbe4b8cc9d7b00d3da058d:/print-null.c

diff --git a/print-null.c b/print-null.c
index b859c831..f1067ff5 100644
--- a/print-null.c
+++ b/print-null.c
@@ -30,8 +30,11 @@
 #include <string.h>
 
 #include "netdissect.h"
+#include "extract.h"
 #include "af.h"
 
+static const char tstr[] = " [|null]";
+
 /*
  * The DLT_NULL packet header is 4 bytes long. It contains a host-byte-order
  * 32-bit integer that specifies the family, e.g. AF_INET.
@@ -52,7 +55,7 @@
 #define	SWAPLONG(y) \
 ((((y)&0xff)<<24) | (((y)&0xff00)<<8) | (((y)&0xff0000)>>8) | (((y)>>24)&0xff))
 
-static inline void
+static void
 null_hdr_print(netdissect_options *ndo, u_int family, u_int length)
 {
 	if (!ndo->ndo_qflag) {
@@ -77,13 +80,12 @@ null_if_print(netdissect_options *ndo, const struct pcap_pkthdr *h, const u_char
 {
 	u_int length = h->len;
 	u_int caplen = h->caplen;
-	u_int family;
+	uint32_t family;
 
-	if (caplen < NULL_HDRLEN) {
-		ND_PRINT("[|null]");
-		return (NULL_HDRLEN);
-	}
+	if (caplen < NULL_HDRLEN)
+		goto trunc;
 
+	ND_TCHECK_4(p);
 	memcpy((char *)&family, (const char *)p, sizeof(family));
 
 	/*
@@ -136,6 +138,9 @@ null_if_print(netdissect_options *ndo, const struct pcap_pkthdr *h, const u_char
 			ND_DEFAULTPRINT(p, caplen);
 	}
 
+	return (NULL_HDRLEN);
+trunc:
+	ND_PRINT("%s", tstr);
 	return (NULL_HDRLEN);
 }