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

[tcpdump] / print-ahcp.c

diff --git a/print-ahcp.c b/print-ahcp.c
index e452e383682e6c8c6ebf038f1ca773ce88676e69..d2efc7f86bd80ee15a70e848b77a985c2143ed5b 100644 (file)
--- a/print-ahcp.c
+++ b/print-ahcp.c
@@ -29,17 +29,15 @@
 
 /* Based on draft-chroboczek-ahcp-00 and source code of ahcpd-0.53 */
 
 
 /* Based on draft-chroboczek-ahcp-00 and source code of ahcpd-0.53 */
 

-#ifdef HAVE_CONFIG_H

 #include <config.h>
 #include <config.h>

-#endif

 
 #include "netdissect-stdinc.h"
 
 
 #include "netdissect-stdinc.h"
 

+#define ND_LONGJMP_FROM_TCHECK

 #include "netdissect.h"
 #include "extract.h"
 #include "addrtoname.h"
 
 #include "netdissect.h"
 #include "extract.h"
 #include "addrtoname.h"
 

-static const char tstr[] = " [|ahcp]";

 
 #define AHCP_MAGIC_NUMBER 43
 #define AHCP_VERSION_1 1
 
 #define AHCP_MAGIC_NUMBER 43
 #define AHCP_VERSION_1 1


@@ -97,151 +95,126 @@ static const struct tok ahcp1_opt_str[] = {
        { 0, NULL }
 };
 
        { 0, NULL }
 };
 

-static int
-ahcp_time_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
+static void
+ahcp_time_print(netdissect_options *ndo,
+                const u_char *cp, uint8_t len)

 {
        time_t t;
 {
        time_t t;

-       struct tm *tm;
-       char buf[BUFSIZE];
+       char buf[sizeof("-yyyyyyyyyy-mm-dd hh:mm:ss UTC")];

 
 

-       if (cp + 4 != ep)
+       if (len != 4)

                goto invalid;
                goto invalid;

-       ND_TCHECK_4(cp);
-       t = EXTRACT_BE_U_4(cp);
-       if (NULL == (tm = gmtime(&t)))
-               ND_PRINT(": gmtime() error");
-       else if (0 == strftime(buf, sizeof(buf), "%Y-%m-%d %H:%M:%S", tm))
-               ND_PRINT(": strftime() error");
-       else
-               ND_PRINT(": %s UTC", buf);
-       return 0;
+       t = GET_BE_U_4(cp);
+       ND_PRINT(": %s",
+           nd_format_time(buf, sizeof(buf), "%Y-%m-%d %H:%M:%S UTC",
+             gmtime(&t)));
+       return;

 
 invalid:
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return 0;
-trunc:
-       ND_PRINT("%s", tstr);
-       return -1;
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);

 }
 
 }
 

-static int
-ahcp_seconds_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
+static void
+ahcp_seconds_print(netdissect_options *ndo,
+                   const u_char *cp, uint8_t len)

 {
 {

-       if (cp + 4 != ep)
+       if (len != 4)

                goto invalid;
                goto invalid;

-       ND_TCHECK_4(cp);
-       ND_PRINT(": %us", EXTRACT_BE_U_4(cp));
-       return 0;
+       ND_PRINT(": %us", GET_BE_U_4(cp));
+       return;

 
 invalid:
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return 0;
-trunc:
-       ND_PRINT("%s", tstr);
-       return -1;
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);

 }
 
 }
 

-static int
-ahcp_ipv6_addresses_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
+static void
+ahcp_ipv6_addresses_print(netdissect_options *ndo,
+                          const u_char *cp, uint8_t len)

 {
        const char *sep = ": ";
 
 {
        const char *sep = ": ";
 

-       while (cp < ep) {
-               if (cp + 16 > ep)
+       while (len) {
+               if (len < 16)

                        goto invalid;
                        goto invalid;

-               ND_TCHECK_16(cp);
-               ND_PRINT("%s%s", sep, ip6addr_string(ndo, cp));
+               ND_PRINT("%s%s", sep, GET_IP6ADDR_STRING(cp));

                cp += 16;
                cp += 16;

+               len -= 16;

                sep = ", ";
        }
                sep = ", ";
        }

-       return 0;
+       return;

 
 invalid:
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return 0;
-trunc:
-       ND_PRINT("%s", tstr);
-       return -1;
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);

 }
 
 }
 

