From b572b65e518937f43b630991c07369d8e0c79f53 Mon Sep 17 00:00:00 2001 From: Gerald Combs Date: Wed, 3 Apr 2019 09:41:36 -0700 Subject: DOF: Fix alias binding dissection. When adding generated IID and OID bytes to the tree, use a zero offset and length like we do elsewhere in the dissector. Bug: 15617 Change-Id: Id900f2aeeef7926706b417622d452ffa72949e8a Reviewed-on: https://code.wireshark.org/review/32698 Petri-Dish: Gerald Combs Tested-by: Petri Dish Buildbot Reviewed-by: Gerald Combs --- epan/dissectors/packet-dof.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'epan/dissectors/packet-dof.c') diff --git a/epan/dissectors/packet-dof.c b/epan/dissectors/packet-dof.c index ba803168e7..c3286a538c 100644 --- a/epan/dissectors/packet-dof.c +++ b/epan/dissectors/packet-dof.c @@ -1379,16 +1379,17 @@ static int oap_1_tree_add_alias(dof_api_data *api_data, oap_1_packet_data *oap_p key.sender = packet->sender_id; key.alias = alias; binding = oap_1_resolve_alias(&key); + if (binding) { options_tree = proto_item_add_subtree(ti, ett_oap_1_alias); /* Decode the Interface */ - ti = proto_tree_add_bytes_format_value(tree, hf_oap_1_interfaceid, tvb, offset, alias_length, binding->iid, "%s", dof_iid_create_standard_string(binding->iid_length, binding->iid)); + ti = proto_tree_add_bytes_format_value(tree, hf_oap_1_interfaceid, tvb, 0, 0, binding->iid, "%s", dof_iid_create_standard_string(binding->iid_length, binding->iid)); proto_item_set_generated(ti); /* Decode the Object ID */ - ti = proto_tree_add_bytes_format_value(tree, hf_oap_1_objectid, tvb, offset, alias_length, binding->oid, "%s", dof_oid_create_standard_string(binding->oid_length, binding->oid)); + ti = proto_tree_add_bytes_format_value(tree, hf_oap_1_objectid, tvb, 0, 0, binding->oid, "%s", dof_oid_create_standard_string(binding->oid_length, binding->oid)); proto_item_set_generated(ti); proto_tree_add_uint_format(options_tree, hf_oap_1_alias_frame, -- cgit v1.2.3