diff options
author | Erwin Rol <erwin@erwinrol.com> | 2020-01-11 17:14:44 +0100 |
---|---|---|
committer | Pascal Quantin <pascal@wireshark.org> | 2020-01-25 14:48:06 +0000 |
commit | 28dc014dc290849ad59ba848f0ce9a355bfb899c (patch) | |
tree | 1c696c7e45ba848d6250509fb602714abebc8bf8 | |
parent | 80504ef3ca69b57cd4dbc4d9902634778624f833 (diff) | |
download | wireshark-28dc014dc290849ad59ba848f0ce9a355bfb899c.tar.gz wireshark-28dc014dc290849ad59ba848f0ce9a355bfb899c.tar.bz2 wireshark-28dc014dc290849ad59ba848f0ce9a355bfb899c.zip |
rdm: Remove incorrect dissection of ack overflow data
Ack overflow data was incorrectly dissected causing checksum errors. For now
just display raw data.
Change-Id: Icdd858bdbeeb4dd40e48c45fc46e5e188d53be69
Signed-off-by: Erwin Rol <erwin@erwinrol.com>
Reviewed-on: https://code.wireshark.org/review/35915
Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl>
Tested-by: Petri Dish Buildbot
Reviewed-by: Pascal Quantin <pascal@wireshark.org>
(cherry picked from commit 4f462bb6e3470ccb09b8da1efd4f133169aef03d)
Reviewed-on: https://code.wireshark.org/review/35945
Petri-Dish: Pascal Quantin <pascal@wireshark.org>
-rw-r--r-- | epan/dissectors/packet-rdm.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/epan/dissectors/packet-rdm.c b/epan/dissectors/packet-rdm.c index 4a1280c901..01f2bae907 100644 --- a/epan/dissectors/packet-rdm.c +++ b/epan/dissectors/packet-rdm.c @@ -561,7 +561,7 @@ static int hf_rdm_checksum_status = -1; static int hf_rdm_trailer = -1; static int hf_rdm_pd_ack_timer_estimated_response_time = -1; -static int hf_rdm_pd_ack_overflow_uid = -1; +static int hf_rdm_pd_ack_overflow_raw_data = -1; static int hf_rdm_pd_nack_reason_code = -1; static int hf_rdm_pd_device_label = -1; @@ -2008,16 +2008,11 @@ static guint dissect_rdm_pd_ack_overflow(tvbuff_t *tvb, guint offset, proto_tree *tree, guint8 cc, guint16 param_id _U_, guint8 pdl) { if (pdl > 0) { - int i; - switch(cc) { case RDM_CC_GET_COMMAND_RESPONSE: case RDM_CC_SET_COMMAND_RESPONSE: - for (i = 0; i < (pdl / 6); i++) { - proto_tree_add_item(tree, hf_rdm_pd_ack_overflow_uid, tvb, - offset, 6, ENC_NA); - offset += 6; - } + proto_tree_add_item(tree, hf_rdm_pd_ack_overflow_raw_data, tvb, offset, pdl, ENC_NA); + offset += pdl; break; } } @@ -2302,8 +2297,8 @@ proto_register_rdm(void) FT_BYTES, BASE_NONE, NULL, 0x0, NULL, HFILL }}, - { &hf_rdm_pd_ack_overflow_uid, - { "UID", "rdm.pd.ack_overflow.uid", + { &hf_rdm_pd_ack_overflow_raw_data, + { "Raw Data", "rdm.pd.ack_overflow.raw_data", FT_BYTES, BASE_NONE, NULL, 0x0, NULL, HFILL }}, |