diff options
author | Michael Mann <mmann78@netscape.net> | 2015-03-01 23:03:27 -0500 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2015-03-03 09:02:25 +0000 |
commit | b7ef891af868b6026391d3473c0b43e7229e5b31 (patch) | |
tree | 073d847595f777447f804937957324fcbae9c653 /epan/dissectors/packet-vines.c | |
parent | 599dd4ffcc31bb03e44fd4ed72db4d6b8fbc38f0 (diff) |
Replace tvb_get_ptr calls with a better API choice.
Just reduces the overall tvb_get_ptr usage count in the dissector directory.
Change-Id: I455dc4cc9b082ecccdd254a2e5121f3353b5a812
Reviewed-on: https://code.wireshark.org/review/7491
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-vines.c')
-rw-r--r-- | epan/dissectors/packet-vines.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/epan/dissectors/packet-vines.c b/epan/dissectors/packet-vines.c index 4683d0beed..e9406f9e5c 100644 --- a/epan/dissectors/packet-vines.c +++ b/epan/dissectors/packet-vines.c @@ -535,7 +535,6 @@ dissect_vines_ip(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) guint8 vip_proto; proto_tree *vip_tree, *tctl_tree; proto_item *ti; - const guint8 *dst_addr; gboolean is_broadcast = FALSE; tvbuff_t *next_tvb; @@ -555,12 +554,11 @@ dissect_vines_ip(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) TVB_SET_ADDRESS(&pinfo->net_src, AT_VINES, tvb, offset+12, VINES_ADDR_LEN); COPY_ADDRESS_SHALLOW(&pinfo->src, &pinfo->net_src); - dst_addr = tvb_get_ptr(tvb, offset+6, VINES_ADDR_LEN); TVB_SET_ADDRESS(&pinfo->net_dst, AT_VINES, tvb, offset+6, VINES_ADDR_LEN); COPY_ADDRESS_SHALLOW(&pinfo->dst, &pinfo->net_dst); /* helpers to transport control */ - if (memcmp(dst_addr, bcast_addr, VINES_ADDR_LEN) == 0) + if (tvb_memeql(tvb, offset+6, bcast_addr, VINES_ADDR_LEN) == 0) is_broadcast = TRUE; /* |