bgp: Parse BGP extended message support capability

[tcpdump] / print-pptp.c

diff --git a/print-pptp.c b/print-pptp.c
index b2adc1d95250c7d059fad3e048e01e837eb224ad..8e1b3038885a3d97bc954462c112d49f845b8772 100644 (file)
--- a/print-pptp.c
+++ b/print-pptp.c
@@ -276,17 +276,13 @@ struct pptp_msg_sli {
 /* Attribute-specific print out functions */
 /******************************************/
 
 /* Attribute-specific print out functions */
 /******************************************/
 

-/* In these attribute-specific print-out functions, it't not necessary
-   to do ND_TCHECK because they are already checked in the caller of
-   these functions. */
-

 static void
 pptp_bearer_cap_print(netdissect_options *ndo,
 static void
 pptp_bearer_cap_print(netdissect_options *ndo,

-                      const nd_uint32_t *bearer_cap)
+                      const nd_uint32_t bearer_cap)

 {
        ND_PRINT(" BEARER_CAP(%s%s)",
 {
        ND_PRINT(" BEARER_CAP(%s%s)",

-                 GET_BE_U_4(*bearer_cap) & PPTP_BEARER_CAP_DIGITAL_MASK ? "D" : "",
-                 GET_BE_U_4(*bearer_cap) & PPTP_BEARER_CAP_ANALOG_MASK ? "A" : "");
+                 GET_BE_U_4(bearer_cap) & PPTP_BEARER_CAP_DIGITAL_MASK ? "D" : "",
+                 GET_BE_U_4(bearer_cap) & PPTP_BEARER_CAP_ANALOG_MASK ? "A" : "");

 }
 
 static const struct tok pptp_btype_str[] = {
 }
 
 static const struct tok pptp_btype_str[] = {


@@ -298,38 +294,38 @@ static const struct tok pptp_btype_str[] = {
 
 static void
 pptp_bearer_type_print(netdissect_options *ndo,
 
 static void
 pptp_bearer_type_print(netdissect_options *ndo,

-                       const nd_uint32_t *bearer_type)
+                       const nd_uint32_t bearer_type)

 {
        ND_PRINT(" BEARER_TYPE(%s)",
 {
        ND_PRINT(" BEARER_TYPE(%s)",

-                 tok2str(pptp_btype_str, "?", GET_BE_U_4(*bearer_type)));
+                 tok2str(pptp_btype_str, "?", GET_BE_U_4(bearer_type)));

 }
 
 static void
 pptp_call_id_print(netdissect_options *ndo,
 }
 
 static void
 pptp_call_id_print(netdissect_options *ndo,

-                   const nd_uint16_t *call_id)
+                   const nd_uint16_t call_id)

 {
 {

-       ND_PRINT(" CALL_ID(%u)", GET_BE_U_2(*call_id));
+       ND_PRINT(" CALL_ID(%u)", GET_BE_U_2(call_id));

 }
 
 static void
 pptp_call_ser_print(netdissect_options *ndo,
 }
 
 static void
 pptp_call_ser_print(netdissect_options *ndo,

-                    const nd_uint16_t *call_ser)
+                    const nd_uint16_t call_ser)

 {
 {

-       ND_PRINT(" CALL_SER_NUM(%u)", GET_BE_U_2(*call_ser));
+       ND_PRINT(" CALL_SER_NUM(%u)", GET_BE_U_2(call_ser));

 }
 
 static void
 pptp_cause_code_print(netdissect_options *ndo,
 }
 
 static void
 pptp_cause_code_print(netdissect_options *ndo,

-                      const nd_uint16_t *cause_code)
+                      const nd_uint16_t cause_code)

 {
 {

-       ND_PRINT(" CAUSE_CODE(%u)", GET_BE_U_2(*cause_code));
+       ND_PRINT(" CAUSE_CODE(%u)", GET_BE_U_2(cause_code));

 }
 
 static void
 pptp_conn_speed_print(netdissect_options *ndo,
 }
 
 static void
 pptp_conn_speed_print(netdissect_options *ndo,

-                      const nd_uint32_t *conn_speed)
+                      const nd_uint32_t conn_speed)

 {
 {

-       ND_PRINT(" CONN_SPEED(%u)", GET_BE_U_4(*conn_speed));
+       ND_PRINT(" CONN_SPEED(%u)", GET_BE_U_4(conn_speed));

 }
 
 static const struct tok pptp_errcode_str[] = {
 }
 
 static const struct tok pptp_errcode_str[] = {


@@ -345,32 +341,32 @@ static const struct tok pptp_errcode_str[] = {
 
 static void
 pptp_err_code_print(netdissect_options *ndo,
 
 static void
 pptp_err_code_print(netdissect_options *ndo,

-                    const nd_uint8_t *err_code)
+                    const nd_uint8_t err_code)

 {
 {

-       ND_PRINT(" ERR_CODE(%u", GET_U_1(*err_code));
+       ND_PRINT(" ERR_CODE(%u", GET_U_1(err_code));

        if (ndo->ndo_vflag) {
                ND_PRINT(":%s",
        if (ndo->ndo_vflag) {
                ND_PRINT(":%s",

-                        tok2str(pptp_errcode_str, "?", GET_U_1(*err_code)));
+                        tok2str(pptp_errcode_str, "?", GET_U_1(err_code)));

        }
        ND_PRINT(")");
 }
 
 static void
 pptp_firm_rev_print(netdissect_options *ndo,
        }
        ND_PRINT(")");
 }
 
 static void
 pptp_firm_rev_print(netdissect_options *ndo,

-                    const nd_uint16_t *firm_rev)
+                    const nd_uint16_t firm_rev)

 {
 {

-       ND_PRINT(" FIRM_REV(%u)", GET_BE_U_2(*firm_rev));
+       ND_PRINT(" FIRM_REV(%u)", GET_BE_U_2(firm_rev));

 }
 
 static void
 pptp_framing_cap_print(netdissect_options *ndo,
 }
 
 static void
 pptp_framing_cap_print(netdissect_options *ndo,

-                       const nd_uint32_t *framing_cap)
+                       const nd_uint32_t framing_cap)

 {
        ND_PRINT(" FRAME_CAP(");
 {
        ND_PRINT(" FRAME_CAP(");

-       if (GET_BE_U_4(*framing_cap) & PPTP_FRAMING_CAP_ASYNC_MASK) {
+       if (GET_BE_U_4(framing_cap) & PPTP_FRAMING_CAP_ASYNC_MASK) {

                 ND_PRINT("A");         /* Async */
         }
                 ND_PRINT("A");         /* Async */
         }

-        if (GET_BE_U_4(*framing_cap) & PPTP_FRAMING_CAP_SYNC_MASK) {
+        if (GET_BE_U_4(framing_cap) & PPTP_FRAMING_CAP_SYNC_MASK) {

                 ND_PRINT("S");         /* Sync */
         }
        ND_PRINT(")");
                 ND_PRINT("S");         /* Sync */
         }
        ND_PRINT(")");


@@ -385,68 +381,70 @@ static const struct tok pptp_ftype_str[] = {
 
 static void
 pptp_framing_type_print(netdissect_options *ndo,
 
 static void
 pptp_framing_type_print(netdissect_options *ndo,

-                        const nd_uint32_t *framing_type)
+                        const nd_uint32_t framing_type)

 {
        ND_PRINT(" FRAME_TYPE(%s)",
 {
        ND_PRINT(" FRAME_TYPE(%s)",

-                 tok2str(pptp_ftype_str, "?", GET_BE_U_4(*framing_type)));
+                 tok2str(pptp_ftype_str, "?", GET_BE_U_4(framing_type)));

 }
 
 static void
 pptp_hostname_print(netdissect_options *ndo,
                     const u_char *hostname)
 {
 }
 
 static void
 pptp_hostname_print(netdissect_options *ndo,
                     const u_char *hostname)
 {

-       ND_PRINT(" HOSTNAME(%.64s)", hostname);
+       ND_PRINT(" HOSTNAME(");
+       nd_printjnp(ndo, hostname, 64);
+       ND_PRINT(")");

 }
 
 static void
 pptp_id_print(netdissect_options *ndo,
 }
 
 static void
 pptp_id_print(netdissect_options *ndo,

-              const nd_uint32_t *id)
+              const nd_uint32_t id)

 {
 {

-       ND_PRINT(" ID(%u)", GET_BE_U_4(*id));
+       ND_PRINT(" ID(%u)", GET_BE_U_4(id));

 }
 
 static void
 pptp_max_channel_print(netdissect_options *ndo,
 }
 
 static void
 pptp_max_channel_print(netdissect_options *ndo,

-                       const nd_uint16_t *max_channel)
+                       const nd_uint16_t max_channel)

 {
 {

-       ND_PRINT(" MAX_CHAN(%u)", GET_BE_U_2(*max_channel));
+       ND_PRINT(" MAX_CHAN(%u)", GET_BE_U_2(max_channel));

 }
 
 static void
 pptp_peer_call_id_print(netdissect_options *ndo,
 }
 
 static void
 pptp_peer_call_id_print(netdissect_options *ndo,

-                        const nd_uint16_t *peer_call_id)
+                        const nd_uint16_t peer_call_id)

 {
 {

-       ND_PRINT(" PEER_CALL_ID(%u)", GET_BE_U_2(*peer_call_id));
+       ND_PRINT(" PEER_CALL_ID(%u)", GET_BE_U_2(peer_call_id));

 }
 
 static void
 pptp_phy_chan_id_print(netdissect_options *ndo,
 }
 
 static void
 pptp_phy_chan_id_print(netdissect_options *ndo,

-                       const nd_uint32_t *phy_chan_id)
+                       const nd_uint32_t phy_chan_id)

 {
 {

-       ND_PRINT(" PHY_CHAN_ID(%u)", GET_BE_U_4(*phy_chan_id));
+       ND_PRINT(" PHY_CHAN_ID(%u)", GET_BE_U_4(phy_chan_id));

 }
 
 static void
 pptp_pkt_proc_delay_print(netdissect_options *ndo,
 }
 
 static void
 pptp_pkt_proc_delay_print(netdissect_options *ndo,

-                          const nd_uint16_t *pkt_proc_delay)
+                          const nd_uint16_t pkt_proc_delay)

 {
 {

-       ND_PRINT(" PROC_DELAY(%u)", GET_BE_U_2(*pkt_proc_delay));
+       ND_PRINT(" PROC_DELAY(%u)", GET_BE_U_2(pkt_proc_delay));

 }
 
 static void
 pptp_proto_ver_print(netdissect_options *ndo,
 }
 
 static void
 pptp_proto_ver_print(netdissect_options *ndo,

-                     const nd_uint16_t *proto_ver)
+                     const nd_uint16_t proto_ver)

 {
        ND_PRINT(" PROTO_VER(%u.%u)",   /* Version.Revision */
 {
        ND_PRINT(" PROTO_VER(%u.%u)",   /* Version.Revision */

-              GET_BE_U_2(*proto_ver) >> 8,
-              GET_BE_U_2(*proto_ver) & 0xff);
+              GET_BE_U_2(proto_ver) >> 8,
+              GET_BE_U_2(proto_ver) & 0xff);

 }
 
 static void
 pptp_recv_winsiz_print(netdissect_options *ndo,
 }
 
 static void
 pptp_recv_winsiz_print(netdissect_options *ndo,

-                       const nd_uint16_t *recv_winsiz)
+                       const nd_uint16_t recv_winsiz)

 {
 {

-       ND_PRINT(" RECV_WIN(%u)", GET_BE_U_2(*recv_winsiz));
+       ND_PRINT(" RECV_WIN(%u)", GET_BE_U_2(recv_winsiz));

 }
 
 static const struct tok pptp_scrrp_str[] = {
 }
 
 static const struct tok pptp_scrrp_str[] = {


@@ -492,9 +490,9 @@ static const struct tok pptp_cdn_str[] = {
 
 static void
 pptp_result_code_print(netdissect_options *ndo,
 
 static void
 pptp_result_code_print(netdissect_options *ndo,

-                       const nd_uint8_t *result_code, int ctrl_msg_type)
+                       const nd_uint8_t result_code, int ctrl_msg_type)

 {
 {

-       ND_PRINT(" RESULT_CODE(%u", GET_U_1(*result_code));
+       ND_PRINT(" RESULT_CODE(%u", GET_U_1(result_code));

        if (ndo->ndo_vflag) {
                const struct tok *dict =
                        ctrl_msg_type == PPTP_CTRL_MSG_TYPE_SCCRP    ? pptp_scrrp_str :
        if (ndo->ndo_vflag) {
                const struct tok *dict =
                        ctrl_msg_type == PPTP_CTRL_MSG_TYPE_SCCRP    ? pptp_scrrp_str :


@@ -506,7 +504,7 @@ pptp_result_code_print(netdissect_options *ndo,
                        NULL; /* assertion error */
                if (dict != NULL)
                        ND_PRINT(":%s",
                        NULL; /* assertion error */
                if (dict != NULL)
                        ND_PRINT(":%s",

-                                tok2str(dict, "?", GET_U_1(*result_code)));
+                                tok2str(dict, "?", GET_U_1(result_code)));

        }
        ND_PRINT(")");
 }
        }
        ND_PRINT(")");
 }


@@ -515,14 +513,18 @@ static void
 pptp_subaddr_print(netdissect_options *ndo,
                    const u_char *subaddr)
 {
 pptp_subaddr_print(netdissect_options *ndo,
                    const u_char *subaddr)
 {

-       ND_PRINT(" SUB_ADDR(%.64s)", subaddr);
+       ND_PRINT(" SUB_ADDR(");
+       nd_printjnp(ndo, subaddr, 64);
+       ND_PRINT(")");

 }
 
 static void
 pptp_vendor_print(netdissect_options *ndo,
                   const u_char *vendor)
 {
 }
 
 static void
 pptp_vendor_print(netdissect_options *ndo,
                   const u_char *vendor)
 {

-       ND_PRINT(" VENDOR(%.64s)", vendor);
+       ND_PRINT(" VENDOR(");
+       nd_printjnp(ndo, vendor, 64);
+       ND_PRINT(")");

 }
 
 /************************************/
 }
 
 /************************************/


@@ -534,27 +536,14 @@ pptp_sccrq_print(netdissect_options *ndo,
 {
        const struct pptp_msg_sccrq *ptr = (const struct pptp_msg_sccrq *)dat;
 
 {
        const struct pptp_msg_sccrq *ptr = (const struct pptp_msg_sccrq *)dat;
 

-       ND_TCHECK_2(ptr->proto_ver);
-       pptp_proto_ver_print(ndo, &ptr->proto_ver);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_proto_ver_print(ndo, ptr->proto_ver);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-       ND_TCHECK_4(ptr->framing_cap);
-       pptp_framing_cap_print(ndo, &ptr->framing_cap);
-       ND_TCHECK_4(ptr->bearer_cap);
-       pptp_bearer_cap_print(ndo, &ptr->bearer_cap);
-       ND_TCHECK_2(ptr->max_channel);
-       pptp_max_channel_print(ndo, &ptr->max_channel);
-       ND_TCHECK_2(ptr->firm_rev);
-       pptp_firm_rev_print(ndo, &ptr->firm_rev);
-       ND_TCHECK_SIZE(&ptr->hostname);
-       pptp_hostname_print(ndo, &ptr->hostname[0]);
-       ND_TCHECK_SIZE(&ptr->vendor);
-       pptp_vendor_print(ndo, &ptr->vendor[0]);
-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);
+       pptp_framing_cap_print(ndo, ptr->framing_cap);
+       pptp_bearer_cap_print(ndo, ptr->bearer_cap);
+       pptp_max_channel_print(ndo, ptr->max_channel);
+       pptp_firm_rev_print(ndo, ptr->firm_rev);
+       pptp_hostname_print(ndo, ptr->hostname);
+       pptp_vendor_print(ndo, ptr->vendor);

 }
 
 static void
 }
 
 static void


@@ -563,29 +552,15 @@ pptp_sccrp_print(netdissect_options *ndo,
 {
        const struct pptp_msg_sccrp *ptr = (const struct pptp_msg_sccrp *)dat;
 
 {
        const struct pptp_msg_sccrp *ptr = (const struct pptp_msg_sccrp *)dat;
 

-       ND_TCHECK_2(ptr->proto_ver);
-       pptp_proto_ver_print(ndo, &ptr->proto_ver);
-       ND_TCHECK_1(ptr->result_code);
-       pptp_result_code_print(ndo, &ptr->result_code, PPTP_CTRL_MSG_TYPE_SCCRP);
-       ND_TCHECK_1(ptr->err_code);
-       pptp_err_code_print(ndo, &ptr->err_code);
-       ND_TCHECK_4(ptr->framing_cap);
-       pptp_framing_cap_print(ndo, &ptr->framing_cap);
-       ND_TCHECK_4(ptr->bearer_cap);
-       pptp_bearer_cap_print(ndo, &ptr->bearer_cap);
-       ND_TCHECK_2(ptr->max_channel);
-       pptp_max_channel_print(ndo, &ptr->max_channel);
-       ND_TCHECK_2(ptr->firm_rev);
-       pptp_firm_rev_print(ndo, &ptr->firm_rev);
-       ND_TCHECK_SIZE(&ptr->hostname);
-       pptp_hostname_print(ndo, &ptr->hostname[0]);
-       ND_TCHECK_SIZE(&ptr->vendor);
-       pptp_vendor_print(ndo, &ptr->vendor[0]);
-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);
+       pptp_proto_ver_print(ndo, ptr->proto_ver);
+       pptp_result_code_print(ndo, ptr->result_code, PPTP_CTRL_MSG_TYPE_SCCRP);
+       pptp_err_code_print(ndo, ptr->err_code);
+       pptp_framing_cap_print(ndo, ptr->framing_cap);
+       pptp_bearer_cap_print(ndo, ptr->bearer_cap);
+       pptp_max_channel_print(ndo, ptr->max_channel);
+       pptp_firm_rev_print(ndo, ptr->firm_rev);
+       pptp_hostname_print(ndo, ptr->hostname);
+       pptp_vendor_print(ndo, ptr->vendor);

 }
 
 static void
 }
 
 static void


@@ -594,7 +569,6 @@ pptp_stopccrq_print(netdissect_options *ndo,
 {
        const struct pptp_msg_stopccrq *ptr = (const struct pptp_msg_stopccrq *)dat;
 
 {
        const struct pptp_msg_stopccrq *ptr = (const struct pptp_msg_stopccrq *)dat;
 

-       ND_TCHECK_1(ptr->reason);

        ND_PRINT(" REASON(%u", GET_U_1(ptr->reason));
        if (ndo->ndo_vflag) {
                switch (GET_U_1(ptr->reason)) {
        ND_PRINT(" REASON(%u", GET_U_1(ptr->reason));
        if (ndo->ndo_vflag) {
                switch (GET_U_1(ptr->reason)) {


@@ -613,15 +587,8 @@ pptp_stopccrq_print(netdissect_options *ndo,
                }
        }
        ND_PRINT(")");
                }
        }
        ND_PRINT(")");

-       ND_TCHECK_1(ptr->reserved1);

        PRINT_RESERVED_IF_NOT_ZERO_1(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_1(ptr->reserved1);

-       ND_TCHECK_2(ptr->reserved2);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved2);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved2);

-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);

 }
 
 static void
 }
 
 static void


@@ -630,17 +597,9 @@ pptp_stopccrp_print(netdissect_options *ndo,
 {
        const struct pptp_msg_stopccrp *ptr = (const struct pptp_msg_stopccrp *)dat;
 
 {
        const struct pptp_msg_stopccrp *ptr = (const struct pptp_msg_stopccrp *)dat;
 

-       ND_TCHECK_1(ptr->result_code);
-       pptp_result_code_print(ndo, &ptr->result_code, PPTP_CTRL_MSG_TYPE_StopCCRP);
-       ND_TCHECK_1(ptr->err_code);
-       pptp_err_code_print(ndo, &ptr->err_code);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_result_code_print(ndo, ptr->result_code, PPTP_CTRL_MSG_TYPE_StopCCRP);
+       pptp_err_code_print(ndo, ptr->err_code);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);

 }
 
 static void
 }
 
 static void


@@ -649,13 +608,7 @@ pptp_echorq_print(netdissect_options *ndo,
 {
        const struct pptp_msg_echorq *ptr = (const struct pptp_msg_echorq *)dat;
 
 {
        const struct pptp_msg_echorq *ptr = (const struct pptp_msg_echorq *)dat;
 

-       ND_TCHECK_4(ptr->id);
-       pptp_id_print(ndo, &ptr->id);
-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);
+       pptp_id_print(ndo, ptr->id);

 }
 
 static void
 }
 
 static void


@@ -664,19 +617,10 @@ pptp_echorp_print(netdissect_options *ndo,
 {
        const struct pptp_msg_echorp *ptr = (const struct pptp_msg_echorp *)dat;
 
 {
        const struct pptp_msg_echorp *ptr = (const struct pptp_msg_echorp *)dat;
 

-       ND_TCHECK_4(ptr->id);
-       pptp_id_print(ndo, &ptr->id);
-       ND_TCHECK_1(ptr->result_code);
-       pptp_result_code_print(ndo, &ptr->result_code, PPTP_CTRL_MSG_TYPE_ECHORP);
-       ND_TCHECK_1(ptr->err_code);
-       pptp_err_code_print(ndo, &ptr->err_code);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_id_print(ndo, ptr->id);
+       pptp_result_code_print(ndo, ptr->result_code, PPTP_CTRL_MSG_TYPE_ECHORP);
+       pptp_err_code_print(ndo, ptr->err_code);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);

 }
 
 static void
 }
 
 static void


@@ -685,35 +629,21 @@ pptp_ocrq_print(netdissect_options *ndo,
 {
        const struct pptp_msg_ocrq *ptr = (const struct pptp_msg_ocrq *)dat;
 
 {
        const struct pptp_msg_ocrq *ptr = (const struct pptp_msg_ocrq *)dat;
 

-       ND_TCHECK_2(ptr->call_id);
-       pptp_call_id_print(ndo, &ptr->call_id);
-       ND_TCHECK_2(ptr->call_ser);
-       pptp_call_ser_print(ndo, &ptr->call_ser);
-       ND_TCHECK_4(ptr->min_bps);
+       pptp_call_id_print(ndo, ptr->call_id);
+       pptp_call_ser_print(ndo, ptr->call_ser);

        ND_PRINT(" MIN_BPS(%u)", GET_BE_U_4(ptr->min_bps));
        ND_PRINT(" MIN_BPS(%u)", GET_BE_U_4(ptr->min_bps));

-       ND_TCHECK_4(ptr->max_bps);

        ND_PRINT(" MAX_BPS(%u)", GET_BE_U_4(ptr->max_bps));
        ND_PRINT(" MAX_BPS(%u)", GET_BE_U_4(ptr->max_bps));

-       ND_TCHECK_4(ptr->bearer_type);
-       pptp_bearer_type_print(ndo, &ptr->bearer_type);
-       ND_TCHECK_4(ptr->framing_type);
-       pptp_framing_type_print(ndo, &ptr->framing_type);
-       ND_TCHECK_2(ptr->recv_winsiz);
-       pptp_recv_winsiz_print(ndo, &ptr->recv_winsiz);
-       ND_TCHECK_2(ptr->pkt_proc_delay);
-       pptp_pkt_proc_delay_print(ndo, &ptr->pkt_proc_delay);
-       ND_TCHECK_2(ptr->phone_no_len);
+       pptp_bearer_type_print(ndo, ptr->bearer_type);
+       pptp_framing_type_print(ndo, ptr->framing_type);
+       pptp_recv_winsiz_print(ndo, ptr->recv_winsiz);
+       pptp_pkt_proc_delay_print(ndo, ptr->pkt_proc_delay);

        ND_PRINT(" PHONE_NO_LEN(%u)", GET_BE_U_2(ptr->phone_no_len));
        ND_PRINT(" PHONE_NO_LEN(%u)", GET_BE_U_2(ptr->phone_no_len));

-       ND_TCHECK_2(ptr->reserved1);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-       ND_TCHECK_SIZE(&ptr->phone_no);
-       ND_PRINT(" PHONE_NO(%.64s)", ptr->phone_no);
-       ND_TCHECK_SIZE(&ptr->subaddr);
-       pptp_subaddr_print(ndo, &ptr->subaddr[0]);
-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);
+       ND_PRINT(" PHONE_NO(");
+       nd_printjnp(ndo, ptr->phone_no,
+                   ND_MIN(64, GET_BE_U_2(ptr->phone_no_len)));
+       ND_PRINT(")");
+       pptp_subaddr_print(ndo, ptr->subaddr);

 }
 
 static void
 }
 
 static void


@@ -722,29 +652,15 @@ pptp_ocrp_print(netdissect_options *ndo,
 {
        const struct pptp_msg_ocrp *ptr = (const struct pptp_msg_ocrp *)dat;
 
 {
        const struct pptp_msg_ocrp *ptr = (const struct pptp_msg_ocrp *)dat;
 

-       ND_TCHECK_2(ptr->call_id);
-       pptp_call_id_print(ndo, &ptr->call_id);
-       ND_TCHECK_2(ptr->peer_call_id);
-       pptp_peer_call_id_print(ndo, &ptr->peer_call_id);
-       ND_TCHECK_1(ptr->result_code);
-       pptp_result_code_print(ndo, &ptr->result_code, PPTP_CTRL_MSG_TYPE_OCRP);
-       ND_TCHECK_1(ptr->err_code);
-       pptp_err_code_print(ndo, &ptr->err_code);
-       ND_TCHECK_2(ptr->cause_code);
-       pptp_cause_code_print(ndo, &ptr->cause_code);
-       ND_TCHECK_4(ptr->conn_speed);
-       pptp_conn_speed_print(ndo, &ptr->conn_speed);
-       ND_TCHECK_2(ptr->recv_winsiz);
-       pptp_recv_winsiz_print(ndo, &ptr->recv_winsiz);
-       ND_TCHECK_2(ptr->pkt_proc_delay);
-       pptp_pkt_proc_delay_print(ndo, &ptr->pkt_proc_delay);
-       ND_TCHECK_4(ptr->phy_chan_id);
-       pptp_phy_chan_id_print(ndo, &ptr->phy_chan_id);
-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);
+       pptp_call_id_print(ndo, ptr->call_id);
+       pptp_peer_call_id_print(ndo, ptr->peer_call_id);
+       pptp_result_code_print(ndo, ptr->result_code, PPTP_CTRL_MSG_TYPE_OCRP);
+       pptp_err_code_print(ndo, ptr->err_code);
+       pptp_cause_code_print(ndo, ptr->cause_code);
+       pptp_conn_speed_print(ndo, ptr->conn_speed);
+       pptp_recv_winsiz_print(ndo, ptr->recv_winsiz);
+       pptp_pkt_proc_delay_print(ndo, ptr->pkt_proc_delay);
+       pptp_phy_chan_id_print(ndo, ptr->phy_chan_id);

 }
 
 static void
 }
 
 static void


@@ -753,29 +669,21 @@ pptp_icrq_print(netdissect_options *ndo,
 {
        const struct pptp_msg_icrq *ptr = (const struct pptp_msg_icrq *)dat;
 
 {
        const struct pptp_msg_icrq *ptr = (const struct pptp_msg_icrq *)dat;
 

-       ND_TCHECK_2(ptr->call_id);
-       pptp_call_id_print(ndo, &ptr->call_id);
-       ND_TCHECK_2(ptr->call_ser);
-       pptp_call_ser_print(ndo, &ptr->call_ser);
-       ND_TCHECK_4(ptr->bearer_type);
-       pptp_bearer_type_print(ndo, &ptr->bearer_type);
-       ND_TCHECK_4(ptr->phy_chan_id);
-       pptp_phy_chan_id_print(ndo, &ptr->phy_chan_id);
-       ND_TCHECK_2(ptr->dialed_no_len);
+       pptp_call_id_print(ndo, ptr->call_id);
+       pptp_call_ser_print(ndo, ptr->call_ser);
+       pptp_bearer_type_print(ndo, ptr->bearer_type);
+       pptp_phy_chan_id_print(ndo, ptr->phy_chan_id);

        ND_PRINT(" DIALED_NO_LEN(%u)", GET_BE_U_2(ptr->dialed_no_len));
        ND_PRINT(" DIALED_NO_LEN(%u)", GET_BE_U_2(ptr->dialed_no_len));

-       ND_TCHECK_2(ptr->dialing_no_len);

        ND_PRINT(" DIALING_NO_LEN(%u)", GET_BE_U_2(ptr->dialing_no_len));
        ND_PRINT(" DIALING_NO_LEN(%u)", GET_BE_U_2(ptr->dialing_no_len));

-       ND_TCHECK_SIZE(&ptr->dialed_no);
-       ND_PRINT(" DIALED_NO(%.64s)", ptr->dialed_no);
-       ND_TCHECK_SIZE(&ptr->dialing_no);
-       ND_PRINT(" DIALING_NO(%.64s)", ptr->dialing_no);
-       ND_TCHECK_SIZE(&ptr->subaddr);
-       pptp_subaddr_print(ndo, &ptr->subaddr[0]);
-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);
+       ND_PRINT(" DIALED_NO(");
+       nd_printjnp(ndo, ptr->dialed_no,
+                   ND_MIN(64, GET_BE_U_2(ptr->dialed_no_len)));
+       ND_PRINT(")");
+       ND_PRINT(" DIALING_NO(");
+       nd_printjnp(ndo, ptr->dialing_no,
+                   ND_MIN(64, GET_BE_U_2(ptr->dialing_no_len)));
+       ND_PRINT(")");
+       pptp_subaddr_print(ndo, ptr->subaddr);

 }
 
 static void
 }
 
 static void


@@ -784,25 +692,13 @@ pptp_icrp_print(netdissect_options *ndo,
 {
        const struct pptp_msg_icrp *ptr = (const struct pptp_msg_icrp *)dat;
 
 {
        const struct pptp_msg_icrp *ptr = (const struct pptp_msg_icrp *)dat;
 

-       ND_TCHECK_2(ptr->call_id);
-       pptp_call_id_print(ndo, &ptr->call_id);
-       ND_TCHECK_2(ptr->peer_call_id);
-       pptp_peer_call_id_print(ndo, &ptr->peer_call_id);
-       ND_TCHECK_1(ptr->result_code);
-       pptp_result_code_print(ndo, &ptr->result_code, PPTP_CTRL_MSG_TYPE_ICRP);
-       ND_TCHECK_1(ptr->err_code);
-       pptp_err_code_print(ndo, &ptr->err_code);
-       ND_TCHECK_2(ptr->recv_winsiz);
-       pptp_recv_winsiz_print(ndo, &ptr->recv_winsiz);
-       ND_TCHECK_2(ptr->pkt_proc_delay);
-       pptp_pkt_proc_delay_print(ndo, &ptr->pkt_proc_delay);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_call_id_print(ndo, ptr->call_id);
+       pptp_peer_call_id_print(ndo, ptr->peer_call_id);
+       pptp_result_code_print(ndo, ptr->result_code, PPTP_CTRL_MSG_TYPE_ICRP);
+       pptp_err_code_print(ndo, ptr->err_code);
+       pptp_recv_winsiz_print(ndo, ptr->recv_winsiz);
+       pptp_pkt_proc_delay_print(ndo, ptr->pkt_proc_delay);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);

 }
 
 static void
 }
 
 static void


@@ -811,23 +707,12 @@ pptp_iccn_print(netdissect_options *ndo,
 {
        const struct pptp_msg_iccn *ptr = (const struct pptp_msg_iccn *)dat;
 
 {
        const struct pptp_msg_iccn *ptr = (const struct pptp_msg_iccn *)dat;
 

-       ND_TCHECK_2(ptr->peer_call_id);
-       pptp_peer_call_id_print(ndo, &ptr->peer_call_id);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_peer_call_id_print(ndo, ptr->peer_call_id);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-       ND_TCHECK_4(ptr->conn_speed);
-       pptp_conn_speed_print(ndo, &ptr->conn_speed);
-       ND_TCHECK_2(ptr->recv_winsiz);
-       pptp_recv_winsiz_print(ndo, &ptr->recv_winsiz);
-       ND_TCHECK_2(ptr->pkt_proc_delay);
-       pptp_pkt_proc_delay_print(ndo, &ptr->pkt_proc_delay);
-       ND_TCHECK_4(ptr->framing_type);
-       pptp_framing_type_print(ndo, &ptr->framing_type);
-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);
+       pptp_conn_speed_print(ndo, ptr->conn_speed);
+       pptp_recv_winsiz_print(ndo, ptr->recv_winsiz);
+       pptp_pkt_proc_delay_print(ndo, ptr->pkt_proc_delay);
+       pptp_framing_type_print(ndo, ptr->framing_type);

 }
 
 static void
 }
 
 static void


@@ -836,15 +721,8 @@ pptp_ccrq_print(netdissect_options *ndo,
 {
        const struct pptp_msg_ccrq *ptr = (const struct pptp_msg_ccrq *)dat;
 
 {
        const struct pptp_msg_ccrq *ptr = (const struct pptp_msg_ccrq *)dat;
 

-       ND_TCHECK_2(ptr->call_id);
-       pptp_call_id_print(ndo, &ptr->call_id);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_call_id_print(ndo, ptr->call_id);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);

 }
 
 static void
 }
 
 static void


@@ -853,23 +731,14 @@ pptp_cdn_print(netdissect_options *ndo,
 {
        const struct pptp_msg_cdn *ptr = (const struct pptp_msg_cdn *)dat;
 
 {
        const struct pptp_msg_cdn *ptr = (const struct pptp_msg_cdn *)dat;
 

-       ND_TCHECK_2(ptr->call_id);
-       pptp_call_id_print(ndo, &ptr->call_id);
-       ND_TCHECK_1(ptr->result_code);
-       pptp_result_code_print(ndo, &ptr->result_code, PPTP_CTRL_MSG_TYPE_CDN);
-       ND_TCHECK_1(ptr->err_code);
-       pptp_err_code_print(ndo, &ptr->err_code);
-       ND_TCHECK_2(ptr->cause_code);
-       pptp_cause_code_print(ndo, &ptr->cause_code);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_call_id_print(ndo, ptr->call_id);
+       pptp_result_code_print(ndo, ptr->result_code, PPTP_CTRL_MSG_TYPE_CDN);
+       pptp_err_code_print(ndo, ptr->err_code);
+       pptp_cause_code_print(ndo, ptr->cause_code);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-       ND_TCHECK_SIZE(&ptr->call_stats);
-       ND_PRINT(" CALL_STATS(%.128s)", ptr->call_stats);
-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);
+       ND_PRINT(" CALL_STATS(");
+       nd_printjnp(ndo, ptr->call_stats, 128);
+       ND_PRINT(")");

 }
 
 static void
 }
 
 static void


@@ -878,27 +747,14 @@ pptp_wen_print(netdissect_options *ndo,
 {
        const struct pptp_msg_wen *ptr = (const struct pptp_msg_wen *)dat;
 
 {
        const struct pptp_msg_wen *ptr = (const struct pptp_msg_wen *)dat;
 

-       ND_TCHECK_2(ptr->peer_call_id);
-       pptp_peer_call_id_print(ndo, &ptr->peer_call_id);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_peer_call_id_print(ndo, ptr->peer_call_id);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-       ND_TCHECK_4(ptr->crc_err);

        ND_PRINT(" CRC_ERR(%u)", GET_BE_U_4(ptr->crc_err));
        ND_PRINT(" CRC_ERR(%u)", GET_BE_U_4(ptr->crc_err));

-       ND_TCHECK_4(ptr->framing_err);

        ND_PRINT(" FRAMING_ERR(%u)", GET_BE_U_4(ptr->framing_err));
        ND_PRINT(" FRAMING_ERR(%u)", GET_BE_U_4(ptr->framing_err));

-       ND_TCHECK_4(ptr->hardware_overrun);

        ND_PRINT(" HARDWARE_OVERRUN(%u)", GET_BE_U_4(ptr->hardware_overrun));
        ND_PRINT(" HARDWARE_OVERRUN(%u)", GET_BE_U_4(ptr->hardware_overrun));

-       ND_TCHECK_4(ptr->buffer_overrun);

        ND_PRINT(" BUFFER_OVERRUN(%u)", GET_BE_U_4(ptr->buffer_overrun));
        ND_PRINT(" BUFFER_OVERRUN(%u)", GET_BE_U_4(ptr->buffer_overrun));

-       ND_TCHECK_4(ptr->timeout_err);

        ND_PRINT(" TIMEOUT_ERR(%u)", GET_BE_U_4(ptr->timeout_err));
        ND_PRINT(" TIMEOUT_ERR(%u)", GET_BE_U_4(ptr->timeout_err));

-       ND_TCHECK_4(ptr->align_err);

        ND_PRINT(" ALIGN_ERR(%u)", GET_BE_U_4(ptr->align_err));
        ND_PRINT(" ALIGN_ERR(%u)", GET_BE_U_4(ptr->align_err));

-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);

 }
 
 static void
 }
 
 static void


@@ -907,19 +763,10 @@ pptp_sli_print(netdissect_options *ndo,
 {
        const struct pptp_msg_sli *ptr = (const struct pptp_msg_sli *)dat;
 
 {
        const struct pptp_msg_sli *ptr = (const struct pptp_msg_sli *)dat;
 

-       ND_TCHECK_2(ptr->peer_call_id);
-       pptp_peer_call_id_print(ndo, &ptr->peer_call_id);
-       ND_TCHECK_2(ptr->reserved1);
+       pptp_peer_call_id_print(ndo, ptr->peer_call_id);

        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);
        PRINT_RESERVED_IF_NOT_ZERO_2(ptr->reserved1);

-       ND_TCHECK_4(ptr->send_accm);

        ND_PRINT(" SEND_ACCM(0x%08x)", GET_BE_U_4(ptr->send_accm));
        ND_PRINT(" SEND_ACCM(0x%08x)", GET_BE_U_4(ptr->send_accm));

-       ND_TCHECK_4(ptr->recv_accm);

        ND_PRINT(" RECV_ACCM(0x%08x)", GET_BE_U_4(ptr->recv_accm));
        ND_PRINT(" RECV_ACCM(0x%08x)", GET_BE_U_4(ptr->recv_accm));

-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);

 }
 
 void
 }
 
 void


@@ -931,15 +778,14 @@ pptp_print(netdissect_options *ndo,
        uint16_t ctrl_msg_type;
 
        ndo->ndo_protocol = "pptp";
        uint16_t ctrl_msg_type;
 
        ndo->ndo_protocol = "pptp";

-       ND_PRINT(": pptp");
+       ND_PRINT(": ");
+       nd_print_protocol(ndo);

 
        hdr = (const struct pptp_hdr *)dat;
 
 
        hdr = (const struct pptp_hdr *)dat;
 

-       ND_TCHECK_2(hdr->length);

        if (ndo->ndo_vflag) {
                ND_PRINT(" Length=%u", GET_BE_U_2(hdr->length));
        }
        if (ndo->ndo_vflag) {
                ND_PRINT(" Length=%u", GET_BE_U_2(hdr->length));
        }

-       ND_TCHECK_2(hdr->msg_type);

        if (ndo->ndo_vflag) {
                switch(GET_BE_U_2(hdr->msg_type)) {
                case PPTP_MSG_TYPE_CTRL:
        if (ndo->ndo_vflag) {
                switch(GET_BE_U_2(hdr->msg_type)) {
                case PPTP_MSG_TYPE_CTRL:


@@ -954,7 +800,6 @@ pptp_print(netdissect_options *ndo,
                }
        }
 
                }
        }
 

-       ND_TCHECK_4(hdr->magic_cookie);

        mc = GET_BE_U_4(hdr->magic_cookie);
        if (mc != PPTP_MAGIC_COOKIE) {
                ND_PRINT(" UNEXPECTED Magic-Cookie!!(%08x)", mc);
        mc = GET_BE_U_4(hdr->magic_cookie);
        if (mc != PPTP_MAGIC_COOKIE) {
                ND_PRINT(" UNEXPECTED Magic-Cookie!!(%08x)", mc);


@@ -962,7 +807,6 @@ pptp_print(netdissect_options *ndo,
        if (ndo->ndo_vflag || mc != PPTP_MAGIC_COOKIE) {
                ND_PRINT(" Magic-Cookie=%08x", mc);
        }
        if (ndo->ndo_vflag || mc != PPTP_MAGIC_COOKIE) {
                ND_PRINT(" Magic-Cookie=%08x", mc);
        }

-       ND_TCHECK_2(hdr->ctrl_msg_type);

        ctrl_msg_type = GET_BE_U_2(hdr->ctrl_msg_type);
        if (ctrl_msg_type < PPTP_MAX_MSGTYPE_INDEX) {
                ND_PRINT(" CTRL_MSGTYPE=%s",
        ctrl_msg_type = GET_BE_U_2(hdr->ctrl_msg_type);
        if (ctrl_msg_type < PPTP_MAX_MSGTYPE_INDEX) {
                ND_PRINT(" CTRL_MSGTYPE=%s",


@@ -970,7 +814,6 @@ pptp_print(netdissect_options *ndo,
        } else {
                ND_PRINT(" UNKNOWN_CTRL_MSGTYPE(%u)", ctrl_msg_type);
        }
        } else {
                ND_PRINT(" UNKNOWN_CTRL_MSGTYPE(%u)", ctrl_msg_type);
        }

-       ND_TCHECK_2(hdr->reserved0);

        PRINT_RESERVED_IF_NOT_ZERO_2(hdr->reserved0);
 
        dat += 12;
        PRINT_RESERVED_IF_NOT_ZERO_2(hdr->reserved0);
 
        dat += 12;


@@ -1025,9 +868,4 @@ pptp_print(netdissect_options *ndo,
                /* do nothing */
                break;
        }
                /* do nothing */
                break;
        }

-
-       return;
-
-trunc:
-       nd_print_trunc(ndo);

 }
 }