diff options
author | morriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-01-16 20:51:21 +0000 |
---|---|---|
committer | morriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-01-16 20:51:21 +0000 |
commit | e7ea07c0bddf8f45ac652a040ce85eaf769a16aa (patch) | |
tree | 420d5401a47ed145c8aba98c7891e5f098f86e19 /epan/dissectors/packet-pagp.c | |
parent | 45393f35e39fa3e077c1d784cb47de0b4074ee27 (diff) |
There's no need to pass the result of tvb_get_ptr() as the 'value' in
proto_tree_add_*(): just use proto_tree_add_item().
Replace some tvb_get_ptr()s with tvb_get_ephemeral_string() or
tvb_get_const_stringz().
Use tvb_memeql() & tvb_memcmp().
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@35558 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-pagp.c')
-rw-r--r-- | epan/dissectors/packet-pagp.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/epan/dissectors/packet-pagp.c b/epan/dissectors/packet-pagp.c index 07e5307e6d..1f6cd7c47b 100644 --- a/epan/dissectors/packet-pagp.c +++ b/epan/dissectors/packet-pagp.c @@ -175,8 +175,6 @@ dissect_pagp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) struct _address device_id; - const guint8 *p_sys; - guchar *ch; proto_tree *pagp_tree = NULL; @@ -390,9 +388,8 @@ dissect_pagp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) tvb, offset+4, len-4, ch); break; case PAGP_TLV_AGPORT_MAC: - p_sys = tvb_get_ptr(tvb, offset+4, 6); - proto_tree_add_ether(tlv_tree, hf_pagp_tlv_agport_mac, - tvb, offset+4, 6, p_sys); + proto_tree_add_item(tlv_tree, hf_pagp_tlv_agport_mac, + tvb, offset+4, 6, ENC_NA); break; case PAGP_TLV_RESERVED: break; |