diff options
author | Bill Meier <wmeier@newsguy.com> | 2012-10-11 19:39:32 +0000 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2012-10-11 19:39:32 +0000 |
commit | 82d40100d2ee06d520158e1c4de4262d3ad9a7b8 (patch) | |
tree | 52e3c0313c9fe57de85dd47a22a60b70338e69e0 /epan/dissectors/packet-clique-rm.c | |
parent | ad0219d033baa2db7e7ae2361cbf346205878591 (diff) |
Change 'for (i=1; i<=n;...' to 'for (i=0; i<n; ...)'
Done on general principles altho upon inspection
none of the cases changed would have actually
resulted in an infinite loop.
svn path=/trunk/; revision=45478
Diffstat (limited to 'epan/dissectors/packet-clique-rm.c')
-rw-r--r-- | epan/dissectors/packet-clique-rm.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/epan/dissectors/packet-clique-rm.c b/epan/dissectors/packet-clique-rm.c index 03b1e44aa2..c8fbc2531b 100644 --- a/epan/dissectors/packet-clique-rm.c +++ b/epan/dissectors/packet-clique-rm.c @@ -127,7 +127,7 @@ dissect_sender_array(proto_tree *clique_rm_tree, int hf_header, gint ett_header, tree = proto_item_add_subtree(ti, ett_header); offset++; - for (i = 1; i <= count; i++, offset += 4) + for (i = 0; i < count; i++, offset += 4) proto_tree_add_item(tree, hf_header_sender, tvb, offset, 4, ENC_BIG_ENDIAN); } @@ -160,7 +160,7 @@ dissect_depends(proto_tree *clique_rm_tree, tvbuff_t *tvb, int offset) { proto_item *ti, *depend_item; proto_tree *tree, *depend_tree; - guint i, count; + guint ii, count; int len; count = tvb_get_guint8(tvb, offset); @@ -172,9 +172,9 @@ dissect_depends(proto_tree *clique_rm_tree, tvbuff_t *tvb, int offset) offset += 1; tree = proto_item_add_subtree(ti, ett_clique_rm_depends); - for (i = 1; i <= count; i++) + for (ii = 0; ii < count; ii++) { - depend_item = proto_tree_add_text(tree, tvb, offset, 8, "Depend item %d", i); + depend_item = proto_tree_add_text(tree, tvb, offset, 8, "Depend item %d", ii+1); depend_tree = proto_item_add_subtree(depend_item, ett_clique_rm_depends_item); proto_tree_add_item(depend_tree, hf_clique_rm_depend_sender, |