Use the 'proto_tree *' (not the 'proto_item *') for a proto_tree_add_item(): Coverity...

svn path=/trunk/; revision=37369

diff --git a/epan/dissectors/packet-maccontrol.c b/epan/dissectors/packet-maccontrol.c
index 225457825326c3b54fc98d0a9fa13b4a9bb43942..8066d3681f9e1a87250a5373af064f1f947afcb5 100644 (file)
--- a/epan/dissectors/packet-maccontrol.c
+++ b/epan/dissectors/packet-maccontrol.c
@@ -130,14 +130,14 @@ dissect_macctrl(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
       col_set_str(pinfo->cinfo, COL_INFO, "MAC CLASS BASED FLOW CONTROL PAUSE");
       if (tree) {
         int i;
-        proto_tree_add_bitmask(macctrl_tree, tvb, 2, hf_macctrl_cbfc_enbv, 
+        proto_tree_add_bitmask(macctrl_tree, tvb, 2, hf_macctrl_cbfc_enbv,
                                ett_macctrl_cbfc_enbv, macctrl_cbfc_enbv_list, FALSE);
 
         ti = proto_tree_add_text(macctrl_tree, tvb, 4, 8*2, "CBFC Class Pause Times");
         pause_times_tree = proto_item_add_subtree(ti, ett_macctrl_cbfc_pause_times);
 
         for (i=0; i<8; i++) {
-          proto_tree_add_item(ti, *macctrl_cbfc_pause_times_list[i], tvb, 4+i*2, 2, FALSE);
+          proto_tree_add_item(pause_times_tree, *macctrl_cbfc_pause_times_list[i], tvb, 4+i*2, 2, FALSE);
         }
       }
       break;