diff options
author | Evan Huus <eapache@gmail.com> | 2014-06-19 23:05:53 -0700 |
---|---|---|
committer | Evan Huus <eapache@gmail.com> | 2014-06-20 06:07:41 +0000 |
commit | 80c070fb517a8c153278d936f0143e283adc01fb (patch) | |
tree | 334a59e2d02f6b47345b8c4ce64fc6eef4e8ee8a /epan | |
parent | 980f5f6711ae5642adc509d9a8ee81d188f3a431 (diff) |
Don't hide tcp option kind/len fields
Hidden fields are deprecated, and we were hiding them inconsistently anyways.
Bug:10211
Change-Id: Iaf1576ae7bc04c0c0bd896c096b117f1b8af2e9e
Reviewed-on: https://code.wireshark.org/review/2474
Reviewed-by: Evan Huus <eapache@gmail.com>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-tcp.c | 54 |
1 files changed, 18 insertions, 36 deletions
diff --git a/epan/dissectors/packet-tcp.c b/epan/dissectors/packet-tcp.c index 3e99f9dea8..bd4c603a7a 100644 --- a/epan/dissectors/packet-tcp.c +++ b/epan/dissectors/packet-tcp.c @@ -2474,12 +2474,10 @@ dissect_tcpopt_sack(const ip_tcp_opt *optp, tvbuff_t *tvb, } } - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, offset + 1, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); tf = proto_tree_add_text(opt_tree, tvb, offset, optlen, "%s:", optp->name); offset += 2; /* skip past type and length */ @@ -2544,12 +2542,10 @@ dissect_tcpopt_echo(const ip_tcp_opt *optp, tvbuff_t *tvb, proto_item *hidden_item; guint32 echo; - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, offset + 1, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); echo = tvb_get_ntohl(tvb, offset + 2); hidden_item = proto_tree_add_boolean(opt_tree, hf_tcp_option_echo, tvb, offset, @@ -2871,12 +2867,10 @@ dissect_tcpopt_cc(const ip_tcp_opt *optp, tvbuff_t *tvb, proto_item *hidden_item; guint32 cc; - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, offset + 1, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); cc = tvb_get_ntohl(tvb, offset + 2); hidden_item = proto_tree_add_boolean(opt_tree, hf_tcp_option_cc, tvb, offset, @@ -2896,12 +2890,10 @@ dissect_tcpopt_qs(const ip_tcp_opt *optp, tvbuff_t *tvb, guint8 rate = tvb_get_guint8(tvb, offset + 2) & 0x0f; - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, offset + 1, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); hidden_item = proto_tree_add_boolean(opt_tree, hf_tcp_option_qs, tvb, offset, optlen, TRUE); @@ -2927,12 +2919,10 @@ dissect_tcpopt_scps(const ip_tcp_opt *optp _U_, tvbuff_t *tvb, guint8 capvector; guint8 connid; - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, offset + 1, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); tcpd = get_tcp_conversation_data(NULL,pinfo); @@ -3092,12 +3082,10 @@ dissect_tcpopt_user_to(const ip_tcp_opt *optp, tvbuff_t *tvb, gboolean g; guint16 to; - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, offset + 1, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); g = tvb_get_ntohs(tvb, offset + 2) & 0x8000; to = tvb_get_ntohs(tvb, offset + 2) & 0x7FFF; @@ -3155,12 +3143,10 @@ dissect_tcpopt_snack(const ip_tcp_opt *optp, tvbuff_t *tvb, char *modifier = null_modifier; proto_item *hidden_item; - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); - hidden_item = proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(opt_tree, hf_tcp_option_len, tvb, offset + 1, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(hidden_item); tcpd = get_tcp_conversation_data(NULL,pinfo); @@ -3333,12 +3319,10 @@ dissect_tcpopt_rvbd_probe(const ip_tcp_opt *optp _U_, tvbuff_t *tvb, int offset, /* optlen, type, ver are common for all probes */ field_tree = proto_item_add_subtree(pitem, ett_tcp_opt_rvbd_probe); - pitem = proto_tree_add_item(field_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(field_tree, hf_tcp_option_len, tvb, offset + PROBE_OPTLEN_OFFSET, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(pitem); - pitem = proto_tree_add_item(field_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(field_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(pitem); proto_tree_add_item(field_tree, hf_tcp_option_rvbd_probe_optlen, tvb, offset + PROBE_OPTLEN_OFFSET, 1, ENC_BIG_ENDIAN); @@ -3545,12 +3529,10 @@ dissect_tcpopt_rvbd_trpy(const ip_tcp_opt *optp _U_, tvbuff_t *tvb, "%s", ""); field_tree = proto_item_add_subtree(pitem, ett_tcp_opt_rvbd_trpy); - pitem = proto_tree_add_item(field_tree, hf_tcp_option_len, tvb, + proto_tree_add_item(field_tree, hf_tcp_option_len, tvb, offset + PROBE_OPTLEN_OFFSET, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(pitem); - pitem = proto_tree_add_item(field_tree, hf_tcp_option_kind, tvb, + proto_tree_add_item(field_tree, hf_tcp_option_kind, tvb, offset, 1, ENC_BIG_ENDIAN); - PROTO_ITEM_SET_HIDDEN(pitem); proto_tree_add_item(field_tree, hf_tcp_option_rvbd_probe_optlen, tvb, offset + PROBE_OPTLEN_OFFSET, 1, ENC_BIG_ENDIAN); |