CI: Add warning exemptions for Sun C (suncc-5.14) on Solaris 10

[tcpdump] / print-atm.c

diff --git a/print-atm.c b/print-atm.c
index 4e19366a2a41f25572f7412956addf52cbc0036c..2dee4e65e97a92d010b5465bc86a18b3a066ba57 100644 (file)
--- a/print-atm.c
+++ b/print-atm.c
@@ -21,12 +21,11 @@
 
 /* \summary: Asynchronous Transfer Mode (ATM) printer */
 
-#ifdef HAVE_CONFIG_H
 #include <config.h>
-#endif
 
 #include "netdissect-stdinc.h"
 
+#define ND_LONGJMP_FROM_TCHECK
 #include "netdissect.h"
 #include "extract.h"
 #include "addrtoname.h"
@@ -105,7 +104,7 @@
 #define B_BEARER               0x5e    /* broadband bearer capability */
 #define B_LOWER                        0x5f    /* broadband lower information */
 #define CALLING_PARTY          0x6c    /* calling party number */
-#define CALLED_PARTY           0x70    /* called party nmber */
+#define CALLED_PARTY           0x70    /* called party number */
 
 #define Q2931                  0x09
 
@@ -113,13 +112,15 @@
 #define PROTO_POS       0      /* offset of protocol discriminator */
 #define CALL_REF_POS    2      /* offset of call reference value */
 #define MSG_TYPE_POS    5      /* offset of message type */
-#define MSG_LEN_POS     7      /* offset of mesage length */
+#if 0
+#define MSG_LEN_POS     7      /* offset of message length */
 #define IE_BEGIN_POS    9      /* offset of first information element */
 
 /* format of signalling messages */
 #define TYPE_POS       0
 #define LEN_POS                2
 #define FIELD_BEGIN_POS 4
+#endif
 
 /* end of the original atmuni31.h */
 
@@ -193,23 +194,11 @@ static const struct tok oam_fm_loopback_indicator_values[] = {
     { 0, NULL }
 };
 
-static const struct tok *oam_functype_values[16] = {
-    NULL,
-    oam_fm_functype_values, /* 1 */
-    oam_pm_functype_values, /* 2 */
-    NULL,
-    NULL,
-    NULL,
-    NULL,
-    NULL,
-    oam_ad_functype_values, /* 8 */
-    NULL,
-    NULL,
-    NULL,
-    NULL,
-    NULL,
-    NULL,
-    NULL
+static const struct uint_tokary oam_celltype2tokary[] = {
+       { OAM_CELLTYPE_FM, oam_fm_functype_values },
+       { OAM_CELLTYPE_PM, oam_pm_functype_values },
+       { OAM_CELLTYPE_AD, oam_ad_functype_values },
+       /* uint2tokary() does not use array termination. */
 };
 
 /*
@@ -253,18 +242,13 @@ atm_if_print(netdissect_options *ndo,
        u_int hdrlen = 0;
 
        ndo->ndo_protocol = "atm";
-       if (caplen < 1) {
-               nd_print_trunc(ndo);
-               ndo->ndo_ll_hdr_len += caplen;
-               return;
-       }
 
         /* Cisco Style NLPID ? */
         if (GET_U_1(p) == LLC_UI) {
             if (ndo->ndo_eflag)
                 ND_PRINT("CNLPID ");
+            ndo->ndo_ll_hdr_len += 1;
             isoclns_print(ndo, p + 1, length - 1);
-            ndo->ndo_ll_hdr_len += hdrlen;
             return;
         }
 
@@ -272,11 +256,6 @@ atm_if_print(netdissect_options *ndo,
         * Must have at least a DSAP, an SSAP, and the first byte of the
         * control field.
         */
-       if (caplen < 3) {
-               nd_print_trunc(ndo);
-               ndo->ndo_ll_hdr_len += caplen;
-               return;
-       }
 
        /*
         * Extract the presumed LLC header into a variable, for quick
@@ -305,24 +284,21 @@ atm_if_print(netdissect_options *ndo,
                 * packets?  If so, could it be changed to use a
                 * new DLT_IEEE802_6 value if we added it?
                 */
-               if (caplen < 20) {
-                       nd_print_trunc(ndo);
-                       ndo->ndo_ll_hdr_len += caplen;
-                       return;
-               }
                if (ndo->ndo_eflag)
                        ND_PRINT("%08x%08x %08x%08x ",
                               GET_BE_U_4(p),
                               GET_BE_U_4(p + 4),
                               GET_BE_U_4(p + 8),
                               GET_BE_U_4(p + 12));
+               /* Always cover the full header. */
+               ND_TCHECK_LEN(p, 20);
                p += 20;
                length -= 20;
                caplen -= 20;
                hdrlen += 20;
        }
-       hdrlen += atm_llc_print(ndo, p, length, caplen);
        ndo->ndo_ll_hdr_len += hdrlen;
+       ndo->ndo_ll_hdr_len += atm_llc_print(ndo, p, length, caplen);
 }
 
 /*
@@ -452,6 +428,7 @@ oam_print(netdissect_options *ndo,
     uint32_t cell_header;
     uint16_t vpi, vci, cksum, cksum_shouldbe, idx;
     uint8_t  cell_type, func_type, payload, clp;
+    const struct tok *oam_functype_str;
 
     union {
         const struct oam_fm_loopback_t *oam_fm_loopback;
@@ -459,7 +436,6 @@ oam_print(netdissect_options *ndo,
     } oam_ptr;
 
     ndo->ndo_protocol = "oam";
-    ND_TCHECK_1(p + ATM_HDR_LEN_NOHEC + hec);
     cell_header = GET_BE_U_4(p + hec);
     cell_type = (GET_U_1((p + ATM_HDR_LEN_NOHEC + hec)) >> 4) & 0x0f;
     func_type = GET_U_1((p + ATM_HDR_LEN_NOHEC + hec)) & 0x0f;
@@ -483,11 +459,12 @@ oam_print(netdissect_options *ndo,
            tok2str(oam_celltype_values, "unknown", cell_type),
            cell_type);
 
-    if (oam_functype_values[cell_type] == NULL)
+    oam_functype_str = uint2tokary(oam_celltype2tokary, cell_type);
+    if (oam_functype_str == NULL)
         ND_PRINT(", func-type unknown (%u)", func_type);
     else
         ND_PRINT(", func-type %s (%u)",
-               tok2str(oam_functype_values[cell_type],"none",func_type),
+               tok2str(oam_functype_str, "none", func_type),
                func_type);
 
     p += ATM_HDR_LEN_NOHEC + hec;
@@ -548,10 +525,4 @@ oam_print(netdissect_options *ndo,
     ND_PRINT("\n\tcksum 0x%03x (%scorrect)",
            cksum,
            cksum_shouldbe == 0 ? "" : "in");
-
-    return;
-
-trunc:
-    nd_print_trunc(ndo);
-    return;
 }