diff options
author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2018-05-25 07:57:18 +0200 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2018-05-26 05:53:19 +0000 |
commit | 5f35fddc2349e876de21118a112627cc551bfe5f (patch) | |
tree | 09a5ea8751152476e623b6512144e005c7d8af70 /epan/dissectors/packet-gquic.c | |
parent | 169887f170580a864da52a77400c7476b43116dd (diff) |
gQUIC: it is possible to have multiple COPT Tag
And it is a STRING display
Change-Id: I0f6521e936219bc98cd8eae2862bf6c582123f57
Reviewed-on: https://code.wireshark.org/review/27797
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-gquic.c')
-rw-r--r-- | epan/dissectors/packet-gquic.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/packet-gquic.c b/epan/dissectors/packet-gquic.c index e65463df1a..33a6559b53 100644 --- a/epan/dissectors/packet-gquic.c +++ b/epan/dissectors/packet-gquic.c @@ -1504,8 +1504,8 @@ dissect_gquic_tag(tvbuff_t *tvb, packet_info *pinfo, proto_tree *gquic_tree, gui tag_offset += 4; break; case TAG_COPT: - if(tag_len){ - proto_tree_add_item(tag_tree, hf_gquic_tag_copt, tvb, tag_offset_start + tag_offset, 4, ENC_LITTLE_ENDIAN); + while(offset_end - tag_offset >= 4){ + proto_tree_add_item(tag_tree, hf_gquic_tag_copt, tvb, tag_offset_start + tag_offset, 4, ENC_ASCII|ENC_NA); tag_offset += 4; } break; @@ -2735,7 +2735,7 @@ proto_register_gquic(void) }, { &hf_gquic_tag_copt, { "Connection options", "gquic.tag.copt", - FT_UINT32, BASE_DEC_HEX, NULL, 0x0, + FT_STRING, BASE_NONE, NULL, 0x0, NULL, HFILL } }, { &hf_gquic_tag_ccrt, |