aboutsummaryrefslogtreecommitdiff
path: root/print-forces.c
diff options
context:
space:
mode:
Diffstat (limited to 'print-forces.c')
-rw-r--r--print-forces.c761
1 files changed, 358 insertions, 403 deletions
diff --git a/print-forces.c b/print-forces.c
index de6c8264ddc5..1c7beb6a7692 100644
--- a/print-forces.c
+++ b/print-forces.c
@@ -19,15 +19,14 @@
/* specification: RFC 5810 */
#ifdef HAVE_CONFIG_H
-#include "config.h"
+#include <config.h>
#endif
-#include <netdissect-stdinc.h>
+#include "netdissect-stdinc.h"
#include "netdissect.h"
#include "extract.h"
-static const char tstr[] = "[|forces]";
#define ForCES_VERS 1
#define ForCES_HDRL 24
@@ -35,17 +34,17 @@ static const char tstr[] = "[|forces]";
#define TLV_HDRL 4
#define ILV_HDRL 8
-#define TOM_RSVD 0x0
-#define TOM_ASSNSETUP 0x1
-#define TOM_ASSNTEARD 0x2
-#define TOM_CONFIG 0x3
-#define TOM_QUERY 0x4
-#define TOM_EVENTNOT 0x5
-#define TOM_PKTREDIR 0x6
-#define TOM_HEARTBT 0x0F
-#define TOM_ASSNSETREP 0x11
-#define TOM_CONFIGREP 0x13
-#define TOM_QUERYREP 0x14
+#define TOM_RSVD 0x0
+#define TOM_ASSNSETUP 0x1
+#define TOM_ASSNTEARD 0x2
+#define TOM_CONFIG 0x3
+#define TOM_QUERY 0x4
+#define TOM_EVENTNOT 0x5
+#define TOM_PKTREDIR 0x6
+#define TOM_HEARTBT 0x0F
+#define TOM_ASSNSETREP 0x11
+#define TOM_CONFIGREP 0x13
+#define TOM_QUERYREP 0x14
/*
* tom_h Flags: resv1(8b):maxtlvs(4b):resv2(2b):mintlv(2b)
@@ -66,7 +65,7 @@ struct tom_h {
uint16_t flags;
uint16_t op_msk;
const char *s;
- int (*print) (netdissect_options *ndo, register const u_char * pptr, register u_int len,
+ int (*print) (netdissect_options *ndo, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
};
@@ -86,7 +85,8 @@ enum {
};
#define TOM_MAX_IND (_TOM_RSV_MAX - 1)
-static inline int tom_valid(uint8_t tom)
+static int
+tom_valid(uint8_t tom)
{
if (tom > 0) {
if (tom >= 0x7 && tom <= 0xe)
@@ -100,7 +100,8 @@ static inline int tom_valid(uint8_t tom)
return 0;
}
-static inline const char *ForCES_node(uint32_t node)
+static const char *
+ForCES_node(uint32_t node)
{
if (node <= 0x3FFFFFFF)
return "FE";
@@ -154,23 +155,23 @@ static const struct tok ForCES_TPs[] = {
*/
struct forcesh {
nd_uint8_t fm_vrsvd; /* version and reserved */
-#define ForCES_V(forcesh) ((forcesh)->fm_vrsvd >> 4)
+#define ForCES_V(forcesh) (GET_U_1((forcesh)->fm_vrsvd) >> 4)
nd_uint8_t fm_tom; /* type of message */
nd_uint16_t fm_len; /* total length * 4 bytes */
-#define ForCES_BLN(forcesh) ((uint32_t)(EXTRACT_16BITS(&(forcesh)->fm_len) << 2))
+#define ForCES_BLN(forcesh) ((uint32_t)(GET_BE_U_2((forcesh)->fm_len) << 2))
nd_uint32_t fm_sid; /* Source ID */
-#define ForCES_SID(forcesh) EXTRACT_32BITS(&(forcesh)->fm_sid)
+#define ForCES_SID(forcesh) GET_BE_U_4((forcesh)->fm_sid)
nd_uint32_t fm_did; /* Destination ID */
-#define ForCES_DID(forcesh) EXTRACT_32BITS(&(forcesh)->fm_did)
+#define ForCES_DID(forcesh) GET_BE_U_4((forcesh)->fm_did)
nd_uint8_t fm_cor[8]; /* correlator */
nd_uint32_t fm_flags; /* flags */
-#define ForCES_ACK(forcesh) ((EXTRACT_32BITS(&(forcesh)->fm_flags)&0xC0000000) >> 30)
-#define ForCES_PRI(forcesh) ((EXTRACT_32BITS(&(forcesh)->fm_flags)&0x38000000) >> 27)
-#define ForCES_RS1(forcesh) ((EXTRACT_32BITS(&(forcesh)->fm_flags)&0x07000000) >> 24)
-#define ForCES_EM(forcesh) ((EXTRACT_32BITS(&(forcesh)->fm_flags)&0x00C00000) >> 22)
-#define ForCES_AT(forcesh) ((EXTRACT_32BITS(&(forcesh)->fm_flags)&0x00200000) >> 21)
-#define ForCES_TP(forcesh) ((EXTRACT_32BITS(&(forcesh)->fm_flags)&0x00180000) >> 19)
-#define ForCES_RS2(forcesh) ((EXTRACT_32BITS(&(forcesh)->fm_flags)&0x0007FFFF) >> 0)
+#define ForCES_ACK(forcesh) ((GET_BE_U_4((forcesh)->fm_flags)&0xC0000000) >> 30)
+#define ForCES_PRI(forcesh) ((GET_BE_U_4((forcesh)->fm_flags)&0x38000000) >> 27)
+#define ForCES_RS1(forcesh) ((GET_BE_U_4((forcesh)->fm_flags)&0x07000000) >> 24)
+#define ForCES_EM(forcesh) ((GET_BE_U_4((forcesh)->fm_flags)&0x00C00000) >> 22)
+#define ForCES_AT(forcesh) ((GET_BE_U_4((forcesh)->fm_flags)&0x00200000) >> 21)
+#define ForCES_TP(forcesh) ((GET_BE_U_4((forcesh)->fm_flags)&0x00180000) >> 19)
+#define ForCES_RS2(forcesh) ((GET_BE_U_4((forcesh)->fm_flags)&0x0007FFFF) >> 0)
};
#define ForCES_HLN_VALID(fhl,tlen) ((tlen) >= ForCES_HDRL && \
@@ -188,7 +189,7 @@ static const struct tok ForCES_LFBs[] = {
};
/* this is defined in RFC5810 section A.2 */
-/* http://www.iana.org/assignments/forces/forces.xhtml#oper-tlv-types */
+/* https://www.iana.org/assignments/forces/forces.xhtml#oper-tlv-types */
enum {
F_OP_RSV = 0,
F_OP_SET = 1,
@@ -230,15 +231,15 @@ struct optlv_h {
uint16_t flags;
uint16_t op_msk;
const char *s;
- int (*print) (netdissect_options *ndo, register const u_char * pptr, register u_int len,
+ int (*print) (netdissect_options *ndo, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
};
-static int genoptlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int genoptlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
-static int recpdoptlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int recpdoptlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
-static int invoptlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int invoptlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
#define OP_MIN_SIZ 8
@@ -248,7 +249,7 @@ struct pathdata_h {
};
#define B_FULLD 0x1
-#define B_SPARD 0x2
+#define B_SPARD 0x2
#define B_RESTV 0x4
#define B_KEYIN 0x8
#define B_APPND 0x10
@@ -276,9 +277,10 @@ static const struct optlv_h OPTLV_msg[F_OP_MAX + 1] = {
/* F_OP_RTRCOMP */ {ZERO_TTLV, 0, " RTRCOMP", NULL},
};
-static inline const struct optlv_h *get_forces_optlv_h(uint16_t opt)
+static const struct optlv_h *
+get_forces_optlv_h(uint16_t opt)
{
- if (opt > F_OP_MAX || opt <= F_OP_RSV)
+ if (opt > F_OP_MAX || opt == F_OP_RSV)
return &OPTLV_msg[F_OP_RSV];
return &OPTLV_msg[opt];
@@ -290,7 +292,8 @@ static inline const struct optlv_h *get_forces_optlv_h(uint16_t opt)
#define IND_SUF 0x0
static char ind_buf[IND_SIZE];
-static inline char *indent_pr(int indent, int nlpref)
+static char *
+indent_pr(int indent, int nlpref)
{
int i = 0;
char *r = ind_buf;
@@ -311,14 +314,13 @@ static inline char *indent_pr(int indent, int nlpref)
return r;
}
-static inline int op_valid(uint16_t op, uint16_t mask)
+static int
+op_valid(uint16_t op, uint16_t mask)
{
- int opb = 1 << (op - 1);
-
if (op == 0)
return 0;
- if (opb & mask)
- return 1;
+ if (op <= F_OP_MAX)
+ return (1 << (op - 1)) & mask; /* works only for 0x0001 through 0x0010 */
/* I guess we should allow vendor operations? */
if (op >= 0x8000)
return 1;
@@ -359,7 +361,8 @@ static const struct tok ForCES_TLV[] = {
};
#define TLV_HLN 4
-static inline int ttlv_valid(uint16_t ttlv)
+static int
+ttlv_valid(uint16_t ttlv)
{
if (ttlv > 0) {
if (ttlv == 1 || ttlv == 0x1000)
@@ -385,22 +388,18 @@ struct forces_tlv {
nd_uint16_t length;
};
-#define F_ALN_LEN(len) ( ((len)+ForCES_ALNL-1) & ~(ForCES_ALNL-1) )
+#define F_ALN_LEN(len) roundup2(len, ForCES_ALNL)
#define GET_TOP_TLV(fhdr) ((const struct forces_tlv *)((fhdr) + sizeof (struct forcesh)))
#define TLV_SET_LEN(len) (F_ALN_LEN(TLV_HDRL) + (len))
-#define TLV_ALN_LEN(len) F_ALN_LEN(TLV_SET_LEN(len))
-#define TLV_RDAT_LEN(tlv) ((int)(EXTRACT_16BITS(&(tlv)->length) - TLV_SET_LEN(0))
#define TLV_DATA(tlvp) ((const void*)(((const char*)(tlvp)) + TLV_SET_LEN(0)))
-#define GO_NXT_TLV(tlv,rlen) ((rlen) -= F_ALN_LEN(EXTRACT_16BITS(&(tlv)->length)), \
+#define GO_NXT_TLV(tlv,rlen) ((rlen) -= F_ALN_LEN(GET_BE_U_2((tlv)->length)), \
(const struct forces_tlv*)(((const char*)(tlv)) \
- + F_ALN_LEN(EXTRACT_16BITS(&(tlv)->length))))
+ + F_ALN_LEN(GET_BE_U_2((tlv)->length))))
#define ILV_SET_LEN(len) (F_ALN_LEN(ILV_HDRL) + (len))
-#define ILV_ALN_LEN(len) F_ALN_LEN(ILV_SET_LEN(len))
-#define ILV_RDAT_LEN(ilv) ((int)(EXTRACT_32BITS(&(ilv)->length)) - ILV_SET_LEN(0))
#define ILV_DATA(ilvp) ((const void*)(((const char*)(ilvp)) + ILV_SET_LEN(0)))
-#define GO_NXT_ILV(ilv,rlen) ((rlen) -= F_ALN_LEN(EXTRACT_32BITS(&(ilv)->length)), \
+#define GO_NXT_ILV(ilv,rlen) ((rlen) -= F_ALN_LEN(GET_BE_U_4((ilv)->length)), \
(const struct forces_ilv *)(((const char*)(ilv)) \
- + F_ALN_LEN(EXTRACT_32BITS(&(ilv)->length))))
+ + F_ALN_LEN(GET_BE_U_4((ilv)->length))))
#define INVALID_RLEN 1
#define INVALID_STLN 2
#define INVALID_LTLN 3
@@ -414,41 +413,43 @@ static const struct tok ForCES_TLV_err[] = {
{0, NULL}
};
-static inline u_int tlv_valid(const struct forces_tlv *tlv, u_int rlen)
+static u_int
+tlv_valid(u_int tlvl, u_int rlen)
{
if (rlen < TLV_HDRL)
return INVALID_RLEN;
- if (EXTRACT_16BITS(&tlv->length) < TLV_HDRL)
+ if (tlvl < TLV_HDRL)
return INVALID_STLN;
- if (EXTRACT_16BITS(&tlv->length) > rlen)
+ if (tlvl > rlen)
return INVALID_LTLN;
- if (rlen < F_ALN_LEN(EXTRACT_16BITS(&tlv->length)))
+ if (rlen < F_ALN_LEN(tlvl))
return INVALID_ALEN;
return 0;
}
-static inline int ilv_valid(const struct forces_ilv *ilv, u_int rlen)
+static int
+ilv_valid(netdissect_options *ndo, const struct forces_ilv *ilv, u_int rlen)
{
if (rlen < ILV_HDRL)
return INVALID_RLEN;
- if (EXTRACT_32BITS(&ilv->length) < ILV_HDRL)
+ if (GET_BE_U_4(ilv->length) < ILV_HDRL)
return INVALID_STLN;
- if (EXTRACT_32BITS(&ilv->length) > rlen)
+ if (GET_BE_U_4(ilv->length) > rlen)
return INVALID_LTLN;
- if (rlen < F_ALN_LEN(EXTRACT_32BITS(&ilv->length)))
+ if (rlen < F_ALN_LEN(GET_BE_U_4(ilv->length)))
return INVALID_ALEN;
return 0;
}
-static int lfbselect_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int lfbselect_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
-static int redirect_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int redirect_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
-static int asrtlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int asrtlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
-static int asttlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int asttlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
struct forces_lfbsh {
@@ -484,7 +485,8 @@ static const struct tom_h ForCES_msg[TOM_MAX_IND + 1] = {
{TOM_QUERYREP, TTLV_T2, CFG_QYR, "Query Response", lfbselect_print},
};
-static inline const struct tom_h *get_forces_tom(uint8_t tom)
+static const struct tom_h *
+get_forces_tom(uint8_t tom)
{
int i;
for (i = TOM_RSV_I; i <= TOM_MAX_IND; i++) {
@@ -500,7 +502,7 @@ struct pdata_ops {
uint16_t flags;
uint16_t op_msk;
const char *s;
- int (*print) (netdissect_options *, register const u_char * pptr, register u_int len,
+ int (*print) (netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
};
@@ -515,26 +517,27 @@ enum {
};
#define PD_MAX_IND (_TOM_RSV_MAX - 1)
-static inline int pd_valid(uint16_t pd)
+static int
+pd_valid(uint16_t pd)
{
if (pd >= F_TLV_PDAT && pd <= F_TLV_REST)
return 1;
return 0;
}
-static inline void
+static void
chk_op_type(netdissect_options *ndo,
uint16_t type, uint16_t msk, uint16_t omsk)
{
if (type != F_TLV_PDAT) {
if (msk & B_KEYIN) {
if (type != F_TLV_KEYI) {
- ND_PRINT((ndo, "Based on flags expected KEYINFO TLV!\n"));
+ ND_PRINT("Based on flags expected KEYINFO TLV!\n");
}
} else {
if (!(msk & omsk)) {
- ND_PRINT((ndo, "Illegal DATA encoding for type 0x%x programmed %x got %x \n",
- type, omsk, msk));
+ ND_PRINT("Illegal DATA encoding for type 0x%x programmed %x got %x\n",
+ type, omsk, msk);
}
}
}
@@ -551,13 +554,13 @@ struct res_val {
nd_uint16_t resv2;
};
-static int prestlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int prestlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
-static int pkeyitlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int pkeyitlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
-static int fdatatlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int fdatatlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
-static int sdatatlv_print(netdissect_options *, register const u_char * pptr, register u_int len,
+static int sdatatlv_print(netdissect_options *, const u_char * pptr, u_int len,
uint16_t op_msk, int indent);
static const struct pdata_ops ForCES_pdata[PD_MAX_IND + 1] = {
@@ -570,7 +573,8 @@ static const struct pdata_ops ForCES_pdata[PD_MAX_IND + 1] = {
{F_TLV_PDAT, 0, 0, "Inner PATH-DATA TLV", recpdoptlv_print},
};
-static inline const struct pdata_ops *get_forces_pd(uint16_t pd)
+static const struct pdata_ops *
+get_forces_pd(uint16_t pd)
{
int i;
for (i = PD_RSV_I + 1; i <= PD_MAX_IND; i++) {
@@ -643,13 +647,14 @@ static const struct tok ForCES_errs[] = {
static int
prestlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
const struct forces_tlv *tlv = (const struct forces_tlv *)pptr;
- register const u_char *tdp = (const u_char *) TLV_DATA(tlv);
+ const u_char *tdp = (const u_char *) TLV_DATA(tlv);
const struct res_val *r = (const struct res_val *)tdp;
u_int dlen;
+ uint8_t result;
/*
* pdatacnt_print() has ensured that len (the TLV length)
@@ -657,36 +662,37 @@ prestlv_print(netdissect_options *ndo,
*/
dlen = len - TLV_HDRL;
if (dlen != RESLEN) {
- ND_PRINT((ndo, "illegal RESULT-TLV: %d bytes!\n", dlen));
+ ND_PRINT("illegal RESULT-TLV: %u bytes!\n", dlen);
return -1;
}
- ND_TCHECK(*r);
- if (r->result >= 0x18 && r->result <= 0xFE) {
- ND_PRINT((ndo, "illegal reserved result code: 0x%x!\n", r->result));
+ ND_TCHECK_SIZE(r);
+ result = GET_U_1(r->result);
+ if (result >= 0x18 && result <= 0xFE) {
+ ND_PRINT("illegal reserved result code: 0x%x!\n", result);
return -1;
}
if (ndo->ndo_vflag >= 3) {
char *ib = indent_pr(indent, 0);
- ND_PRINT((ndo, "%s Result: %s (code 0x%x)\n", ib,
- tok2str(ForCES_errs, NULL, r->result), r->result));
+ ND_PRINT("%s Result: %s (code 0x%x)\n", ib,
+ tok2str(ForCES_errs, NULL, result), result);
}
return 0;
trunc:
- ND_PRINT((ndo, "%s", tstr));
+ nd_print_trunc(ndo);
return -1;
}
static int
fdatatlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
const struct forces_tlv *tlv = (const struct forces_tlv *)pptr;
u_int rlen;
- register const u_char *tdp = (const u_char *) TLV_DATA(tlv);
+ const u_char *tdp = (const u_char *) TLV_DATA(tlv);
uint16_t type;
/*
@@ -694,29 +700,29 @@ fdatatlv_print(netdissect_options *ndo,
* (the TLV length) >= TLV_HDRL.
*/
rlen = len - TLV_HDRL;
- ND_TCHECK(*tlv);
- type = EXTRACT_16BITS(&tlv->type);
+ ND_TCHECK_SIZE(tlv);
+ type = GET_BE_U_2(tlv->type);
if (type != F_TLV_FULD) {
- ND_PRINT((ndo, "Error: expecting FULLDATA!\n"));
+ ND_PRINT("Error: expecting FULLDATA!\n");
return -1;
}
if (ndo->ndo_vflag >= 3) {
char *ib = indent_pr(indent + 2, 1);
- ND_PRINT((ndo, "%s[", &ib[1]));
- hex_print_with_offset(ndo, ib, tdp, rlen, 0);
- ND_PRINT((ndo, "\n%s]\n", &ib[1]));
+ ND_PRINT("%s[", ib + 1);
+ hex_print(ndo, ib, tdp, rlen);
+ ND_PRINT("\n%s]", ib + 1);
}
return 0;
trunc:
- ND_PRINT((ndo, "%s", tstr));
+ nd_print_trunc(ndo);
return -1;
}
static int
sdatailv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
u_int rlen;
@@ -724,50 +730,44 @@ sdatailv_print(netdissect_options *ndo,
int invilv;
if (len < ILV_HDRL) {
- ND_PRINT((ndo, "Error: BAD SPARSEDATA-TLV!\n"));
+ ND_PRINT("Error: BAD SPARSEDATA-TLV!\n");
return -1;
}
rlen = len;
indent += 1;
while (rlen != 0) {
#if 0
- ND_PRINT((ndo, "Jamal - outstanding length <%d>\n", rlen));
+ ND_PRINT("Jamal - outstanding length <%u>\n", rlen);
#endif
char *ib = indent_pr(indent, 1);
- register const u_char *tdp = (const u_char *) ILV_DATA(ilv);
- ND_TCHECK(*ilv);
- invilv = ilv_valid(ilv, rlen);
+ const u_char *tdp = (const u_char *) ILV_DATA(ilv);
+ invilv = ilv_valid(ndo, ilv, rlen);
if (invilv) {
- ND_PRINT((ndo, "%s[", &ib[1]));
- hex_print_with_offset(ndo, ib, tdp, rlen, 0);
- ND_PRINT((ndo, "\n%s]\n", &ib[1]));
+ ND_PRINT("Error: %s, rlen %u\n",
+ tok2str(ForCES_TLV_err, NULL, invilv), rlen);
return -1;
}
if (ndo->ndo_vflag >= 3) {
- int ilvl = EXTRACT_32BITS(&ilv->length);
- ND_PRINT((ndo, "\n%s ILV: type %x length %d\n", &ib[1],
- EXTRACT_32BITS(&ilv->type), ilvl));
- hex_print_with_offset(ndo, "\t\t[", tdp, ilvl-ILV_HDRL, 0);
+ u_int ilvl = GET_BE_U_4(ilv->length);
+ ND_PRINT("\n%s ILV: type %x length %u\n", ib + 1,
+ GET_BE_U_4(ilv->type), ilvl);
+ hex_print(ndo, "\t\t[", tdp, ilvl-ILV_HDRL);
}
ilv = GO_NXT_ILV(ilv, rlen);
}
return 0;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
static int
sdatatlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk, int indent)
{
const struct forces_tlv *tlv = (const struct forces_tlv *)pptr;
u_int rlen;
- register const u_char *tdp = (const u_char *) TLV_DATA(tlv);
+ const u_char *tdp = (const u_char *) TLV_DATA(tlv);
uint16_t type;
/*
@@ -775,45 +775,44 @@ sdatatlv_print(netdissect_options *ndo,
* >= TLV_HDRL.
*/
rlen = len - TLV_HDRL;
- ND_TCHECK(*tlv);
- type = EXTRACT_16BITS(&tlv->type);
+ ND_TCHECK_SIZE(tlv);
+ type = GET_BE_U_2(tlv->type);
if (type != F_TLV_SPAD) {
- ND_PRINT((ndo, "Error: expecting SPARSEDATA!\n"));
+ ND_PRINT("Error: expecting SPARSEDATA!\n");
return -1;
}
return sdatailv_print(ndo, tdp, rlen, op_msk, indent);
trunc:
- ND_PRINT((ndo, "%s", tstr));
+ nd_print_trunc(ndo);
return -1;
}
static int
pkeyitlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk, int indent)
{
const struct forces_tlv *tlv = (const struct forces_tlv *)pptr;
- register const u_char *tdp = (const u_char *) TLV_DATA(tlv);
- register const u_char *dp = tdp + 4;
+ const u_char *tdp = (const u_char *) TLV_DATA(tlv);
+ const u_char *dp = tdp + 4;
const struct forces_tlv *kdtlv = (const struct forces_tlv *)dp;
uint32_t id;
char *ib = indent_pr(indent, 0);
uint16_t type, tll;
u_int invtlv;
- ND_TCHECK(*tdp);
- id = EXTRACT_32BITS(tdp);
- ND_PRINT((ndo, "%sKeyinfo: Key 0x%x\n", ib, id));
- ND_TCHECK(*kdtlv);
- type = EXTRACT_16BITS(&kdtlv->type);
- invtlv = tlv_valid(kdtlv, len);
+ id = GET_BE_U_4(tdp);
+ ND_PRINT("%sKeyinfo: Key 0x%x\n", ib, id);
+ type = GET_BE_U_2(kdtlv->type);
+ tll = GET_BE_U_2(kdtlv->length);
+ invtlv = tlv_valid(tll, len);
if (invtlv) {
- ND_PRINT((ndo, "%s TLV type 0x%x len %d\n",
+ ND_PRINT("%s TLV type 0x%x len %u\n",
tok2str(ForCES_TLV_err, NULL, invtlv), type,
- EXTRACT_16BITS(&kdtlv->length)));
+ tll);
return -1;
}
/*
@@ -821,20 +820,16 @@ pkeyitlv_print(netdissect_options *ndo,
* length is large enough but not too large (it doesn't
* go past the end of the containing TLV).
*/
- tll = EXTRACT_16BITS(&kdtlv->length);
+ tll = GET_BE_U_2(kdtlv->length);
dp = (const u_char *) TLV_DATA(kdtlv);
return fdatatlv_print(ndo, dp, tll, op_msk, indent);
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
#define PTH_DESC_SIZE 12
static int
pdatacnt_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t IDcnt, uint16_t op_msk, int indent)
{
u_int i;
@@ -842,15 +837,15 @@ pdatacnt_print(netdissect_options *ndo,
char *ib = indent_pr(indent, 0);
if ((op_msk & B_APPND) && ndo->ndo_vflag >= 3) {
- ND_PRINT((ndo, "%sTABLE APPEND\n", ib));
+ ND_PRINT("%sTABLE APPEND\n", ib);
}
for (i = 0; i < IDcnt; i++) {
- ND_TCHECK2(*pptr, 4);
+ ND_TCHECK_4(pptr);
if (len < 4)
goto trunc;
- id = EXTRACT_32BITS(pptr);
+ id = GET_BE_U_4(pptr);
if (ndo->ndo_vflag >= 3)
- ND_PRINT((ndo, "%sID#%02u: %d\n", ib, i + 1, id));
+ ND_PRINT("%sID#%02u: %u\n", ib, i + 1, id);
len -= 4;
pptr += 4;
}
@@ -860,24 +855,24 @@ pdatacnt_print(netdissect_options *ndo,
uint32_t starti, endi;
if (len < PTH_DESC_SIZE) {
- ND_PRINT((ndo, "pathlength %d with key/range too short %d\n",
- len, PTH_DESC_SIZE));
+ ND_PRINT("pathlength %u with key/range too short %u\n",
+ len, PTH_DESC_SIZE);
return -1;
}
pptr += sizeof(struct forces_tlv);
len -= sizeof(struct forces_tlv);
- starti = EXTRACT_32BITS(pptr);
+ starti = GET_BE_U_4(pptr);
pptr += 4;
len -= 4;
- endi = EXTRACT_32BITS(pptr);
+ endi = GET_BE_U_4(pptr);
pptr += 4;
len -= 4;
if (ndo->ndo_vflag >= 3)
- ND_PRINT((ndo, "%sTable range: [%d,%d]\n", ib, starti, endi));
+ ND_PRINT("%sTable range: [%u,%u]\n", ib, starti, endi);
}
if (op_msk & B_KEYIN) {
@@ -885,8 +880,8 @@ pdatacnt_print(netdissect_options *ndo,
uint16_t tll;
if (len < PTH_DESC_SIZE) {
- ND_PRINT((ndo, "pathlength %d with key/range too short %d\n",
- len, PTH_DESC_SIZE));
+ ND_PRINT("pathlength %u with key/range too short %u\n",
+ len, PTH_DESC_SIZE);
return -1;
}
@@ -898,15 +893,15 @@ pdatacnt_print(netdissect_options *ndo,
pptr += sizeof(struct forces_tlv);
len -= sizeof(struct forces_tlv);
/* skip key content */
- tll = EXTRACT_16BITS(&keytlv->length);
+ tll = GET_BE_U_2(keytlv->length);
if (tll < TLV_HDRL) {
- ND_PRINT((ndo, "key content length %u < %u\n",
- tll, TLV_HDRL));
+ ND_PRINT("key content length %u < %u\n",
+ tll, TLV_HDRL);
return -1;
}
tll -= TLV_HDRL;
if (len < tll) {
- ND_PRINT((ndo, "key content too short\n"));
+ ND_PRINT("key content too short\n");
return -1;
}
pptr += tll;
@@ -918,18 +913,18 @@ pdatacnt_print(netdissect_options *ndo,
if (len) {
const struct forces_tlv *pdtlv = (const struct forces_tlv *)pptr;
uint16_t type;
- uint16_t tll;
- int pad = 0;
+ uint16_t tlvl, tll;
+ u_int pad = 0;
u_int aln;
u_int invtlv;
- ND_TCHECK(*pdtlv);
- type = EXTRACT_16BITS(&pdtlv->type);
- invtlv = tlv_valid(pdtlv, len);
+ type = GET_BE_U_2(pdtlv->type);
+ tlvl = GET_BE_U_2(pdtlv->length);
+ invtlv = tlv_valid(tlvl, len);
if (invtlv) {
- ND_PRINT((ndo, "%s Outstanding bytes %d for TLV type 0x%x TLV len %d\n",
+ ND_PRINT("%s Outstanding bytes %u for TLV type 0x%x TLV len %u\n",
tok2str(ForCES_TLV_err, NULL, invtlv), len, type,
- EXTRACT_16BITS(&pdtlv->length)));
+ tlvl);
goto pd_err;
}
/*
@@ -937,15 +932,14 @@ pdatacnt_print(netdissect_options *ndo,
* length is large enough but not too large (it doesn't
* go past the end of the containing TLV).
*/
- tll = EXTRACT_16BITS(&pdtlv->length) - TLV_HDRL;
- aln = F_ALN_LEN(EXTRACT_16BITS(&pdtlv->length));
- if (aln > EXTRACT_16BITS(&pdtlv->length)) {
+ tll = tlvl - TLV_HDRL;
+ aln = F_ALN_LEN(tlvl);
+ if (aln > tlvl) {
if (aln > len) {
- ND_PRINT((ndo,
- "Invalid padded pathdata TLV type 0x%x len %d missing %d pad bytes\n",
- type, EXTRACT_16BITS(&pdtlv->length), aln - len));
+ ND_PRINT("Invalid padded pathdata TLV type 0x%x len %u missing %u pad bytes\n",
+ type, tlvl, aln - len);
} else {
- pad = aln - EXTRACT_16BITS(&pdtlv->length);
+ pad = aln - tlvl;
}
}
if (pd_valid(type)) {
@@ -953,11 +947,11 @@ pdatacnt_print(netdissect_options *ndo,
if (ndo->ndo_vflag >= 3 && ops->v != F_TLV_PDAT) {
if (pad)
- ND_PRINT((ndo, "%s %s (Length %d DataLen %d pad %d Bytes)\n",
- ib, ops->s, EXTRACT_16BITS(&pdtlv->length), tll, pad));
+ ND_PRINT("%s %s (Length %u DataLen %u pad %u Bytes)\n",
+ ib, ops->s, tlvl, tll, pad);
else
- ND_PRINT((ndo, "%s %s (Length %d DataLen %d Bytes)\n",
- ib, ops->s, EXTRACT_16BITS(&pdtlv->length), tll));
+ ND_PRINT("%s %s (Length %u DataLen %u Bytes)\n",
+ ib, ops->s, tlvl, tll);
}
chk_op_type(ndo, type, op_msk, ops->op_msk);
@@ -968,13 +962,13 @@ pdatacnt_print(netdissect_options *ndo,
return -1;
len -= (TLV_HDRL + pad + tll);
} else {
- ND_PRINT((ndo, "Invalid path data content type 0x%x len %d\n",
- type, EXTRACT_16BITS(&pdtlv->length)));
+ ND_PRINT("Invalid path data content type 0x%x len %u\n",
+ type, tlvl);
pd_err:
- if (EXTRACT_16BITS(&pdtlv->length)) {
- hex_print_with_offset(ndo, "Bad Data val\n\t [",
- pptr, len, 0);
- ND_PRINT((ndo, "]\n"));
+ if (tlvl) {
+ hex_print(ndo, "Bad Data val\n\t [",
+ pptr, len);
+ ND_PRINT("]\n");
return -1;
}
@@ -983,13 +977,13 @@ pd_err:
return len;
trunc:
- ND_PRINT((ndo, "%s", tstr));
+ nd_print_trunc(ndo);
return -1;
}
static int
pdata_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk, int indent)
{
const struct pathdata_h *pdh = (const struct pathdata_h *)pptr;
@@ -998,41 +992,42 @@ pdata_print(netdissect_options *ndo,
int more_pd = 0;
uint16_t idcnt = 0;
- ND_TCHECK(*pdh);
+ ND_TCHECK_SIZE(pdh);
if (len < sizeof(struct pathdata_h))
goto trunc;
if (ndo->ndo_vflag >= 3) {
- ND_PRINT((ndo, "\n%sPathdata: Flags 0x%x ID count %d\n",
- ib, EXTRACT_16BITS(&pdh->pflags), EXTRACT_16BITS(&pdh->pIDcnt)));
+ ND_PRINT("\n%sPathdata: Flags 0x%x ID count %u\n",
+ ib, GET_BE_U_2(pdh->pflags),
+ GET_BE_U_2(pdh->pIDcnt));
}
- if (EXTRACT_16BITS(&pdh->pflags) & F_SELKEY) {
+ if (GET_BE_U_2(pdh->pflags) & F_SELKEY) {
op_msk |= B_KEYIN;
}
/* Table GET Range operation */
- if (EXTRACT_16BITS(&pdh->pflags) & F_SELTABRANGE) {
+ if (GET_BE_U_2(pdh->pflags) & F_SELTABRANGE) {
op_msk |= B_TRNG;
}
/* Table SET append operation */
- if (EXTRACT_16BITS(&pdh->pflags) & F_TABAPPEND) {
+ if (GET_BE_U_2(pdh->pflags) & F_TABAPPEND) {
op_msk |= B_APPND;
}
pptr += sizeof(struct pathdata_h);
len -= sizeof(struct pathdata_h);
- idcnt = EXTRACT_16BITS(&pdh->pIDcnt);
+ idcnt = GET_BE_U_2(pdh->pIDcnt);
minsize = idcnt * 4;
if (len < minsize) {
- ND_PRINT((ndo, "\t\t\ttruncated IDs expected %uB got %uB\n", minsize,
- len));
- hex_print_with_offset(ndo, "\t\t\tID Data[", pptr, len, 0);
- ND_PRINT((ndo, "]\n"));
+ ND_PRINT("\t\t\ttruncated IDs expected %uB got %uB\n", minsize,
+ len);
+ hex_print(ndo, "\t\t\tID Data[", pptr, len);
+ ND_PRINT("]\n");
return -1;
}
if ((op_msk & B_TRNG) && (op_msk & B_KEYIN)) {
- ND_PRINT((ndo, "\t\t\tIllegal to have both Table ranges and keys\n"));
+ ND_PRINT("\t\t\tIllegal to have both Table ranges and keys\n");
return -1;
}
@@ -1047,71 +1042,67 @@ pdata_print(netdissect_options *ndo,
return 0;
trunc:
- ND_PRINT((ndo, "%s", tstr));
+ nd_print_trunc(ndo);
return -1;
}
static int
genoptlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk, int indent)
{
const struct forces_tlv *pdtlv = (const struct forces_tlv *)pptr;
uint16_t type;
- int tll;
+ u_int tlvl;
u_int invtlv;
char *ib = indent_pr(indent, 0);
- ND_TCHECK(*pdtlv);
- type = EXTRACT_16BITS(&pdtlv->type);
- tll = EXTRACT_16BITS(&pdtlv->length) - TLV_HDRL;
- invtlv = tlv_valid(pdtlv, len);
- ND_PRINT((ndo, "genoptlvprint - %s TLV type 0x%x len %d\n",
- tok2str(ForCES_TLV, NULL, type), type, EXTRACT_16BITS(&pdtlv->length)));
+ type = GET_BE_U_2(pdtlv->type);
+ tlvl = GET_BE_U_2(pdtlv->length);
+ invtlv = tlv_valid(tlvl, len);
+ ND_PRINT("genoptlvprint - %s TLV type 0x%x len %u\n",
+ tok2str(ForCES_TLV, NULL, type), type, tlvl);
if (!invtlv) {
/*
* At this point, tlv_valid() has ensured that the TLV
* length is large enough but not too large (it doesn't
* go past the end of the containing TLV).
*/
- register const u_char *dp = (const u_char *) TLV_DATA(pdtlv);
+ const u_char *dp = (const u_char *) TLV_DATA(pdtlv);
+
if (!ttlv_valid(type)) {
- ND_PRINT((ndo, "%s TLV type 0x%x len %d\n",
+ ND_PRINT("%s TLV type 0x%x len %u\n",
tok2str(ForCES_TLV_err, NULL, invtlv), type,
- EXTRACT_16BITS(&pdtlv->length)));
+ tlvl);
return -1;
}
if (ndo->ndo_vflag >= 3)
- ND_PRINT((ndo, "%s%s, length %d (data length %d Bytes)",
+ ND_PRINT("%s%s, length %u (data length %u Bytes)",
ib, tok2str(ForCES_TLV, NULL, type),
- EXTRACT_16BITS(&pdtlv->length), tll));
+ tlvl, tlvl - TLV_HDRL);
- return pdata_print(ndo, dp, tll, op_msk, indent + 1);
+ return pdata_print(ndo, dp, tlvl - TLV_HDRL, op_msk, indent + 1);
} else {
- ND_PRINT((ndo, "\t\t\tInvalid ForCES TLV type=%x", type));
+ ND_PRINT("\t\t\tInvalid ForCES TLV type=%x", type);
return -1;
}
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
static int
recpdoptlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk, int indent)
{
const struct forces_tlv *pdtlv = (const struct forces_tlv *)pptr;
- int tll;
- u_int invtlv;
- uint16_t type;
- register const u_char *dp;
- char *ib;
while (len != 0) {
- ND_TCHECK(*pdtlv);
- invtlv = tlv_valid(pdtlv, len);
+ uint16_t type, tlvl;
+ u_int invtlv;
+ char *ib;
+ const u_char *dp;
+
+ tlvl = GET_BE_U_2(pdtlv->length);
+ invtlv = tlv_valid(tlvl, len);
if (invtlv) {
break;
}
@@ -1122,46 +1113,41 @@ recpdoptlv_print(netdissect_options *ndo,
* go past the end of the containing TLV).
*/
ib = indent_pr(indent, 0);
- type = EXTRACT_16BITS(&pdtlv->type);
+ type = GET_BE_U_2(pdtlv->type);
dp = (const u_char *) TLV_DATA(pdtlv);
- tll = EXTRACT_16BITS(&pdtlv->length) - TLV_HDRL;
if (ndo->ndo_vflag >= 3)
- ND_PRINT((ndo, "%s%s, length %d (data encapsulated %d Bytes)",
+ ND_PRINT("%s%s, length %u (data encapsulated %u Bytes)",
ib, tok2str(ForCES_TLV, NULL, type),
- EXTRACT_16BITS(&pdtlv->length),
- EXTRACT_16BITS(&pdtlv->length) - TLV_HDRL));
+ tlvl,
+ tlvl - TLV_HDRL);
- if (pdata_print(ndo, dp, tll, op_msk, indent + 1) == -1)
+ if (pdata_print(ndo, dp, tlvl - TLV_HDRL, op_msk, indent + 1) == -1)
return -1;
pdtlv = GO_NXT_TLV(pdtlv, len);
}
if (len) {
- ND_PRINT((ndo,
- "\n\t\tMessy PATHDATA TLV header, type (0x%x)\n\t\texcess of %d Bytes ",
- EXTRACT_16BITS(&pdtlv->type), len - EXTRACT_16BITS(&pdtlv->length)));
+ ND_PRINT("\n\t\tMessy PATHDATA TLV header, type (0x%x)\n\t\texcess of %u Bytes ",
+ GET_BE_U_2(pdtlv->type),
+ len - GET_BE_U_2(pdtlv->length));
return -1;
}
return 0;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
static int
invoptlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
char *ib = indent_pr(indent, 1);
if (ndo->ndo_vflag >= 3) {
- ND_PRINT((ndo, "%sData[", &ib[1]));
- hex_print_with_offset(ndo, ib, pptr, len, 0);
- ND_PRINT((ndo, "%s]\n", ib));
+ ND_PRINT("%sData[", ib + 1);
+ hex_print(ndo, ib, pptr, len);
+ ND_PRINT("%s]\n", ib);
}
return -1;
}
@@ -1171,30 +1157,29 @@ otlv_print(netdissect_options *ndo,
const struct forces_tlv *otlv, uint16_t op_msk _U_, int indent)
{
int rc = 0;
- register const u_char *dp = (const u_char *) TLV_DATA(otlv);
+ const u_char *dp = (const u_char *) TLV_DATA(otlv);
uint16_t type;
- int tll;
+ u_int tll;
char *ib = indent_pr(indent, 0);
const struct optlv_h *ops;
/*
- * lfbselect_print() has ensured that EXTRACT_16BITS(&otlv->length)
+ * lfbselect_print() has ensured that GET_BE_U_2(otlv->length)
* >= TLV_HDRL.
*/
- ND_TCHECK(*otlv);
- type = EXTRACT_16BITS(&otlv->type);
- tll = EXTRACT_16BITS(&otlv->length) - TLV_HDRL;
+ type = GET_BE_U_2(otlv->type);
+ tll = GET_BE_U_2(otlv->length) - TLV_HDRL;
ops = get_forces_optlv_h(type);
if (ndo->ndo_vflag >= 3) {
- ND_PRINT((ndo, "%sOper TLV %s(0x%x) length %d\n", ib, ops->s, type,
- EXTRACT_16BITS(&otlv->length)));
+ ND_PRINT("%sOper TLV %s(0x%x) length %u\n", ib, ops->s, type,
+ GET_BE_U_2(otlv->length));
}
/* rest of ops must at least have 12B {pathinfo} */
if (tll < OP_MIN_SIZ) {
- ND_PRINT((ndo, "\t\tOper TLV %s(0x%x) length %d\n", ops->s, type,
- EXTRACT_16BITS(&otlv->length)));
- ND_PRINT((ndo, "\t\tTruncated data size %d minimum required %d\n", tll,
- OP_MIN_SIZ));
+ ND_PRINT("\t\tOper TLV %s(0x%x) length %u\n", ops->s, type,
+ GET_BE_U_2(otlv->length));
+ ND_PRINT("\t\tTruncated data size %u minimum required %u\n", tll,
+ OP_MIN_SIZ);
return invoptlv_print(ndo, dp, tll, ops->op_msk, indent);
}
@@ -1204,17 +1189,13 @@ otlv_print(netdissect_options *ndo,
rc = ops->print(ndo, dp, tll, ops->op_msk, indent + 1);
}
return rc;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
#define ASTDLN 4
#define ASTMCD 255
static int
asttlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
uint32_t rescode;
@@ -1227,52 +1208,47 @@ asttlv_print(netdissect_options *ndo,
*/
dlen = len - TLV_HDRL;
if (dlen != ASTDLN) {
- ND_PRINT((ndo, "illegal ASTresult-TLV: %d bytes!\n", dlen));
+ ND_PRINT("illegal ASTresult-TLV: %u bytes!\n", dlen);
return -1;
}
- ND_TCHECK2(*pptr, 4);
- rescode = EXTRACT_32BITS(pptr);
+ rescode = GET_BE_U_4(pptr);
if (rescode > ASTMCD) {
- ND_PRINT((ndo, "illegal ASTresult result code: %d!\n", rescode));
+ ND_PRINT("illegal ASTresult result code: %u!\n", rescode);
return -1;
}
if (ndo->ndo_vflag >= 3) {
- ND_PRINT((ndo, "Teardown reason:\n%s", ib));
+ ND_PRINT("Teardown reason:\n%s", ib);
switch (rescode) {
case 0:
- ND_PRINT((ndo, "Normal Teardown"));
+ ND_PRINT("Normal Teardown");
break;
case 1:
- ND_PRINT((ndo, "Loss of Heartbeats"));
+ ND_PRINT("Loss of Heartbeats");
break;
case 2:
- ND_PRINT((ndo, "Out of bandwidth"));
+ ND_PRINT("Out of bandwidth");
break;
case 3:
- ND_PRINT((ndo, "Out of Memory"));
+ ND_PRINT("Out of Memory");
break;
case 4:
- ND_PRINT((ndo, "Application Crash"));
+ ND_PRINT("Application Crash");
break;
default:
- ND_PRINT((ndo, "Unknown Teardown reason"));
+ ND_PRINT("Unknown Teardown reason");
break;
}
- ND_PRINT((ndo, "(%x)\n%s", rescode, ib));
+ ND_PRINT("(%x)\n%s", rescode, ib);
}
return 0;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
#define ASRDLN 4
#define ASRMCD 3
static int
asrtlv_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
uint32_t rescode;
@@ -1285,40 +1261,35 @@ asrtlv_print(netdissect_options *ndo,
*/
dlen = len - TLV_HDRL;
if (dlen != ASRDLN) { /* id, instance, oper tlv */
- ND_PRINT((ndo, "illegal ASRresult-TLV: %d bytes!\n", dlen));
+ ND_PRINT("illegal ASRresult-TLV: %u bytes!\n", dlen);
return -1;
}
- ND_TCHECK2(*pptr, 4);
- rescode = EXTRACT_32BITS(pptr);
+ rescode = GET_BE_U_4(pptr);
if (rescode > ASRMCD) {
- ND_PRINT((ndo, "illegal ASRresult result code: %d!\n", rescode));
+ ND_PRINT("illegal ASRresult result code: %u!\n", rescode);
return -1;
}
if (ndo->ndo_vflag >= 3) {
- ND_PRINT((ndo, "\n%s", ib));
+ ND_PRINT("\n%s", ib);
switch (rescode) {
case 0:
- ND_PRINT((ndo, "Success "));
+ ND_PRINT("Success ");
break;
case 1:
- ND_PRINT((ndo, "FE ID invalid "));
+ ND_PRINT("FE ID invalid ");
break;
case 2:
- ND_PRINT((ndo, "permission denied "));
+ ND_PRINT("permission denied ");
break;
default:
- ND_PRINT((ndo, "Unknown "));
+ ND_PRINT("Unknown ");
break;
}
- ND_PRINT((ndo, "(%x)\n%s", rescode, ib));
+ ND_PRINT("(%x)\n%s", rescode, ib);
}
return 0;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
#if 0
@@ -1327,13 +1298,13 @@ trunc:
*/
static int
gentltlv_print(netdissect_options *ndo,
- register const u_char * pptr _U_, register u_int len,
+ const u_char * pptr _U_, u_int len,
uint16_t op_msk _U_, int indent _U_)
{
u_int dlen = len - TLV_HDRL;
if (dlen < 4) { /* at least 32 bits must exist */
- ND_PRINT((ndo, "truncated TLV: %d bytes missing! ", 4 - dlen));
+ ND_PRINT("truncated TLV: %u bytes missing! ", 4 - dlen);
return -1;
}
return 0;
@@ -1344,7 +1315,7 @@ gentltlv_print(netdissect_options *ndo,
static int
print_metailv(netdissect_options *ndo,
- register const u_char * pptr, uint16_t op_msk _U_, int indent)
+ const u_char * pptr, uint16_t op_msk _U_, int indent)
{
u_int rlen;
char *ib = indent_pr(indent, 0);
@@ -1355,24 +1326,19 @@ print_metailv(netdissect_options *ndo,
* print_metatlv() has ensured that len (what remains in the
* ILV) >= ILV_HDRL.
*/
- rlen = EXTRACT_32BITS(&ilv->length) - ILV_HDRL;
- ND_TCHECK(*ilv);
- ND_PRINT((ndo, "%sMetaID 0x%x length %d\n", ib, EXTRACT_32BITS(&ilv->type),
- EXTRACT_32BITS(&ilv->length)));
+ rlen = GET_BE_U_4(ilv->length) - ILV_HDRL;
+ ND_PRINT("%sMetaID 0x%x length %u\n", ib, GET_BE_U_4(ilv->type),
+ GET_BE_U_4(ilv->length));
if (ndo->ndo_vflag >= 3) {
- hex_print_with_offset(ndo, "\t\t[", ILV_DATA(ilv), rlen, 0);
- ND_PRINT((ndo, " ]\n"));
+ hex_print(ndo, "\t\t[", ILV_DATA(ilv), rlen);
+ ND_PRINT(" ]\n");
}
return 0;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
static int
print_metatlv(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
u_int dlen;
@@ -1387,10 +1353,9 @@ print_metatlv(netdissect_options *ndo,
*/
dlen = len - TLV_HDRL;
rlen = dlen;
- ND_PRINT((ndo, "\n%s METADATA length %d \n", ib, rlen));
+ ND_PRINT("\n%s METADATA length %u\n", ib, rlen);
while (rlen != 0) {
- ND_TCHECK(*ilv);
- invilv = ilv_valid(ilv, rlen);
+ invilv = ilv_valid(ndo, ilv, rlen);
if (invilv) {
break;
}
@@ -1405,16 +1370,12 @@ print_metatlv(netdissect_options *ndo,
}
return 0;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
static int
print_reddata(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
u_int dlen;
@@ -1423,12 +1384,12 @@ print_reddata(netdissect_options *ndo,
dlen = len - TLV_HDRL;
rlen = dlen;
- ND_PRINT((ndo, "\n%s Redirect Data length %d \n", ib, rlen));
+ ND_PRINT("\n%s Redirect Data length %u\n", ib, rlen);
if (ndo->ndo_vflag >= 3) {
- ND_PRINT((ndo, "\t\t["));
- hex_print_with_offset(ndo, "\n\t\t", pptr, rlen, 0);
- ND_PRINT((ndo, "\n\t\t]"));
+ ND_PRINT("\t\t[");
+ hex_print(ndo, "\n\t\t", pptr, rlen);
+ ND_PRINT("\n\t\t]");
}
return 0;
@@ -1436,7 +1397,7 @@ print_reddata(netdissect_options *ndo,
static int
redirect_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk _U_, int indent)
{
const struct forces_tlv *tlv = (const struct forces_tlv *)pptr;
@@ -1450,18 +1411,21 @@ redirect_print(netdissect_options *ndo,
*/
dlen = len - TLV_HDRL;
if (dlen <= RD_MIN) {
- ND_PRINT((ndo, "\n\t\ttruncated Redirect TLV: %d bytes missing! ",
- RD_MIN - dlen));
+ ND_PRINT("\n\t\ttruncated Redirect TLV: %u bytes missing! ",
+ RD_MIN - dlen);
return -1;
}
rlen = dlen;
indent += 1;
while (rlen != 0) {
- ND_TCHECK(*tlv);
- invtlv = tlv_valid(tlv, rlen);
+ uint16_t type, tlvl;
+
+ type = GET_BE_U_2(tlv->type);
+ tlvl = GET_BE_U_2(tlv->length);
+ invtlv = tlv_valid(tlvl, rlen);
if (invtlv) {
- ND_PRINT((ndo, "Bad Redirect data\n"));
+ ND_PRINT("Bad Redirect data\n");
break;
}
@@ -1470,34 +1434,31 @@ redirect_print(netdissect_options *ndo,
* length is large enough but not too large (it doesn't
* go past the end of the containing TLV).
*/
- if (EXTRACT_16BITS(&tlv->type) == F_TLV_METD) {
+ if (type == F_TLV_METD) {
print_metatlv(ndo, (const u_char *) TLV_DATA(tlv),
- EXTRACT_16BITS(&tlv->length), 0, indent);
- } else if ((EXTRACT_16BITS(&tlv->type) == F_TLV_REDD)) {
+ tlvl, 0,
+ indent);
+ } else if (type == F_TLV_REDD) {
print_reddata(ndo, (const u_char *) TLV_DATA(tlv),
- EXTRACT_16BITS(&tlv->length), 0, indent);
+ tlvl, 0,
+ indent);
} else {
- ND_PRINT((ndo, "Unknown REDIRECT TLV 0x%x len %d\n",
- EXTRACT_16BITS(&tlv->type),
- EXTRACT_16BITS(&tlv->length)));
+ ND_PRINT("Unknown REDIRECT TLV 0x%x len %u\n",
+ type,
+ tlvl);
}
tlv = GO_NXT_TLV(tlv, rlen);
}
if (rlen) {
- ND_PRINT((ndo,
- "\n\t\tMessy Redirect TLV header, type (0x%x)\n\t\texcess of %d Bytes ",
- EXTRACT_16BITS(&tlv->type),
- rlen - EXTRACT_16BITS(&tlv->length)));
+ ND_PRINT("\n\t\tMessy Redirect TLV header, type (0x%x)\n\t\texcess of %u Bytes ",
+ GET_BE_U_2(tlv->type),
+ rlen - GET_BE_U_2(tlv->length));
return -1;
}
return 0;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
#define OP_OFF 8
@@ -1505,7 +1466,7 @@ trunc:
static int
lfbselect_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len,
+ const u_char * pptr, u_int len,
uint16_t op_msk, int indent)
{
const struct forces_lfbsh *lfbs;
@@ -1521,8 +1482,8 @@ lfbselect_print(netdissect_options *ndo,
*/
dlen = len - TLV_HDRL;
if (dlen <= OP_MIN) { /* id, instance, oper tlv header .. */
- ND_PRINT((ndo, "\n\t\ttruncated lfb selector: %d bytes missing! ",
- OP_MIN - dlen));
+ ND_PRINT("\n\t\ttruncated lfb selector: %u bytes missing! ",
+ OP_MIN - dlen);
return -1;
}
@@ -1533,20 +1494,24 @@ lfbselect_print(netdissect_options *ndo,
rlen = dlen - OP_OFF;
lfbs = (const struct forces_lfbsh *)pptr;
- ND_TCHECK(*lfbs);
+ ND_TCHECK_SIZE(lfbs);
if (ndo->ndo_vflag >= 3) {
- ND_PRINT((ndo, "\n%s%s(Classid %x) instance %x\n",
- ib, tok2str(ForCES_LFBs, NULL, EXTRACT_32BITS(&lfbs->class)),
- EXTRACT_32BITS(&lfbs->class),
- EXTRACT_32BITS(&lfbs->instance)));
+ ND_PRINT("\n%s%s(Classid %x) instance %x\n",
+ ib,
+ tok2str(ForCES_LFBs, NULL, GET_BE_U_4(lfbs->class)),
+ GET_BE_U_4(lfbs->class),
+ GET_BE_U_4(lfbs->instance));
}
otlv = (const struct forces_tlv *)(lfbs + 1);
indent += 1;
while (rlen != 0) {
- ND_TCHECK(*otlv);
- invtlv = tlv_valid(otlv, rlen);
+ uint16_t type, tlvl;
+
+ type = GET_BE_U_2(otlv->type);
+ tlvl = GET_BE_U_2(otlv->length);
+ invtlv = tlv_valid(tlvl, rlen);
if (invtlv)
break;
@@ -1555,43 +1520,42 @@ lfbselect_print(netdissect_options *ndo,
* length is large enough but not too large (it doesn't
* go past the end of the containing TLV).
*/
- if (op_valid(EXTRACT_16BITS(&otlv->type), op_msk)) {
+ if (op_valid(type, op_msk)) {
otlv_print(ndo, otlv, 0, indent);
} else {
if (ndo->ndo_vflag < 3)
- ND_PRINT((ndo, "\n"));
- ND_PRINT((ndo,
- "\t\tINValid oper-TLV type 0x%x length %d for this ForCES message\n",
- EXTRACT_16BITS(&otlv->type), EXTRACT_16BITS(&otlv->length)));
+ ND_PRINT("\n");
+ ND_PRINT("\t\tINValid oper-TLV type 0x%x length %u for this ForCES message\n",
+ type, tlvl);
invoptlv_print(ndo, (const u_char *)otlv, rlen, 0, indent);
}
otlv = GO_NXT_TLV(otlv, rlen);
}
if (rlen) {
- ND_PRINT((ndo,
- "\n\t\tMessy oper TLV header, type (0x%x)\n\t\texcess of %d Bytes ",
- EXTRACT_16BITS(&otlv->type), rlen - EXTRACT_16BITS(&otlv->length)));
+ ND_PRINT("\n\t\tMessy oper TLV header, type (0x%x)\n\t\texcess of %u Bytes ",
+ GET_BE_U_2(otlv->type),
+ rlen - GET_BE_U_2(otlv->length));
return -1;
}
return 0;
trunc:
- ND_PRINT((ndo, "%s", tstr));
+ nd_print_trunc(ndo);
return -1;
}
static int
forces_type_print(netdissect_options *ndo,
- register const u_char * pptr, const struct forcesh *fhdr _U_,
- register u_int mlen, const struct tom_h *tops)
+ const u_char * pptr, const struct forcesh *fhdr _U_,
+ u_int mlen, const struct tom_h *tops)
{
const struct forces_tlv *tltlv;
u_int rlen;
u_int invtlv;
int rc = 0;
- int ttlv = 0;
+ u_int ttlv = 0;
/*
* forces_print() has already checked that mlen >= ForCES_HDRL
@@ -1601,14 +1565,14 @@ forces_type_print(netdissect_options *ndo,
if (rlen > TLV_HLN) {
if (tops->flags & ZERO_TTLV) {
- ND_PRINT((ndo, "<0x%x>Illegal Top level TLV!\n", tops->flags));
+ ND_PRINT("<0x%x>Illegal Top level TLV!\n", tops->flags);
return -1;
}
} else {
if (tops->flags & ZERO_MORE_TTLV)
return 0;
if (tops->flags & ONE_MORE_TTLV) {
- ND_PRINT((ndo, "\tTop level TLV Data missing!\n"));
+ ND_PRINT("\tTop level TLV Data missing!\n");
return -1;
}
}
@@ -1623,8 +1587,11 @@ forces_type_print(netdissect_options *ndo,
/*XXX: 15 top level tlvs will probably be fine
You are nuts if you send more ;-> */
while (rlen != 0) {
- ND_TCHECK(*tltlv);
- invtlv = tlv_valid(tltlv, rlen);
+ uint16_t type, tlvl;
+
+ type = GET_BE_U_2(tltlv->type);
+ tlvl = GET_BE_U_2(tltlv->length);
+ invtlv = tlv_valid(tlvl, rlen);
if (invtlv)
break;
@@ -1633,20 +1600,21 @@ forces_type_print(netdissect_options *ndo,
* length is large enough but not too large (it doesn't
* go past the end of the packet).
*/
- if (!ttlv_valid(EXTRACT_16BITS(&tltlv->type))) {
- ND_PRINT((ndo, "\n\tInvalid ForCES Top TLV type=0x%x",
- EXTRACT_16BITS(&tltlv->type)));
+ if (!ttlv_valid(type)) {
+ ND_PRINT("\n\tInvalid ForCES Top TLV type=0x%x",
+ type);
return -1;
}
if (ndo->ndo_vflag >= 3)
- ND_PRINT((ndo, "\t%s, length %d (data length %d Bytes)",
- tok2str(ForCES_TLV, NULL, EXTRACT_16BITS(&tltlv->type)),
- EXTRACT_16BITS(&tltlv->length),
- EXTRACT_16BITS(&tltlv->length) - TLV_HDRL));
+ ND_PRINT("\t%s, length %u (data length %u Bytes)",
+ tok2str(ForCES_TLV, NULL, type),
+ tlvl,
+ tlvl - TLV_HDRL);
rc = tops->print(ndo, (const u_char *) TLV_DATA(tltlv),
- EXTRACT_16BITS(&tltlv->length), tops->op_msk, 9);
+ tlvl,
+ tops->op_msk, 9);
if (rc < 0) {
return -1;
}
@@ -1660,66 +1628,61 @@ forces_type_print(netdissect_options *ndo,
* short, and didn't have *enough* TLVs in it?
*/
if (rlen) {
- ND_PRINT((ndo, "\tMess TopTLV header: min %u, total %d advertised %d ",
- TLV_HDRL, rlen, EXTRACT_16BITS(&tltlv->length)));
+ ND_PRINT("\tMess TopTLV header: min %u, total %u advertised %u ",
+ TLV_HDRL, rlen, GET_BE_U_2(tltlv->length));
return -1;
}
return 0;
-
-trunc:
- ND_PRINT((ndo, "%s", tstr));
- return -1;
}
void
forces_print(netdissect_options *ndo,
- register const u_char * pptr, register u_int len)
+ const u_char * pptr, u_int len)
{
const struct forcesh *fhdr;
u_int mlen;
uint32_t flg_raw;
+ uint8_t tom;
const struct tom_h *tops;
int rc = 0;
+ ndo->ndo_protocol = "forces";
fhdr = (const struct forcesh *)pptr;
- ND_TCHECK(*fhdr);
- if (!tom_valid(fhdr->fm_tom)) {
- ND_PRINT((ndo, "Invalid ForCES message type %d\n", fhdr->fm_tom));
+ ND_TCHECK_SIZE(fhdr);
+ tom = GET_U_1(fhdr->fm_tom);
+ if (!tom_valid(tom)) {
+ ND_PRINT("Invalid ForCES message type %u\n", tom);
goto error;
}
mlen = ForCES_BLN(fhdr);
- tops = get_forces_tom(fhdr->fm_tom);
+ tops = get_forces_tom(tom);
if (tops->v == TOM_RSVD) {
- ND_PRINT((ndo, "\n\tUnknown ForCES message type=0x%x", fhdr->fm_tom));
+ ND_PRINT("\n\tUnknown ForCES message type=0x%x", tom);
goto error;
}
- ND_PRINT((ndo, "\n\tForCES %s ", tops->s));
+ ND_PRINT("\n\tForCES %s ", tops->s);
if (!ForCES_HLN_VALID(mlen, len)) {
- ND_PRINT((ndo,
- "Illegal ForCES pkt len - min %u, total recvd %d, advertised %d ",
- ForCES_HDRL, len, ForCES_BLN(fhdr)));
+ ND_PRINT("Illegal ForCES pkt len - min %u, total recvd %u, advertised %u ",
+ ForCES_HDRL, len, ForCES_BLN(fhdr));
goto error;
}
- ND_TCHECK2(*(pptr + 20), 4);
- flg_raw = EXTRACT_32BITS(pptr + 20);
+ flg_raw = GET_BE_U_4(pptr + 20);
if (ndo->ndo_vflag >= 1) {
- ND_PRINT((ndo, "\n\tForCES Version %d len %uB flags 0x%08x ",
- ForCES_V(fhdr), mlen, flg_raw));
- ND_PRINT((ndo,
- "\n\tSrcID 0x%x(%s) DstID 0x%x(%s) Correlator 0x%" PRIx64,
+ ND_PRINT("\n\tForCES Version %u len %uB flags 0x%08x ",
+ ForCES_V(fhdr), mlen, flg_raw);
+ ND_PRINT("\n\tSrcID 0x%x(%s) DstID 0x%x(%s) Correlator 0x%" PRIx64,
ForCES_SID(fhdr), ForCES_node(ForCES_SID(fhdr)),
ForCES_DID(fhdr), ForCES_node(ForCES_DID(fhdr)),
- EXTRACT_64BITS(fhdr->fm_cor)));
+ GET_BE_U_8(fhdr->fm_cor));
}
if (ndo->ndo_vflag >= 2) {
- ND_PRINT((ndo,
- "\n\tForCES flags:\n\t %s(0x%x), prio=%d, %s(0x%x),\n\t %s(0x%x), %s(0x%x)\n",
+ ND_PRINT("\n\tForCES flags:\n\t %s(0x%x), prio=%u, %s(0x%x),\n\t %s(0x%x), %s(0x%x)\n",
tok2str(ForCES_ACKs, "ACKUnknown", ForCES_ACK(fhdr)),
ForCES_ACK(fhdr),
ForCES_PRI(fhdr),
@@ -1728,33 +1691,25 @@ forces_print(netdissect_options *ndo,
tok2str(ForCES_ATs, "ATUnknown", ForCES_AT(fhdr)),
ForCES_AT(fhdr),
tok2str(ForCES_TPs, "TPUnknown", ForCES_TP(fhdr)),
- ForCES_TP(fhdr)));
- ND_PRINT((ndo,
- "\t Extra flags: rsv(b5-7) 0x%x rsv(b13-31) 0x%x\n",
- ForCES_RS1(fhdr), ForCES_RS2(fhdr)));
+ ForCES_TP(fhdr));
+ ND_PRINT("\t Extra flags: rsv(b5-7) 0x%x rsv(b13-31) 0x%x\n",
+ ForCES_RS1(fhdr), ForCES_RS2(fhdr));
}
rc = forces_type_print(ndo, pptr, fhdr, mlen, tops);
if (rc < 0) {
error:
- hex_print_with_offset(ndo, "\n\t[", pptr, len, 0);
- ND_PRINT((ndo, "\n\t]"));
+ hex_print(ndo, "\n\t[", pptr, len);
+ ND_PRINT("\n\t]");
return;
}
if (ndo->ndo_vflag >= 4) {
- ND_PRINT((ndo, "\n\t Raw ForCES message\n\t ["));
- hex_print_with_offset(ndo, "\n\t ", pptr, len, 0);
- ND_PRINT((ndo, "\n\t ]"));
+ ND_PRINT("\n\t Raw ForCES message\n\t [");
+ hex_print(ndo, "\n\t ", pptr, len);
+ ND_PRINT("\n\t ]");
}
- ND_PRINT((ndo, "\n"));
return;
trunc:
- ND_PRINT((ndo, "%s", tstr));
+ nd_print_trunc(ndo);
}
-/*
- * Local Variables:
- * c-style: whitesmith
- * c-basic-offset: 8
- * End:
- */