diff options
| author | Anders Broman <anders.broman@ericsson.com> | 2008-05-21 05:41:39 +0000 |
|---|---|---|
| committer | Anders Broman <anders.broman@ericsson.com> | 2008-05-21 05:41:39 +0000 |
| commit | 45b3155321b9c080a7bd27c262a132d4eef7a873 (patch) | |
| tree | 1f9bc7a3f013fc8d55588e5bdd0414a03bb70406 /asn1/h225 | |
| parent | aaeee9accaddf89263d40fbe20a57affecbeefc4 (diff) | |
| download | wireshark-45b3155321b9c080a7bd27c262a132d4eef7a873.tar.gz wireshark-45b3155321b9c080a7bd27c262a132d4eef7a873.tar.bz2 wireshark-45b3155321b9c080a7bd27c262a132d4eef7a873.zip | |
Fix some of the Errors/warnings detected by checkapi.
svn path=/trunk/; revision=25335
Diffstat (limited to 'asn1/h225')
| -rw-r--r-- | asn1/h225/Makefile.nmake | 4 | ||||
| -rw-r--r-- | asn1/h225/packet-h225-template.c | 10 |
2 files changed, 11 insertions, 3 deletions
diff --git a/asn1/h225/Makefile.nmake b/asn1/h225/Makefile.nmake index 5a32997c60..6aa21c4986 100644 --- a/asn1/h225/Makefile.nmake +++ b/asn1/h225/Makefile.nmake @@ -27,3 +27,7 @@ include ../Makefile.preinc.nmake include Makefile.common include ../Makefile.inc.nmake +checkapi: + $(PERL) ../../tools/checkAPIs.pl \ + $(PROTOCOL_NAME).cnf \ + packet-$(PROTOCOL_NAME)-template.c
\ No newline at end of file diff --git a/asn1/h225/packet-h225-template.c b/asn1/h225/packet-h225-template.c index 0d997d35f2..17646e0ef7 100644 --- a/asn1/h225/packet-h225-template.c +++ b/asn1/h225/packet-h225-template.c @@ -382,6 +382,7 @@ static void reset_h225_packet_info(h225_packet_info *pi) static void ras_call_matching(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, h225_packet_info *pi) { + proto_item *hidden_item; conversation_t* conversation = NULL; h225ras_call_info_key h225ras_call_key; h225ras_call_t *h225ras_call = NULL; @@ -443,7 +444,8 @@ static void ras_call_matching(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tre /* No, so it's a duplicate request. Mark it as such. */ pi->is_duplicate = TRUE; - proto_tree_add_uint_hidden(tree, hf_h225_ras_dup, tvb, 0,0, pi->requestSeqNum); + hidden_item = proto_tree_add_uint(tree, hf_h225_ras_dup, tvb, 0,0, pi->requestSeqNum); + PROTO_ITEM_SET_HIDDEN(hidden_item); } break; } @@ -493,7 +495,8 @@ static void ras_call_matching(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tre /* if this is an ACF, ARJ or DCF, DRJ, give guid to tap and make it filterable */ if (msg_category == 3 || msg_category == 5) { pi->guid = h225ras_call->guid; - proto_tree_add_guid_hidden(tree, hf_h225_guid, tvb, 0, GUID_LEN, &pi->guid); + hidden_item = proto_tree_add_guid(tree, hf_h225_guid, tvb, 0, GUID_LEN, &pi->guid); + PROTO_ITEM_SET_HIDDEN(hidden_item); } if (h225ras_call->rsp_num == 0) { @@ -509,7 +512,8 @@ static void ras_call_matching(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tre /* No, so it's a duplicate response. Mark it as such. */ pi->is_duplicate = TRUE; - proto_tree_add_uint_hidden(tree, hf_h225_ras_dup, tvb, 0,0, pi->requestSeqNum); + hidden_item = proto_tree_add_uint(tree, hf_h225_ras_dup, tvb, 0,0, pi->requestSeqNum); + PROTO_ITEM_SET_HIDDEN(hidden_item); } } |