diff options
author | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-04-04 17:24:17 +0000 |
---|---|---|
committer | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-04-04 17:24:17 +0000 |
commit | 2edade70fe1edc2a82581e951b224e5b96eb967f (patch) | |
tree | d4a08adc8ecdf01e23b67be970ab8a9ed31b7fb8 /epan/dissectors/packet-amqp.c | |
parent | b5b1f2dfbab55efb891a51e3032e29a30ed582c5 (diff) |
Don't assign to a proto_item * if the value won't be used: Coverity 827;
Comment out some currently unused code: Coverity 828.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@36453 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-amqp.c')
-rw-r--r-- | epan/dissectors/packet-amqp.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/epan/dissectors/packet-amqp.c b/epan/dissectors/packet-amqp.c index c864f6b132..389aaafbd4 100644 --- a/epan/dissectors/packet-amqp.c +++ b/epan/dissectors/packet-amqp.c @@ -31,7 +31,7 @@ /* * See * - * http://www.amqp.org/confluence/display/AMQP/AMQP+Specification + * http://www.amqp.org/confluence/display/AMQP/AMQP+Specification * * for specifications for various versions of the AMQP protocol. */ @@ -2263,10 +2263,10 @@ dissect_amqp_0_10_array(tvbuff_t *tvb, offset - element_start, "(unknown type %d)", type); - /* Don't bother continuing through the loop: we don't know how - * much to increment the offset by and the type doesn't change - * so there's nothing interesting to do... - */ + /* Don't bother continuing through the loop: we don't know how + * much to increment the offset by and the type doesn't change + * so there's nothing interesting to do... + */ return; } @@ -2569,7 +2569,7 @@ dissect_amqp_0_10_connection(tvbuff_t *tvb, proto_item_set_expert_flags(flags_item, PI_PROTOCOL, PI_WARN); if (flag1 & 0x01) { /* virtual-host (str8) */ - ti = proto_tree_add_item(args_tree, + proto_tree_add_item(args_tree, hf_amqp_method_connection_open_virtual_host, tvb, offset + 1, @@ -5609,12 +5609,15 @@ dissect_amqp_0_10_frame(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) guint32 struct_length; int offset; emem_strbuf_t *strbuf; + +#if 0 /* XXX: Not currently used ?? *(/ conversation_t *conv; amqp_conv *conn; /* Find (or build) conversation to remember the protocol version */ conv = find_or_create_conversation(pinfo); conn = conversation_get_proto_data(conv, proto_amqp); +#endif col_set_str(pinfo->cinfo, COL_PROTOCOL, "AMQP"); col_clear(pinfo->cinfo, COL_INFO); |