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

diff --git a/print-beep.c b/print-beep.c
index 4c5bc84a..76017eaf 100644
--- a/print-beep.c
+++ b/print-beep.c
@@ -2,22 +2,24 @@
  * Copyright (C) 2000, Richard Sharpe
  *
  * This software may be distributed either under the terms of the
- * BSD-style licence that accompanies tcpdump or under the GNU GPL
+ * BSD-style license that accompanies tcpdump or under the GNU GPL
  * version 2 or later.
  *
  * print-beep.c
  *
  */
 
+/* \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>
 
-#include "interface.h"
+#include "netdissect.h"
 
 /* Check for a string but not go beyond length
  * Return TRUE on match, FALSE otherwise
@@ -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)");
 }