X-Git-Url: https://git.tcpdump.org/tcpdump/blobdiff_plain/cca75c770451e027e2ee7bc4c087bc29c62cdf3a..refs/pull/1034/head:/print-beep.c

diff --git a/print-beep.c b/print-beep.c
index c0f6d524..76017eaf 100644
--- a/print-beep.c
+++ b/print-beep.c
@@ -9,11 +9,13 @@
  *
  */
 
+/* \summary: Blocks Extensible Exchange Protocol (BEEP) printer */
+
 #ifdef HAVE_CONFIG_H
-#include "config.h"
+#include <config.h>
 #endif
 
-#include <tcpdump-stdinc.h>
+#include "netdissect-stdinc.h"
 
 #include <string.h>
 
@@ -26,9 +28,17 @@
  */
 
 static int
-l_strnstart(const char *tstr1, u_int tl1, const char *str2, u_int l2)
+l_strnstart(netdissect_options *ndo, const char *tstr1, u_int tl1,
+    const char *str2, u_int l2)
 {
-
+	if (!ND_TTEST_LEN(str2, tl1)) {
+		/*
+		 * We don't have tl1 bytes worth of captured data
+		 * for the string, so we can't check for this
+		 * string.
+		 */
+		return 0;
+	}
 	if (tl1 > l2)
 		return 0;
 
@@ -39,20 +49,21 @@ void
 beep_print(netdissect_options *ndo, const u_char *bp, u_int length)
 {
 
-	if (l_strnstart("MSG", 4, (const char *)bp, length)) /* A REQuest */
-		ND_PRINT((ndo, " BEEP MSG"));
-	else if (l_strnstart("RPY ", 4, (const char *)bp, length))
-		ND_PRINT((ndo, " BEEP RPY"));
-	else if (l_strnstart("ERR ", 4, (const char *)bp, length))
-		ND_PRINT((ndo, " BEEP ERR"));
-	else if (l_strnstart("ANS ", 4, (const char *)bp, length))
-		ND_PRINT((ndo, " BEEP ANS"));
-	else if (l_strnstart("NUL ", 4, (const char *)bp, length))
-		ND_PRINT((ndo, " BEEP NUL"));
-	else if (l_strnstart("SEQ ", 4, (const char *)bp, length))
-		ND_PRINT((ndo, " BEEP SEQ"));
-	else if (l_strnstart("END", 4, (const char *)bp, length))
-		ND_PRINT((ndo, " BEEP END"));
+	ndo->ndo_protocol = "beep";
+	if (l_strnstart(ndo, "MSG", 4, (const char *)bp, length)) /* A REQuest */
+		ND_PRINT(" BEEP MSG");
+	else if (l_strnstart(ndo, "RPY ", 4, (const char *)bp, length))
+		ND_PRINT(" BEEP RPY");
+	else if (l_strnstart(ndo, "ERR ", 4, (const char *)bp, length))
+		ND_PRINT(" BEEP ERR");
+	else if (l_strnstart(ndo, "ANS ", 4, (const char *)bp, length))
+		ND_PRINT(" BEEP ANS");
+	else if (l_strnstart(ndo, "NUL ", 4, (const char *)bp, length))
+		ND_PRINT(" BEEP NUL");
+	else if (l_strnstart(ndo, "SEQ ", 4, (const char *)bp, length))
+		ND_PRINT(" BEEP SEQ");
+	else if (l_strnstart(ndo, "END", 4, (const char *)bp, length))
+		ND_PRINT(" BEEP END");
 	else
-		ND_PRINT((ndo, " BEEP (payload or undecoded)"));
+		ND_PRINT(" BEEP (payload or undecoded)");
 }