diff options
author | Bill Meier <wmeier@newsguy.com> | 2011-04-12 13:27:08 +0000 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2011-04-12 13:27:08 +0000 |
commit | c424bd7c83439f3175a73369651197ab2d34ed4b (patch) | |
tree | c596c2cb3facebaf762b50a76700206b936bdbab /plugins | |
parent | e3f4dc17a93f2b614d0229ede3b5963d2a4a94c7 (diff) |
Don't assign to a proto_item * if the value won't be used: Coverity 1158 & 1159.
svn path=/trunk/; revision=36581
Diffstat (limited to 'plugins')
-rw-r--r-- | plugins/gryphon/packet-gryphon.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/plugins/gryphon/packet-gryphon.c b/plugins/gryphon/packet-gryphon.c index 63bf50d617..565a585ac9 100644 --- a/plugins/gryphon/packet-gryphon.c +++ b/plugins/gryphon/packet-gryphon.c @@ -1392,7 +1392,6 @@ static int cmd_sched_rep(tvbuff_t *tvb, int offset, proto_tree *pt) { int msglen; - proto_item *item; int save_offset; unsigned int x; const char *type; @@ -1407,8 +1406,8 @@ cmd_sched_rep(tvbuff_t *tvb, int offset, proto_tree *pt) offset += 4; msglen -= 4; x= tvb_get_guint8(tvb, offset); - item = proto_tree_add_text(pt, tvb, offset, 1, "Message index: %d", x); - item = proto_tree_add_text(pt, tvb, offset + 1, 3, "reserved"); + proto_tree_add_text(pt, tvb, offset, 1, "Message index: %d", x); + proto_tree_add_text(pt, tvb, offset + 1, 3, "reserved"); offset += 4; msglen -= 4; save_offset = offset; @@ -1822,9 +1821,9 @@ cmd_options(tvbuff_t *tvb, int offset, proto_tree *pt) const char *string, *string1; msglen = tvb_reported_length_remaining(tvb, offset); - item = proto_tree_add_text(pt, tvb, offset, 1, "Handle: %u", + proto_tree_add_text(pt, tvb, offset, 1, "Handle: %u", tvb_get_guint8(tvb, offset)); - item = proto_tree_add_text(pt, tvb, offset+1, 3, "reserved"); + proto_tree_add_text(pt, tvb, offset+1, 3, "reserved"); offset += 4; msglen -= 4; for (i = 1; msglen > 0; i++) { |