]> The Tcpdump Group git mirrors - tcpdump/blobdiff - print-eap.c
projects / tcpdump / blobdiff
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
the failed/passed count was not kept in the right place
[tcpdump] / print-eap.c
diff --git a/print-eap.c b/print-eap.c
index 827bf6725b0a185bb6934140af31c89f2e0674c8..0009fe2f19eee03bc34bb1856fcc4b4dc10389f5 100644 (file)
--- a/print-eap.c
+++ b/print-eap.c
@@ -21,10 +21,10 @@
 /* \summary: Extensible Authentication Protocol (EAP) printer */
 
 #ifdef HAVE_CONFIG_H
 /* \summary: Extensible Authentication Protocol (EAP) printer */
 
 #ifdef HAVE_CONFIG_H
-#include "config.h"
+#include <config.h>
 #endif
 
 #endif
 
-#include <netdissect-stdinc.h>
+#include "netdissect-stdinc.h"
 
 #include "netdissect.h"
 #include "extract.h"
 
 #include "netdissect.h"
 #include "extract.h"
@@ -36,9 +36,9 @@
 #define        EAP_FRAME_TYPE_ENCAP_ASF_ALERT  4
 
 struct eap_frame_t {
 #define        EAP_FRAME_TYPE_ENCAP_ASF_ALERT  4
 
 struct eap_frame_t {
-    unsigned char   version;
-    unsigned char   type;
-    unsigned char   length[2];
+    nd_uint8_t  version;
+    nd_uint8_t  type;
+    nd_uint16_t length;
 };
 
 static const struct tok eap_frame_type_values[] = {
 };
 
 static const struct tok eap_frame_type_values[] = {
@@ -52,9 +52,9 @@ static const struct tok eap_frame_type_values[] = {
 
 /* RFC 3748 */
 struct eap_packet_t {
 
 /* RFC 3748 */
 struct eap_packet_t {
-    unsigned char      code;
-    unsigned char      id;
-    unsigned char      length[2];
+    nd_uint8_t  code;
+    nd_uint8_t  id;
+    nd_uint16_t length;
 };
 
 #define                EAP_REQUEST     1
 };
 
 #define                EAP_REQUEST     1
@@ -153,66 +153,62 @@ eap_print(netdissect_options *ndo,
 {
     const struct eap_frame_t *eap;
     const u_char *tptr;
 {
     const struct eap_frame_t *eap;
     const u_char *tptr;
-    u_int tlen, type, subtype;
+    u_int eap_type, tlen, type, subtype;
     int count=0, len;
 
     int count=0, len;
 
+    ndo->ndo_protocol = "eap";
     tptr = cp;
     tlen = length;
     eap = (const struct eap_frame_t *)cp;
     tptr = cp;
     tlen = length;
     eap = (const struct eap_frame_t *)cp;
-    ND_TCHECK(*eap);
-
-    /* in non-verbose mode just lets print the basic info */
-    if (ndo->ndo_vflag < 1) {
-       ND_PRINT((ndo, "%s (%u) v%u, len %u",
-               tok2str(eap_frame_type_values, "unknown", eap->type),
-               eap->type,
-               eap->version,
-               EXTRACT_BE_U_2(eap->length)));
-       return;
-    }
+    ND_TCHECK_SIZE(eap);
+    eap_type = GET_U_1(eap->type);
+
+    ND_PRINT("%s (%u) v%u, len %u",
+           tok2str(eap_frame_type_values, "unknown", eap_type),
+           eap_type,
+           GET_U_1(eap->version),
+           GET_BE_U_2(eap->length));
 
 
-    ND_PRINT((ndo, "%s (%u) v%u, len %u",
-           tok2str(eap_frame_type_values, "unknown", eap->type),
-           eap->type,
-           eap->version,
-           EXTRACT_BE_U_2(eap->length)));
+    if (ndo->ndo_vflag < 1)
+        return;
 
     tptr += sizeof(struct eap_frame_t);
     tlen -= sizeof(struct eap_frame_t);
 
 
     tptr += sizeof(struct eap_frame_t);
     tlen -= sizeof(struct eap_frame_t);
 
-    switch (eap->type) {
+    switch (eap_type) {
     case EAP_FRAME_TYPE_PACKET:
         ND_TCHECK_1(tptr);
     case EAP_FRAME_TYPE_PACKET:
         ND_TCHECK_1(tptr);
-        type = EXTRACT_U_1(tptr);
+        type = GET_U_1(tptr);
         ND_TCHECK_2(tptr + 2);
         ND_TCHECK_2(tptr + 2);
-        len = EXTRACT_BE_U_2(tptr + 2);
-        ND_PRINT((ndo, ", %s (%u), id %u, len %u",
+        len = GET_BE_U_2(tptr + 2);
+        ND_PRINT(", %s (%u), id %u, len %u",
                tok2str(eap_code_values, "unknown", type),
                type,
                tok2str(eap_code_values, "unknown", type),
                type,
-               EXTRACT_U_1((tptr + 1)),
-               len));
+               GET_U_1((tptr + 1)),
+               len);
 
         ND_TCHECK_LEN(tptr, len);
 
 
         ND_TCHECK_LEN(tptr, len);
 
-        if (type <= 2) { /* For EAP_REQUEST and EAP_RESPONSE only */
+        if (type == EAP_REQUEST || type == EAP_RESPONSE) {
+            /* RFC 3748 Section 4.1 */
             ND_TCHECK_1(tptr + 4);
             ND_TCHECK_1(tptr + 4);
-            subtype = EXTRACT_U_1(tptr + 4);
-            ND_PRINT((ndo, "\n\t\t Type %s (%u)",
+            subtype = GET_U_1(tptr + 4);
+            ND_PRINT("\n\t\t Type %s (%u)",
                    tok2str(eap_type_values, "unknown", subtype),
                    tok2str(eap_type_values, "unknown", subtype),
-                   subtype));
+                   subtype);
 
             switch (subtype) {
             case EAP_TYPE_IDENTITY:
                 if (len - 5 > 0) {
 
             switch (subtype) {
             case EAP_TYPE_IDENTITY:
                 if (len - 5 > 0) {
-                    ND_PRINT((ndo, ", Identity: "));
-                    safeputs(ndo, tptr + 5, len - 5);
+                    ND_PRINT(", Identity: ");
+                    (void)nd_printzp(ndo, tptr + 5, len - 5, NULL);
                 }
                 break;
 
             case EAP_TYPE_NOTIFICATION:
                 if (len - 5 > 0) {
                 }
                 break;
 
             case EAP_TYPE_NOTIFICATION:
                 if (len - 5 > 0) {
-                    ND_PRINT((ndo, ", Notification: "));
-                    safeputs(ndo, tptr + 5, len - 5);
+                    ND_PRINT(", Notification: ");
+                    (void)nd_printzp(ndo, tptr + 5, len - 5, NULL);
                 }
                 break;
 
                 }
                 break;
 
@@ -226,9 +222,9 @@ eap_print(netdissect_options *ndo,
                  */
                 while (count < len) {
                     ND_TCHECK_1(tptr + count);
                  */
                 while (count < len) {
                     ND_TCHECK_1(tptr + count);
-                    ND_PRINT((ndo, " %s (%u),",
-                           tok2str(eap_type_values, "unknown", EXTRACT_U_1((tptr + count))),
-                           EXTRACT_U_1(tptr + count)));
+                    ND_PRINT(" %s (%u),",
+                           tok2str(eap_type_values, "unknown", GET_U_1((tptr + count))),
+                           GET_U_1(tptr + count));
                     count++;
                 }
                 break;
                     count++;
                 }
                 break;
@@ -237,29 +233,29 @@ eap_print(netdissect_options *ndo,
             case EAP_TYPE_TLS:
                 ND_TCHECK_1(tptr + 5);
                 if (subtype == EAP_TYPE_TTLS)
             case EAP_TYPE_TLS:
                 ND_TCHECK_1(tptr + 5);
                 if (subtype == EAP_TYPE_TTLS)
-                    ND_PRINT((ndo, " TTLSv%u",
-                           EAP_TTLS_VERSION(EXTRACT_U_1((tptr + 5)))));
-                ND_PRINT((ndo, " flags [%s] 0x%02x,",
-                       bittok2str(eap_tls_flags_values, "none", EXTRACT_U_1((tptr + 5))),
-                       EXTRACT_U_1(tptr + 5)));
+                    ND_PRINT(" TTLSv%u",
+                           EAP_TTLS_VERSION(GET_U_1((tptr + 5))));
+                ND_PRINT(" flags [%s] 0x%02x,",
+                       bittok2str(eap_tls_flags_values, "none", GET_U_1((tptr + 5))),
+                       GET_U_1(tptr + 5));
 
 
-                if (EAP_TLS_EXTRACT_BIT_L(EXTRACT_U_1(tptr + 5))) {
+                if (EAP_TLS_EXTRACT_BIT_L(GET_U_1(tptr + 5))) {
                     ND_TCHECK_4(tptr + 6);
                     ND_TCHECK_4(tptr + 6);
-                   ND_PRINT((ndo, " len %u", EXTRACT_BE_U_4(tptr + 6)));
+                   ND_PRINT(" len %u", GET_BE_U_4(tptr + 6));
                 }
                 break;
 
             case EAP_TYPE_FAST:
                 ND_TCHECK_1(tptr + 5);
                 }
                 break;
 
             case EAP_TYPE_FAST:
                 ND_TCHECK_1(tptr + 5);
-                ND_PRINT((ndo, " FASTv%u",
-                       EAP_TTLS_VERSION(EXTRACT_U_1((tptr + 5)))));
-                ND_PRINT((ndo, " flags [%s] 0x%02x,",
-                       bittok2str(eap_tls_flags_values, "none", EXTRACT_U_1((tptr + 5))),
-                       EXTRACT_U_1(tptr + 5)));
+                ND_PRINT(" FASTv%u",
+                       EAP_TTLS_VERSION(GET_U_1((tptr + 5))));
+                ND_PRINT(" flags [%s] 0x%02x,",
+                       bittok2str(eap_tls_flags_values, "none", GET_U_1((tptr + 5))),
+                       GET_U_1(tptr + 5));
 
 
-                if (EAP_TLS_EXTRACT_BIT_L(EXTRACT_U_1(tptr + 5))) {
+                if (EAP_TLS_EXTRACT_BIT_L(GET_U_1(tptr + 5))) {
                     ND_TCHECK_4(tptr + 6);
                     ND_TCHECK_4(tptr + 6);
-                    ND_PRINT((ndo, " len %u", EXTRACT_BE_U_4(tptr + 6)));
+                    ND_PRINT(" len %u", GET_BE_U_4(tptr + 6));
                 }
 
                 /* FIXME - TLV attributes follow */
                 }
 
                 /* FIXME - TLV attributes follow */
@@ -268,9 +264,9 @@ eap_print(netdissect_options *ndo,
             case EAP_TYPE_AKA:
             case EAP_TYPE_SIM:
                 ND_TCHECK_1(tptr + 5);
             case EAP_TYPE_AKA:
             case EAP_TYPE_SIM:
                 ND_TCHECK_1(tptr + 5);
-                ND_PRINT((ndo, " subtype [%s] 0x%02x,",
-                       tok2str(eap_aka_subtype_values, "unknown", EXTRACT_U_1((tptr + 5))),
-                       EXTRACT_U_1(tptr + 5)));
+                ND_PRINT(" subtype [%s] 0x%02x,",
+                       tok2str(eap_aka_subtype_values, "unknown", GET_U_1((tptr + 5))),
+                       GET_U_1(tptr + 5));
 
                 /* FIXME - TLV attributes follow */
                 break;
 
                 /* FIXME - TLV attributes follow */
                 break;
@@ -294,11 +290,5 @@ eap_print(netdissect_options *ndo,
     return;
 
  trunc:
     return;
 
  trunc:
-    ND_PRINT((ndo, "\n\t[|EAP]"));
+    nd_print_trunc(ndo);
 }
 }
-
-/*
- * Local Variables:
- * c-basic-offset: 4
- * End:
- */
[mirror] the TCPdump network dissector