diff options
| author | Bill Meier <wmeier@newsguy.com> | 2008-05-15 01:44:53 +0000 |
|---|---|---|
| committer | Bill Meier <wmeier@newsguy.com> | 2008-05-15 01:44:53 +0000 |
| commit | 931cc922a14049c904a6ac416656c36caf01d6b3 (patch) | |
| tree | 3d1112b08a02413f491bfad24d294022e8ef867c /epan/dissectors/packet-cpfi.c | |
| parent | 78f4e06717899b3e6504300f3b47e61bc3989ee6 (diff) | |
Fix some of the Errors/warnings detected by checkapi.
svn path=/trunk/; revision=25305
Diffstat (limited to 'epan/dissectors/packet-cpfi.c')
| -rw-r--r-- | epan/dissectors/packet-cpfi.c | 37 |
1 files changed, 25 insertions, 12 deletions
diff --git a/epan/dissectors/packet-cpfi.c b/epan/dissectors/packet-cpfi.c index 633ca6ac23..9f7ae45a97 100644 --- a/epan/dissectors/packet-cpfi.c +++ b/epan/dissectors/packet-cpfi.c @@ -167,6 +167,7 @@ dissect_cpfi_header(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) guint8 dst_port = 0; proto_item *extra_item = NULL; proto_tree *extra_tree = NULL; + proto_item *hidden_item; /* add a tree for the header */ if ( tree != NULL) @@ -241,21 +242,33 @@ dissect_cpfi_header(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) /* For "real" TDAs (i.e. not for microTDAs), add hidden addresses to allow filtering */ if ( src != 0 ) { - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_instance, tvb, 0, 1, &src_instance); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_src_instance, tvb, 0, 1, &src_instance); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_board, tvb, 0, 1, &src_board); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_src_board, tvb, 0, 1, &src_board); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_port, tvb, 0, 1, &src_port); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_src_port, tvb, 0, 1, &src_port); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_instance, tvb, 0, 1, &src_instance); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_src_instance, tvb, 0, 1, &src_instance); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_board, tvb, 0, 1, &src_board); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_src_board, tvb, 0, 1, &src_board); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_port, tvb, 0, 1, &src_port); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_src_port, tvb, 0, 1, &src_port); + PROTO_ITEM_SET_HIDDEN(hidden_item); } if ( dst != 0 ) { - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_instance, tvb, 0, 1, &dst_instance); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_dst_instance, tvb, 0, 1, &dst_instance); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_board, tvb, 0, 1, &dst_board); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_dst_board, tvb, 0, 1, &dst_board); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_port, tvb, 0, 1, &dst_port); - proto_tree_add_bytes_hidden(extra_tree, hf_cpfi_t_dst_port, tvb, 0, 1, &dst_port); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_instance, tvb, 0, 1, &dst_instance); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_dst_instance, tvb, 0, 1, &dst_instance); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_board, tvb, 0, 1, &dst_board); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_dst_board, tvb, 0, 1, &dst_board); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_port, tvb, 0, 1, &dst_port); + PROTO_ITEM_SET_HIDDEN(hidden_item); + hidden_item = proto_tree_add_bytes(extra_tree, hf_cpfi_t_dst_port, tvb, 0, 1, &dst_port); + PROTO_ITEM_SET_HIDDEN(hidden_item); } /* add word 1 components to the protocol tree */ |