-static int
-ahcp_ipv4_addresses_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
+static void
+ahcp_ipv4_addresses_print(netdissect_options *ndo,
+                          const u_char *cp, uint8_t len)

 {
        const char *sep = ": ";
 
 {
        const char *sep = ": ";
 

-       while (cp < ep) {
-               if (cp + 4 > ep)
+       while (len) {
+               if (len < 4)

                        goto invalid;
                        goto invalid;

-               ND_TCHECK_4(cp);
-               ND_PRINT("%s%s", sep, ipaddr_string(ndo, cp));
+               ND_PRINT("%s%s", sep, GET_IPADDR_STRING(cp));

                cp += 4;
                cp += 4;

+               len -= 4;

                sep = ", ";
        }
                sep = ", ";
        }

-       return 0;
+       return;

 
 invalid:
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return 0;
-trunc:
-       ND_PRINT("%s", tstr);
-       return -1;
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);

 }
 
 }
 

-static int
-ahcp_ipv6_prefixes_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
+static void
+ahcp_ipv6_prefixes_print(netdissect_options *ndo,
+                         const u_char *cp, uint8_t len)

 {
        const char *sep = ": ";
 
 {
        const char *sep = ": ";
 

-       while (cp < ep) {
-               if (cp + 17 > ep)
+       while (len) {
+               if (len < 17)

                        goto invalid;
                        goto invalid;

-               ND_TCHECK_LEN(cp, 17);
-               ND_PRINT("%s%s/%u", sep, ip6addr_string(ndo, cp), EXTRACT_U_1(cp + 16));
+               ND_PRINT("%s%s/%u", sep, GET_IP6ADDR_STRING(cp), GET_U_1(cp + 16));

                cp += 17;
                cp += 17;

+               len -= 17;

                sep = ", ";
        }
                sep = ", ";
        }

-       return 0;
+       return;

 
 invalid:
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return 0;
-trunc:
-       ND_PRINT("%s", tstr);
-       return -1;
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);

 }
 
 }
 

-static int
-ahcp_ipv4_prefixes_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
+static void
+ahcp_ipv4_prefixes_print(netdissect_options *ndo,
+                         const u_char *cp, uint8_t len)

 {
        const char *sep = ": ";
 
 {
        const char *sep = ": ";
 

-       while (cp < ep) {
-               if (cp + 5 > ep)
+       while (len) {
+               if (len < 5)

                        goto invalid;
                        goto invalid;

-               ND_TCHECK_5(cp);
-               ND_PRINT("%s%s/%u", sep, ipaddr_string(ndo, cp), EXTRACT_U_1(cp + 4));
+               ND_PRINT("%s%s/%u", sep, GET_IPADDR_STRING(cp), GET_U_1(cp + 4));

                cp += 5;
                cp += 5;

+               len -= 5;

                sep = ", ";
        }
                sep = ", ";
        }

-       return 0;
+       return;

 
 invalid:
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return 0;
-trunc:
-       ND_PRINT("%s", tstr);
-       return -1;
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);

 }
 
 }
 

-/* Data decoders signal truncated data with -1. */
-static int
-(* const data_decoders[AHCP1_OPT_MAX + 1])(netdissect_options *, const u_char *, const u_char *) = {
+static void
+(* const data_decoders[AHCP1_OPT_MAX + 1])(netdissect_options *, const u_char *, uint8_t) = {

        /* [AHCP1_OPT_PAD]                    = */  NULL,
        /* [AHCP1_OPT_MANDATORY]              = */  NULL,
        /* [AHCP1_OPT_ORIGIN_TIME]            = */  ahcp_time_print,
        /* [AHCP1_OPT_PAD]                    = */  NULL,
        /* [AHCP1_OPT_MANDATORY]              = */  NULL,
        /* [AHCP1_OPT_ORIGIN_TIME]            = */  ahcp_time_print,


@@ -259,66 +232,65 @@ static int
 };
 
 static void
 };
 
 static void

