diff options
| author | Bill Meier <wmeier@newsguy.com> | 2008-05-13 01:23:37 +0000 |
|---|---|---|
| committer | Bill Meier <wmeier@newsguy.com> | 2008-05-13 01:23:37 +0000 |
| commit | 9ca142b16012f55fb5d9334ce61a7fcbdfd0470f (patch) | |
| tree | 632b7ced652e6df08b06a7843c69fdabff89131d /epan/dissectors/packet-icap.c | |
| parent | 355dfa770d33f15e0be90bb7c8c4f9c1fb1be2fc (diff) | |
Fix some of the Errors/warnings detected by checkapi.
svn path=/trunk/; revision=25282
Diffstat (limited to 'epan/dissectors/packet-icap.c')
| -rw-r--r-- | epan/dissectors/packet-icap.c | 21 |
1 files changed, 13 insertions, 8 deletions
diff --git a/epan/dissectors/packet-icap.c b/epan/dissectors/packet-icap.c index e9b23da6a6..8653df54be 100644 --- a/epan/dissectors/packet-icap.c +++ b/epan/dissectors/packet-icap.c @@ -62,6 +62,7 @@ dissect_icap(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) { proto_tree *icap_tree = NULL; proto_item *ti = NULL; + proto_item *hidden_item; gint offset = 0; const guchar *line; gint next_offset; @@ -231,23 +232,27 @@ is_icap_header: switch (icap_type) { case ICAP_OPTIONS: - proto_tree_add_boolean_hidden(icap_tree, - hf_icap_options, tvb, 0, 0, 1); + hidden_item = proto_tree_add_boolean(icap_tree, + hf_icap_options, tvb, 0, 0, 1); + PROTO_ITEM_SET_HIDDEN(hidden_item); break; case ICAP_REQMOD: - proto_tree_add_boolean_hidden(icap_tree, - hf_icap_reqmod, tvb, 0, 0, 1); + hidden_item = proto_tree_add_boolean(icap_tree, + hf_icap_reqmod, tvb, 0, 0, 1); + PROTO_ITEM_SET_HIDDEN(hidden_item); break; case ICAP_RESPMOD: - proto_tree_add_boolean_hidden(icap_tree, - hf_icap_respmod, tvb, 0, 0, 1); + hidden_item = proto_tree_add_boolean(icap_tree, + hf_icap_respmod, tvb, 0, 0, 1); + PROTO_ITEM_SET_HIDDEN(hidden_item); break; case ICAP_RESPONSE: - proto_tree_add_boolean_hidden(icap_tree, - hf_icap_response, tvb, 0, 0, 1); + hidden_item = proto_tree_add_boolean(icap_tree, + hf_icap_response, tvb, 0, 0, 1); + PROTO_ITEM_SET_HIDDEN(hidden_item); break; case ICAP_OTHER: |