diff options
author | basakkal <basakkalfa@gmail.com> | 2018-10-02 23:49:18 -0700 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2018-10-03 12:49:10 +0000 |
commit | 53ee71c1be98329fd2535eaa2bd354eaf1c570fe (patch) | |
tree | fcdce48933d368d2cd9d974d42c3791598a26aca /plugins | |
parent | f9be57ed361a4651112daf0f87f599c74b486bf9 (diff) |
PROFINET: Decoding of DCP SetResBlock
At SetResBlock, the field "Response" should be "Option"
according to specification. It is corrected.
Moreover, hf and display filter are changed.
Change-Id: Ieae38616b130efc118673814f9fcee4241e75f05
Reviewed-on: https://code.wireshark.org/review/29988
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'plugins')
-rw-r--r-- | plugins/epan/profinet/packet-pn-dcp.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/plugins/epan/profinet/packet-pn-dcp.c b/plugins/epan/profinet/packet-pn-dcp.c index db0f47b043..f440cc1bc4 100644 --- a/plugins/epan/profinet/packet-pn-dcp.c +++ b/plugins/epan/profinet/packet-pn-dcp.c @@ -87,7 +87,7 @@ static int hf_pn_dcp_suboption_dhcp = -1; static int hf_pn_dcp_suboption_dhcp_device_id = -1; static int hf_pn_dcp_suboption_control = -1; -static int hf_pn_dcp_suboption_control_response = -1; +static int hf_pn_dcp_suboption_control_option = -1; static int hf_pn_dcp_suboption_control_signal_value = -1; static int hf_pn_dcp_suboption_deviceinitiative = -1; @@ -824,7 +824,7 @@ dissect_PNDCP_Suboption_Control(tvbuff_t *tvb, int offset, packet_info *pinfo, break; case PNDCP_SUBOPTION_CONTROL_RESPONSE: proto_item_append_text(block_item, "Control/Response"); - offset = dissect_PNDCP_Option(tvb, offset, pinfo, tree, block_item, hf_pn_dcp_suboption_control_response, + offset = dissect_PNDCP_Option(tvb, offset, pinfo, tree, block_item, hf_pn_dcp_suboption_control_option, FALSE /* append_col */); block_error = tvb_get_guint8 (tvb, offset); if (tree) { @@ -1315,8 +1315,8 @@ proto_register_pn_dcp (void) FT_UINT8, BASE_DEC, VALS(pn_dcp_suboption_control), 0x0, NULL, HFILL }}, - { &hf_pn_dcp_suboption_control_response, - { "Response", "pn_dcp.suboption_control_response", + { &hf_pn_dcp_suboption_control_option, + { "Option", "pn_dcp.suboption_control_option", FT_UINT8, BASE_DEC, VALS(pn_dcp_option), 0x0, NULL, HFILL }}, |