X-Git-Url: https://git.tcpdump.org/tcpdump/blobdiff_plain/ed85e20e4d6a27d5405f37366dd34b64c10a9211..refs/pull/494/head:/print-mpcp.c diff --git a/print-mpcp.c b/print-mpcp.c index 175aa758..5bd04669 100644 --- a/print-mpcp.c +++ b/print-mpcp.c @@ -17,14 +17,13 @@ * Original code by Hannes Gredler (hannes@juniper.net) */ -#define NETDISSECT_REWORKED #ifdef HAVE_CONFIG_H #include "config.h" #endif -#include +#include -#include "interface.h" +#include "netdissect.h" #include "extract.h" #define MPCP_TIMESTAMP_LEN 4 @@ -122,8 +121,8 @@ static const struct tok mpcp_reg_ack_flag_values[] = { }; void -mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int length) { - +mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int length) +{ union { const struct mpcp_common_header_t *common_header; const struct mpcp_grant_t *grant; @@ -141,8 +140,7 @@ mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int tptr=pptr; mpcp.common_header = (const struct mpcp_common_header_t *)pptr; - if (!ND_TTEST2(*tptr, sizeof(const struct mpcp_common_header_t))) - goto trunc; + ND_TCHECK2(*tptr, sizeof(const struct mpcp_common_header_t)); opcode = EXTRACT_16BITS(mpcp.common_header->opcode); ND_PRINT((ndo, "MPCP, Opcode %s", tok2str(mpcp_opcode_values, "Unknown (%u)", opcode))); if (opcode != MPCP_OPCODE_PAUSE) { @@ -160,8 +158,7 @@ mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int break; case MPCP_OPCODE_GATE: - if (!ND_TTEST2(*tptr, MPCP_GRANT_NUMBER_LEN)) - goto trunc; + ND_TCHECK2(*tptr, MPCP_GRANT_NUMBER_LEN); grant_numbers = *tptr & MPCP_GRANT_NUMBER_MASK; ND_PRINT((ndo, "\n\tGrant Numbers %u, Flags [ %s ]", grant_numbers, @@ -171,8 +168,7 @@ mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int tptr++; for (grant = 1; grant <= grant_numbers; grant++) { - if (!ND_TTEST2(*tptr, sizeof(const struct mpcp_grant_t))) - goto trunc; + ND_TCHECK2(*tptr, sizeof(const struct mpcp_grant_t)); mpcp.grant = (const struct mpcp_grant_t *)tptr; ND_PRINT((ndo, "\n\tGrant #%u, Start-Time %u ticks, duration %u ticks", grant, @@ -181,22 +177,19 @@ mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int tptr += sizeof(const struct mpcp_grant_t); } - if (!ND_TTEST2(*tptr, MPCP_TIMESTAMP_DURATION_LEN)) - goto trunc; + ND_TCHECK2(*tptr, MPCP_TIMESTAMP_DURATION_LEN); ND_PRINT((ndo, "\n\tSync-Time %u ticks", EXTRACT_16BITS(tptr))); break; case MPCP_OPCODE_REPORT: - if (!ND_TTEST2(*tptr, MPCP_REPORT_QUEUESETS_LEN)) - goto trunc; + ND_TCHECK2(*tptr, MPCP_REPORT_QUEUESETS_LEN); queue_sets = *tptr; tptr+=MPCP_REPORT_QUEUESETS_LEN; ND_PRINT((ndo, "\n\tTotal Queue-Sets %u", queue_sets)); for (queue_set = 1; queue_set < queue_sets; queue_set++) { - if (!ND_TTEST2(*tptr, MPCP_REPORT_REPORTBITMAP_LEN)) - goto trunc; + ND_TCHECK2(*tptr, MPCP_REPORT_REPORTBITMAP_LEN); report_bitmap = *(tptr); ND_PRINT((ndo, "\n\t Queue-Set #%u, Report-Bitmap [ %s ]", queue_sets, @@ -206,8 +199,7 @@ mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int report=1; while (report_bitmap != 0) { if (report_bitmap & 1) { - if (!ND_TTEST2(*tptr, MPCP_TIMESTAMP_DURATION_LEN)) - goto trunc; + ND_TCHECK2(*tptr, MPCP_TIMESTAMP_DURATION_LEN); ND_PRINT((ndo, "\n\t Q%u Report, Duration %u ticks", report, EXTRACT_16BITS(tptr))); @@ -220,8 +212,7 @@ mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int break; case MPCP_OPCODE_REG_REQ: - if (!ND_TTEST2(*tptr, sizeof(const struct mpcp_reg_req_t))) - goto trunc; + ND_TCHECK2(*tptr, sizeof(const struct mpcp_reg_req_t)); mpcp.reg_req = (const struct mpcp_reg_req_t *)tptr; ND_PRINT((ndo, "\n\tFlags [ %s ], Pending-Grants %u", bittok2str(mpcp_reg_req_flag_values, "Reserved", mpcp.reg_req->flags), @@ -229,8 +220,7 @@ mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int break; case MPCP_OPCODE_REG: - if (!ND_TTEST2(*tptr, sizeof(const struct mpcp_reg_t))) - goto trunc; + ND_TCHECK2(*tptr, sizeof(const struct mpcp_reg_t)); mpcp.reg = (const struct mpcp_reg_t *)tptr; ND_PRINT((ndo, "\n\tAssigned-Port %u, Flags [ %s ]" \ "\n\tSync-Time %u ticks, Echoed-Pending-Grants %u", @@ -241,8 +231,7 @@ mpcp_print(netdissect_options *ndo, register const u_char *pptr, register u_int break; case MPCP_OPCODE_REG_ACK: - if (!ND_TTEST2(*tptr, sizeof(const struct mpcp_reg_ack_t))) - goto trunc; + ND_TCHECK2(*tptr, sizeof(const struct mpcp_reg_ack_t)); mpcp.reg_ack = (const struct mpcp_reg_ack_t *)tptr; ND_PRINT((ndo, "\n\tEchoed-Assigned-Port %u, Flags [ %s ]" \ "\n\tEchoed-Sync-Time %u ticks",