-ahcp1_options_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
+ahcp1_options_print(netdissect_options *ndo,
+                    const u_char *cp, uint16_t len)

 {
 {

-       uint8_t option_no, option_len;
+       while (len) {
+               uint8_t option_no, option_len;

 
 

-       while (cp < ep) {

                /* Option no */
                /* Option no */

-               ND_TCHECK_1(cp);
-               option_no = EXTRACT_U_1(cp);
+               option_no = GET_U_1(cp);

                cp += 1;
                cp += 1;

+               len -= 1;

                ND_PRINT("\n\t %s", tok2str(ahcp1_opt_str, "Unknown-%u", option_no));
                if (option_no == AHCP1_OPT_PAD || option_no == AHCP1_OPT_MANDATORY)
                        continue;
                /* Length */
                ND_PRINT("\n\t %s", tok2str(ahcp1_opt_str, "Unknown-%u", option_no));
                if (option_no == AHCP1_OPT_PAD || option_no == AHCP1_OPT_MANDATORY)
                        continue;
                /* Length */

-               if (cp + 1 > ep)
+               if (!len)

                        goto invalid;
                        goto invalid;

-               ND_TCHECK_1(cp);
-               option_len = EXTRACT_U_1(cp);
+               option_len = GET_U_1(cp);

                cp += 1;
                cp += 1;

-               if (cp + option_len > ep)
+               len -= 1;
+               if (option_len > len)

                        goto invalid;
                /* Value */
                if (option_no <= AHCP1_OPT_MAX && data_decoders[option_no] != NULL) {
                        goto invalid;
                /* Value */
                if (option_no <= AHCP1_OPT_MAX && data_decoders[option_no] != NULL) {

-                       if (data_decoders[option_no](ndo, cp, cp + option_len) < 0)
-                               break; /* truncated and already marked up */
+                       data_decoders[option_no](ndo, cp, option_len);

                } else {
                        ND_PRINT(" (Length %u)", option_len);
                        ND_TCHECK_LEN(cp, option_len);
                }
                cp += option_len;
                } else {
                        ND_PRINT(" (Length %u)", option_len);
                        ND_TCHECK_LEN(cp, option_len);
                }
                cp += option_len;

+               len -= option_len;

        }
        return;
 
 invalid:
        }
        return;
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return;
-trunc:
-       ND_PRINT("%s", tstr);
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);

 }
 
 static void
 }
 
 static void

-ahcp1_body_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
+ahcp1_body_print(netdissect_options *ndo,
+                 const u_char *cp, u_int len)

 {
        uint8_t type, mbz;
        uint16_t body_len;
 
 {
        uint8_t type, mbz;
        uint16_t body_len;
 

-       if (cp + AHCP1_BODY_MIN_LEN > ep)
+       if (len < AHCP1_BODY_MIN_LEN)

                goto invalid;
        /* Type */
                goto invalid;
        /* Type */

-       ND_TCHECK_1(cp);
-       type = EXTRACT_U_1(cp);
+       type = GET_U_1(cp);

        cp += 1;
        cp += 1;

+       len -= 1;

        /* MBZ */
        /* MBZ */

-       ND_TCHECK_1(cp);
-       mbz = EXTRACT_U_1(cp);
+       mbz = GET_U_1(cp);

        cp += 1;
        cp += 1;

+       len -= 1;

        /* Length */
        /* Length */

-       ND_TCHECK_2(cp);
-       body_len = EXTRACT_BE_U_2(cp);
+       body_len = GET_BE_U_2(cp);

        cp += 2;
        cp += 2;

+       len -= 2;

 
        if (ndo->ndo_vflag) {
                ND_PRINT("\n\t%s", tok2str(ahcp1_msg_str, "Unknown-%u", type));
 
        if (ndo->ndo_vflag) {
                ND_PRINT("\n\t%s", tok2str(ahcp1_msg_str, "Unknown-%u", type));


@@ -326,86 +298,85 @@ ahcp1_body_print(netdissect_options *ndo, const u_char *cp, const u_char *ep)
                        ND_PRINT(", MBZ %u", mbz);
                ND_PRINT(", Length %u", body_len);
        }
                        ND_PRINT(", MBZ %u", mbz);
                ND_PRINT(", Length %u", body_len);
        }

-       if (cp + body_len > ep)
+       if (body_len > len)

                goto invalid;
 
        /* Options */
                goto invalid;
 
        /* Options */

+       /* Here use "body_len", not "len" (ignore any extra data). */

        if (ndo->ndo_vflag >= 2)
        if (ndo->ndo_vflag >= 2)

-               ahcp1_options_print(ndo, cp, cp + body_len); /* not ep (ignore extra data) */
+               ahcp1_options_print(ndo, cp, body_len);

        else
                ND_TCHECK_LEN(cp, body_len);
        return;
 
 invalid:
        else
                ND_TCHECK_LEN(cp, body_len);
        return;
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return;
-trunc:
-       ND_PRINT("%s", tstr);
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);
+

 }
 
 void
 }
 
 void

