CVE-2016-7985,7986/Change the way protocols print link-layer addresses.

[tcpdump] / print-llc.c

diff --git a/print-llc.c b/print-llc.c
index 99d74c11fe9a172ba6dc3cd6cc8428e4f20b54ca..bca9b5021150c45b36865f8f2ba9cabed25345be 100644 (file)
--- a/print-llc.c
+++ b/print-llc.c
@@ -148,7 +148,7 @@ static const struct oui_tok oui_to_tok[] = {
  */
 int
 llc_print(netdissect_options *ndo, const u_char *p, u_int length, u_int caplen,
-         const u_char *esrc, const u_char *edst)
+         const struct lladdr_info *src, const struct lladdr_info *dst)
 {
        uint8_t dsap_field, dsap, ssap_field, ssap;
        uint16_t control;
@@ -262,7 +262,7 @@ llc_print(netdissect_options *ndo, const u_char *p, u_int length, u_int caplen,
                 * Does anybody ever bridge one form of LAN traffic
                 * over a networking type that uses 802.2 LLC?
                 */
-               if (!snap_print(ndo, p, length, caplen, esrc, edst, 2)) {
+               if (!snap_print(ndo, p, length, caplen, src, dst, 2)) {
                        /*
                         * Unknown packet type; tell our caller, by
                         * returning a negative value, so they
@@ -330,23 +330,23 @@ llc_print(netdissect_options *ndo, const u_char *p, u_int length, u_int caplen,
 
        if (!ndo->ndo_eflag) {
                if (ssap == dsap) {
-                       if (esrc == NULL || edst == NULL)
+                       if (src == NULL || dst == NULL)
                                ND_PRINT((ndo, "%s ", tok2str(llc_values, "Unknown DSAP 0x%02x", dsap)));
                        else
                                ND_PRINT((ndo, "%s > %s %s ",
-                                               etheraddr_string(ndo, esrc),
-                                               etheraddr_string(ndo, edst),
+                                               (src->addr_string)(ndo, src->addr),
+                                               (dst->addr_string)(ndo, dst->addr),
                                                tok2str(llc_values, "Unknown DSAP 0x%02x", dsap)));
                } else {
-                       if (esrc == NULL || edst == NULL)
+                       if (src == NULL || dst == NULL)
                                ND_PRINT((ndo, "%s > %s ",
                                         tok2str(llc_values, "Unknown SSAP 0x%02x", ssap),
                                        tok2str(llc_values, "Unknown DSAP 0x%02x", dsap)));
                        else
                                ND_PRINT((ndo, "%s %s > %s %s ",
-                                       etheraddr_string(ndo, esrc),
+                                       (src->addr_string)(ndo, src->addr),
                                         tok2str(llc_values, "Unknown SSAP 0x%02x", ssap),
-                                       etheraddr_string(ndo, edst),
+                                       (dst->addr_string)(ndo, dst->addr),
                                        tok2str(llc_values, "Unknown DSAP 0x%02x", dsap)));
                }
        }
@@ -399,7 +399,8 @@ oui_to_struct_tok(uint32_t orgcode)
 
 int
 snap_print(netdissect_options *ndo, const u_char *p, u_int length, u_int caplen,
-       const u_char *esrc, const u_char *edst, u_int bridge_pad)
+       const struct lladdr_info *src, const struct lladdr_info *dst,
+       u_int bridge_pad)
 {
        uint32_t orgcode;
        register u_short et;
@@ -437,7 +438,7 @@ snap_print(netdissect_options *ndo, const u_char *p, u_int length, u_int caplen,
                 * Cisco hardware; the protocol ID is
                 * an Ethernet protocol type.
                 */
-               ret = ethertype_print(ndo, et, p, length, caplen);
+               ret = ethertype_print(ndo, et, p, length, caplen, src, dst);
                if (ret)
                        return (ret);
                break;
@@ -452,7 +453,7 @@ snap_print(netdissect_options *ndo, const u_char *p, u_int length, u_int caplen,
                         * but used 0x000000 and an Ethernet
                         * packet type for AARP packets.
                         */
-                       ret = ethertype_print(ndo, et, p, length, caplen);
+                       ret = ethertype_print(ndo, et, p, length, caplen, src, dst);
                        if (ret)
                                return (ret);
                }
@@ -553,13 +554,13 @@ snap_print(netdissect_options *ndo, const u_char *p, u_int length, u_int caplen,
        }
        if (!ndo->ndo_eflag) {
                /*
-                * Nobody printed the MAC addresses, so print them, if
+                * Nobody printed the link-layer addresses, so print them, if
                 * we have any.
                 */
-               if (esrc != NULL && edst != NULL) {
+               if (src != NULL && dst != NULL) {
                        ND_PRINT((ndo, "%s > %s ",
-                               etheraddr_string(ndo, esrc),
-                               etheraddr_string(ndo, edst)));
+                               (src->addr_string)(ndo, src->addr),
+                               (dst->addr_string)(ndo, dst->addr)));
                }
                /*
                 * Print the SNAP header, but if the OUI is 000000, don't