Try marking structures as UNALIGNED.

[tcpdump] / print-arp.c

diff --git a/print-arp.c b/print-arp.c
index fbc8411ef817fe9e114b25343091153f05cef71b..96727faa31f3309d09b1a093741ce35f641b7efc 100644 (file)
--- a/print-arp.c
+++ b/print-arp.c
@@ -19,20 +19,21 @@
  * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
  * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 

+/* \summary: Address Resolution Protocol (ARP) printer */
+

 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
 

-#include <tcpdump-stdinc.h>
+#include <netdissect-stdinc.h>

 
 

-#include <stdio.h>

 #include <string.h>
 
 #include "netdissect.h"
 #include "addrtoname.h"
 #include "ether.h"
 #include "ethertype.h"
 #include <string.h>
 
 #include "netdissect.h"
 #include "addrtoname.h"
 #include "ether.h"
 #include "ethertype.h"

-#include "extract.h"                   /* must come after interface.h */
+#include "extract.h"

 
 static const char tstr[] = "[|ARP]";
 
 
 static const char tstr[] = "[|ARP]";
 


@@ -77,7 +78,7 @@ struct  arp_pkthdr {
        u_char  ar_tha[];       /* target hardware address */
        u_char  ar_tpa[];       /* target protocol address */
 #endif
        u_char  ar_tha[];       /* target hardware address */
        u_char  ar_tpa[];       /* target protocol address */
 #endif

-#define ar_sha(ap)     (((const u_char *)((ap)+1))+0)
+#define ar_sha(ap)     (((const u_char *)((ap)+1))+  0)

 #define ar_spa(ap)     (((const u_char *)((ap)+1))+  (ap)->ar_hln)
 #define ar_tha(ap)     (((const u_char *)((ap)+1))+  (ap)->ar_hln+(ap)->ar_pln)
 #define ar_tpa(ap)     (((const u_char *)((ap)+1))+2*(ap)->ar_hln+(ap)->ar_pln)
 #define ar_spa(ap)     (((const u_char *)((ap)+1))+  (ap)->ar_hln)
 #define ar_tha(ap)     (((const u_char *)((ap)+1))+  (ap)->ar_hln+(ap)->ar_pln)
 #define ar_tpa(ap)     (((const u_char *)((ap)+1))+2*(ap)->ar_hln+(ap)->ar_pln)


@@ -176,7 +177,41 @@ struct  atmarp_pkthdr {
 #define ATMTSA(ap) (aar_tsa(ap))
 #define ATMTPA(ap) (aar_tpa(ap))
 
 #define ATMTSA(ap) (aar_tsa(ap))
 #define ATMTPA(ap) (aar_tpa(ap))
 

-static u_char ezero[6];
+static int
+isnonzero(const u_char *a, size_t len)
+{
+       while (len > 0) {
+               if (*a != 0)
+                       return (1);
+               a++;
+               len--;
+       }
+       return (0);
+}
+
+static void
+tpaddr_print_ip(netdissect_options *ndo,
+               const struct arp_pkthdr *ap, u_short pro)
+{
+       if (pro != ETHERTYPE_IP && pro != ETHERTYPE_TRAIL)
+               ND_PRINT((ndo, "<wrong proto type>"));
+       else if (PROTO_LEN(ap) != 4)
+               ND_PRINT((ndo, "<wrong len>"));
+       else
+               ND_PRINT((ndo, "%s", ipaddr_string(ndo, TPA(ap))));
+}
+
+static void
+spaddr_print_ip(netdissect_options *ndo,
+               const struct arp_pkthdr *ap, u_short pro)
+{
+       if (pro != ETHERTYPE_IP && pro != ETHERTYPE_TRAIL)
+               ND_PRINT((ndo, "<wrong proto type>"));
+       else if (PROTO_LEN(ap) != 4)
+               ND_PRINT((ndo, "<wrong len>"));
+       else
+               ND_PRINT((ndo, "%s", ipaddr_string(ndo, SPA(ap))));
+}

 
 static void
 atmarp_addr_print(netdissect_options *ndo,
 
 static void
 atmarp_addr_print(netdissect_options *ndo,


@@ -186,13 +221,37 @@ atmarp_addr_print(netdissect_options *ndo,
        if (ha_len == 0)
                ND_PRINT((ndo, "<No address>"));
        else {
        if (ha_len == 0)
                ND_PRINT((ndo, "<No address>"));
        else {

-               ND_PRINT((ndo, "%s", linkaddr_string(ha, LINKADDR_ATM, ha_len)));
+               ND_PRINT((ndo, "%s", linkaddr_string(ndo, ha, LINKADDR_ATM, ha_len)));

                if (srca_len != 0)
                        ND_PRINT((ndo, ",%s",
                if (srca_len != 0)
                        ND_PRINT((ndo, ",%s",

-                                 linkaddr_string(srca, LINKADDR_ATM, srca_len)));
+                                 linkaddr_string(ndo, srca, LINKADDR_ATM, srca_len)));

        }
 }
 
        }
 }
 

+static void
+atmarp_tpaddr_print(netdissect_options *ndo,
+                   const struct atmarp_pkthdr *ap, u_short pro)
+{
+       if (pro != ETHERTYPE_IP && pro != ETHERTYPE_TRAIL)
+               ND_PRINT((ndo, "<wrong proto type>"));
+       else if (ATMTPROTO_LEN(ap) != 4)
+               ND_PRINT((ndo, "<wrong tplen>"));
+       else
+               ND_PRINT((ndo, "%s", ipaddr_string(ndo, ATMTPA(ap))));
+}
+
+static void
+atmarp_spaddr_print(netdissect_options *ndo,
+                   const struct atmarp_pkthdr *ap, u_short pro)
+{
+       if (pro != ETHERTYPE_IP && pro != ETHERTYPE_TRAIL)
+               ND_PRINT((ndo, "<wrong proto type>"));
+       else if (ATMSPROTO_LEN(ap) != 4)
+               ND_PRINT((ndo, "<wrong splen>"));
+       else
+               ND_PRINT((ndo, "%s", ipaddr_string(ndo, ATMSPA(ap))));
+}
+

 static void
 atmarp_print(netdissect_options *ndo,
             const u_char *bp, u_int length, u_int caplen)
 static void
 atmarp_print(netdissect_options *ndo,
             const u_char *bp, u_int length, u_int caplen)


@@ -234,25 +293,28 @@ atmarp_print(netdissect_options *ndo,
        }
 
         /* print operation */
        }
 
         /* print operation */

-        printf("%s%s ",
+        ND_PRINT((ndo, "%s%s ",

                ndo->ndo_vflag ? ", " : "",
                ndo->ndo_vflag ? ", " : "",

-               tok2str(arpop_values, "Unknown (%u)", op));
+               tok2str(arpop_values, "Unknown (%u)", op)));

 
        switch (op) {
 
        case ARPOP_REQUEST:
 
        switch (op) {
 
        case ARPOP_REQUEST:

-               ND_PRINT((ndo, "who-has %s", ipaddr_string(ATMTPA(ap))));
+               ND_PRINT((ndo, "who-has "));
+               atmarp_tpaddr_print(ndo, ap, pro);

                if (ATMTHRD_LEN(ap) != 0) {
                        ND_PRINT((ndo, " ("));
                        atmarp_addr_print(ndo, ATMTHA(ap), ATMTHRD_LEN(ap),
                            ATMTSA(ap), ATMTSLN(ap));
                        ND_PRINT((ndo, ")"));
                }
                if (ATMTHRD_LEN(ap) != 0) {
                        ND_PRINT((ndo, " ("));
                        atmarp_addr_print(ndo, ATMTHA(ap), ATMTHRD_LEN(ap),
                            ATMTSA(ap), ATMTSLN(ap));
                        ND_PRINT((ndo, ")"));
                }

-               ND_PRINT((ndo, "tell %s", ipaddr_string(ATMSPA(ap))));
+               ND_PRINT((ndo, " tell "));
+               atmarp_spaddr_print(ndo, ap, pro);

                break;
 
        case ARPOP_REPLY:
                break;
 
        case ARPOP_REPLY:

-               ND_PRINT((ndo, "%s is-at ", ipaddr_string(ATMSPA(ap))));
+               atmarp_spaddr_print(ndo, ap, pro);
+               ND_PRINT((ndo, " is-at "));

                atmarp_addr_print(ndo, ATMSHA(ap), ATMSHRD_LEN(ap), ATMSSA(ap),
                                   ATMSSLN(ap));
                break;
                atmarp_addr_print(ndo, ATMSHA(ap), ATMSHRD_LEN(ap), ATMSSA(ap),
                                   ATMSSLN(ap));
                break;


@@ -269,11 +331,13 @@ atmarp_print(netdissect_options *ndo,
        case ARPOP_INVREPLY:
                atmarp_addr_print(ndo, ATMSHA(ap), ATMSHRD_LEN(ap), ATMSSA(ap),
                    ATMSSLN(ap));
        case ARPOP_INVREPLY:
                atmarp_addr_print(ndo, ATMSHA(ap), ATMSHRD_LEN(ap), ATMSSA(ap),
                    ATMSSLN(ap));

-               ND_PRINT((ndo, "at %s", ipaddr_string(ATMSPA(ap))));
+               ND_PRINT((ndo, "at "));
+               atmarp_spaddr_print(ndo, ap, pro);

                break;
 
        case ARPOP_NAK:
                break;
 
        case ARPOP_NAK:

-               ND_PRINT((ndo, "for %s", ipaddr_string(ATMSPA(ap))));
+               ND_PRINT((ndo, "for "));
+               atmarp_spaddr_print(ndo, ap, pro);

                break;
 
        default:
                break;
 
        default:


@@ -307,7 +371,7 @@ arp_print(netdissect_options *ndo,
         /* if its ATM then call the ATM ARP printer
            for Frame-relay ARP most of the fields
            are similar to Ethernet so overload the Ethernet Printer
         /* if its ATM then call the ATM ARP printer
            for Frame-relay ARP most of the fields
            are similar to Ethernet so overload the Ethernet Printer

-           and set the linkaddr type for linkaddr_string() accordingly */
+           and set the linkaddr type for linkaddr_string(ndo, ) accordingly */

 
         switch(hrd) {
         case ARPHRD_ATM2225:
 
         switch(hrd) {
         case ARPHRD_ATM2225:


@@ -321,7 +385,7 @@ arp_print(netdissect_options *ndo,
             break;
        }
 
             break;
        }
 

-       if (!ND_TTEST2(*ar_tpa(ap), PROTO_LEN(ap))) {
+       if (!ND_TTEST2(*TPA(ap), PROTO_LEN(ap))) {

                ND_PRINT((ndo, "%s", tstr));
                ND_DEFAULTPRINT((const u_char *)ap, length);
                return;
                ND_PRINT((ndo, "%s", tstr));
                ND_DEFAULTPRINT((const u_char *)ap, length);
                return;


@@ -349,48 +413,50 @@ arp_print(netdissect_options *ndo,
        }
 
         /* print operation */
        }
 
         /* print operation */

-        printf("%s%s ",
+        ND_PRINT((ndo, "%s%s ",

                ndo->ndo_vflag ? ", " : "",
                ndo->ndo_vflag ? ", " : "",

-               tok2str(arpop_values, "Unknown (%u)", op));
+               tok2str(arpop_values, "Unknown (%u)", op)));

 
        switch (op) {
 
        case ARPOP_REQUEST:
 
        switch (op) {
 
        case ARPOP_REQUEST:

-               ND_PRINT((ndo, "who-has %s", ipaddr_string(TPA(ap))));
-               if (memcmp((const char *)ezero, (const char *)THA(ap), HRD_LEN(ap)) != 0)
+               ND_PRINT((ndo, "who-has "));
+               tpaddr_print_ip(ndo, ap, pro);
+               if (isnonzero((const u_char *)THA(ap), HRD_LEN(ap)))

                        ND_PRINT((ndo, " (%s)",
                        ND_PRINT((ndo, " (%s)",

-                                 linkaddr_string(THA(ap), linkaddr, HRD_LEN(ap))));
-               ND_PRINT((ndo, " tell %s", ipaddr_string(SPA(ap))));
+                                 linkaddr_string(ndo, THA(ap), linkaddr, HRD_LEN(ap))));
+               ND_PRINT((ndo, " tell "));
+               spaddr_print_ip(ndo, ap, pro);

                break;
 
        case ARPOP_REPLY:
                break;
 
        case ARPOP_REPLY:

-               ND_PRINT((ndo, "%s is-at %s",
-                          ipaddr_string(SPA(ap)),
-                          linkaddr_string(SHA(ap), linkaddr, HRD_LEN(ap))));
+               spaddr_print_ip(ndo, ap, pro);
+               ND_PRINT((ndo, " is-at %s",
+                          linkaddr_string(ndo, SHA(ap), linkaddr, HRD_LEN(ap))));

                break;
 
        case ARPOP_REVREQUEST:
                ND_PRINT((ndo, "who-is %s tell %s",
                break;
 
        case ARPOP_REVREQUEST:
                ND_PRINT((ndo, "who-is %s tell %s",

-                         linkaddr_string(THA(ap), linkaddr, HRD_LEN(ap)),
-                         linkaddr_string(SHA(ap), linkaddr, HRD_LEN(ap))));
+                         linkaddr_string(ndo, THA(ap), linkaddr, HRD_LEN(ap)),
+                         linkaddr_string(ndo, SHA(ap), linkaddr, HRD_LEN(ap))));

                break;
 
        case ARPOP_REVREPLY:
                break;
 
        case ARPOP_REVREPLY:

-               ND_PRINT((ndo, "%s at %s",
-                         linkaddr_string(THA(ap), linkaddr, HRD_LEN(ap)),
-                         ipaddr_string(TPA(ap))));
+               ND_PRINT((ndo, "%s at ",
+                         linkaddr_string(ndo, THA(ap), linkaddr, HRD_LEN(ap))));
+               tpaddr_print_ip(ndo, ap, pro);

                break;
 
        case ARPOP_INVREQUEST:
                ND_PRINT((ndo, "who-is %s tell %s",
                break;
 
        case ARPOP_INVREQUEST:
                ND_PRINT((ndo, "who-is %s tell %s",

-                         linkaddr_string(THA(ap), linkaddr, HRD_LEN(ap)),
-                         linkaddr_string(SHA(ap), linkaddr, HRD_LEN(ap))));
+                         linkaddr_string(ndo, THA(ap), linkaddr, HRD_LEN(ap)),
+                         linkaddr_string(ndo, SHA(ap), linkaddr, HRD_LEN(ap))));

                break;
 
        case ARPOP_INVREPLY:
                break;
 
        case ARPOP_INVREPLY:

-               ND_PRINT((ndo,"%s at %s",
-                         linkaddr_string(THA(ap), linkaddr, HRD_LEN(ap)),
-                         ipaddr_string(TPA(ap))));
+               ND_PRINT((ndo,"%s at ",
+                         linkaddr_string(ndo, SHA(ap), linkaddr, HRD_LEN(ap))));
+               spaddr_print_ip(ndo, ap, pro);

                break;
 
        default:
                break;
 
        default: