diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2013-05-07 13:41:52 +0000 |
|---|---|---|
| committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2013-05-07 13:41:52 +0000 |
| commit | b53ac56b03c9b348065d6242f78a73283f3d3bca (patch) | |
| tree | 05955a1b9d12d51e50d36740c8a2311dba9abb2a /epan/dissectors/packet-nat-pmp.c | |
| parent | 7a588676268b322230ff22a781b1ebcb4cd322fa (diff) | |
From Christian Koenning via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8631 RFC 6887 Port Control Protocol Response 96 bit reserved not 64 bit
As per RFC 6887 the response header has 96bit reserved bit, not 64.
Without the attached patch, the PCP Responses are parsed incorrectly
svn path=/trunk/; revision=49202
Diffstat (limited to 'epan/dissectors/packet-nat-pmp.c')
| -rw-r--r-- | epan/dissectors/packet-nat-pmp.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/epan/dissectors/packet-nat-pmp.c b/epan/dissectors/packet-nat-pmp.c index d45ef30eee..25aa23394f 100644 --- a/epan/dissectors/packet-nat-pmp.c +++ b/epan/dissectors/packet-nat-pmp.c @@ -88,7 +88,7 @@ static int hf_pcp_opcode = -1; static int hf_pcp_result_code = -1; static int hf_reserved1 = -1; static int hf_reserved2 = -1; -static int hf_reserved8 = -1; +static int hf_reserved12 = -1; static int hf_req_lifetime = -1; static int hf_rsp_lifetime = -1; static int hf_client_ip = -1; @@ -330,8 +330,8 @@ dissect_portcontrol_pdu(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, gui proto_tree_add_item (pcp_tree, hf_epoch_time, tvb, offset, 4, ENC_BIG_ENDIAN); offset+=4; - proto_tree_add_item (pcp_tree, hf_reserved8, tvb, offset, 8, ENC_NA); - offset+=8; + proto_tree_add_item (pcp_tree, hf_reserved12, tvb, offset, 12, ENC_NA); + offset+=12; } start_opcode_offset = offset; @@ -554,7 +554,7 @@ void proto_register_nat_pmp (void) { &hf_reserved2, { "Reserved", "portcontrol.reserved", FT_UINT16, BASE_DEC, NULL, 0x0, NULL, HFILL } }, - { &hf_reserved8, + { &hf_reserved12, { "Reserved", "portcontrol.rsp_reserved", FT_BYTES, BASE_NONE, NULL, 0x0, NULL, HFILL } }, { &hf_req_lifetime, |