osmith/wireshark
1
0
Fork 0
forked from osmocom/wireshark
Code Pull requests Releases Wiki Activity

Fix Coverity ID 1300: UNUSED_VALUE in dissect_spotlight: "returned_pointer

Browse source 
is never used" because of the return -1 at the end of this if block.
Removed item_toc = assignment and fixed indentation.


svn path=/trunk/; revision=39469
This commit is contained in:
Stephen Fisher 2011-10-19 16:08:13 +00:00
parent 4b930ccb33
commit 20ae7e93d5
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
epan/dissectors/packet-afp.c
View file

@ -4336,12 +4336,12 @@ dissect_spotlight(tvbuff_t *tvb, proto_tree *tree, gint offset)
/* ToC */
offset += (gint)toc_offset;
if (toc_entries < 1) {
item_toc = proto_tree_add_text(tree,
tvb,
offset,
(gint)querylen - (gint)toc_offset,
"Complex types ToC (%u < 1 - bogus)",
toc_entries);
proto_tree_add_text(tree,
tvb,
offset,
(gint)querylen - (gint)toc_offset,
"Complex types ToC (%u < 1 - bogus)",
toc_entries);
return -1;
}
toc_entries -= 1;