diff options
| author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-05-12 17:26:54 +0000 |
|---|---|---|
| committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2011-05-12 17:26:54 +0000 |
| commit | 29ec75d29e48a5e3511319c5b40a68c80b755249 (patch) | |
| tree | 1932fa0325d0a14f02b4bb375a1571b7d8fd4f09 /epan/dissectors/packet-sna.c | |
| parent | dbc48efbe8bce559024e1818c7f9428016da649e (diff) | |
Add snai, mpf and efi to bf_tree, not tree. Fixes Coverity CID 1045.
svn path=/trunk/; revision=37090
Diffstat (limited to 'epan/dissectors/packet-sna.c')
| -rw-r--r-- | epan/dissectors/packet-sna.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/packet-sna.c b/epan/dissectors/packet-sna.c index f51475686a..57fb1507f7 100644 --- a/epan/dissectors/packet-sna.c +++ b/epan/dissectors/packet-sna.c @@ -2029,14 +2029,14 @@ dissect_fid4(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) bf_tree = proto_item_add_subtree(bf_item, ett_sna_th_fid); /* Byte 16 */ - proto_tree_add_boolean(tree, hf_sna_th_snai, tvb, offset, 1, th_byte); + proto_tree_add_boolean(bf_tree, hf_sna_th_snai, tvb, offset, 1, th_byte); /* We luck out here because in their infinite wisdom the SNA * architects placed the MPF and EFI fields in the same bitfield * locations, even though for FID4 they're not in byte 0. * Thank you IBM! */ - proto_tree_add_uint(tree, hf_sna_th_mpf, tvb, offset, 1, th_byte); - proto_tree_add_uint(tree, hf_sna_th_efi, tvb, offset, 1, th_byte); + proto_tree_add_uint(bf_tree, hf_sna_th_mpf, tvb, offset, 1, th_byte); + proto_tree_add_uint(bf_tree, hf_sna_th_efi, tvb, offset, 1, th_byte); offset += 2; /* 1 for byte 16, 1 for byte 17 which is reserved */ |