-ahcp_print(netdissect_options *ndo, const u_char *cp, const u_int len)
+ahcp_print(netdissect_options *ndo,
+           const u_char *cp, u_int len)

 {
 {

-       const u_char *ep = ndo->ndo_snapend;

        uint8_t version;
 
        uint8_t version;
 

-       ND_PRINT("AHCP");
+       ndo->ndo_protocol = "ahcp";
+       nd_print_protocol_caps(ndo);

        if (len < 2)
                goto invalid;
        /* Magic */
        if (len < 2)
                goto invalid;
        /* Magic */

-       ND_TCHECK_1(cp);
-       if (EXTRACT_U_1(cp) != AHCP_MAGIC_NUMBER)
+       if (GET_U_1(cp) != AHCP_MAGIC_NUMBER)

                goto invalid;
        cp += 1;
                goto invalid;
        cp += 1;

+       len -= 1;

        /* Version */
        /* Version */

-       ND_TCHECK_1(cp);
-       version = EXTRACT_U_1(cp);
+       version = GET_U_1(cp);

        cp += 1;
        cp += 1;

+       len -= 1;

        switch (version) {
                case AHCP_VERSION_1: {
                        ND_PRINT(" Version 1");
        switch (version) {
                case AHCP_VERSION_1: {
                        ND_PRINT(" Version 1");

-                       if (len < AHCP1_HEADER_FIX_LEN)
+                       if (len < AHCP1_HEADER_FIX_LEN - 2)

                                goto invalid;
                        if (!ndo->ndo_vflag) {
                                ND_TCHECK_LEN(cp, AHCP1_HEADER_FIX_LEN - 2);
                                cp += AHCP1_HEADER_FIX_LEN - 2;
                                goto invalid;
                        if (!ndo->ndo_vflag) {
                                ND_TCHECK_LEN(cp, AHCP1_HEADER_FIX_LEN - 2);
                                cp += AHCP1_HEADER_FIX_LEN - 2;

+                               len -= AHCP1_HEADER_FIX_LEN - 2;

                        } else {
                                /* Hopcount */
                        } else {
                                /* Hopcount */

-                               ND_TCHECK_1(cp);
-                               ND_PRINT("\n\tHopcount %u", EXTRACT_U_1(cp));
+                               ND_PRINT("\n\tHopcount %u", GET_U_1(cp));

                                cp += 1;
                                cp += 1;

+                               len -= 1;

                                /* Original Hopcount */
                                /* Original Hopcount */

-                               ND_TCHECK_1(cp);
-                               ND_PRINT(", Original Hopcount %u", EXTRACT_U_1(cp));
+                               ND_PRINT(", Original Hopcount %u", GET_U_1(cp));

                                cp += 1;
                                cp += 1;

+                               len -= 1;

                                /* Nonce */
                                /* Nonce */

-                               ND_TCHECK_4(cp);
-                               ND_PRINT(", Nonce 0x%08x", EXTRACT_BE_U_4(cp));
+                               ND_PRINT(", Nonce 0x%08x", GET_BE_U_4(cp));

                                cp += 4;
                                cp += 4;

+                               len -= 4;

                                /* Source Id */
                                /* Source Id */

-                               ND_TCHECK_8(cp);
-                               ND_PRINT(", Source Id %s", linkaddr_string(ndo, cp, 0, 8));
+                               ND_PRINT(", Source Id %s", GET_LINKADDR_STRING(cp, LINKADDR_OTHER, 8));

                                cp += 8;
                                cp += 8;

+                               len -= 8;

                                /* Destination Id */
                                /* Destination Id */

-                               ND_TCHECK_8(cp);
-                               ND_PRINT(", Destination Id %s", linkaddr_string(ndo, cp, 0, 8));
+                               ND_PRINT(", Destination Id %s", GET_LINKADDR_STRING(cp, LINKADDR_OTHER, 8));

                                cp += 8;
                                cp += 8;

+                               len -= 8;

                        }
                        /* Body */
                        }
                        /* Body */

-                       ahcp1_body_print(ndo, cp, ep);
+                       ahcp1_body_print(ndo, cp, len);

                        break;
                }
                default:
                        ND_PRINT(" Version %u (unknown)", version);
                        break;
                }
                default:
                        ND_PRINT(" Version %u (unknown)", version);

+                       ND_TCHECK_LEN(cp, len);

                        break;
        }
        return;
 
 invalid:
                        break;
        }
        return;
 
 invalid:

-       ND_PRINT("%s", istr);
-       ND_TCHECK_LEN(cp, ep - cp);
-       return;
-trunc:
-       ND_PRINT("%s", tstr);
+       nd_print_invalid(ndo);
+       ND_TCHECK_LEN(cp, len);

 }
 }