diff options
author | Stig Bjørlykke <stig@bjorlykke.org> | 2007-11-28 09:30:10 +0000 |
---|---|---|
committer | Stig Bjørlykke <stig@bjorlykke.org> | 2007-11-28 09:30:10 +0000 |
commit | 60686647d2aefff5065a8869899d15d212115d03 (patch) | |
tree | f59b122f074b66121ea1737a243a8c1ecba333a7 | |
parent | c79cf40b3bb4583fe309edbc3a2a61778f3247be (diff) |
Fixed a possible infinite loop.
svn path=/trunk/; revision=23642
-rw-r--r-- | epan/dissectors/packet-atalk.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/packet-atalk.c b/epan/dissectors/packet-atalk.c index 40c35311a5..8f8dfb9268 100644 --- a/epan/dissectors/packet-atalk.c +++ b/epan/dissectors/packet-atalk.c @@ -1533,7 +1533,7 @@ dissect_atp_zip(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) ti = proto_tree_add_item(zip_tree, hf_zip_count, tvb, offset, 2, FALSE); offset += 2; sub_tree = proto_item_add_subtree(ti, ett_zip_zones_list); - for (i= 1; i <= count; i++) { + for (i= 0; i < count; i++) { len = tvb_get_guint8(tvb, offset); proto_tree_add_item(sub_tree, hf_zip_zone_name, tvb, offset, 1,FALSE); offset += len +1; @@ -1585,7 +1585,7 @@ dissect_ddp_zip(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) ti = proto_tree_add_item(zip_tree, hf_zip_network_count, tvb, offset, 1, FALSE); offset++; sub_tree = proto_item_add_subtree(ti, ett_zip_network_list); - for (i= 1; i <= count; i++) { + for (i= 0; i < count; i++) { proto_tree_add_item(sub_tree, hf_zip_network, tvb, offset, 2, FALSE); offset += 2; } @@ -1621,7 +1621,7 @@ dissect_ddp_zip(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) ti = proto_tree_add_item(zip_tree, hf_zip_network_count, tvb, offset, 1, FALSE); offset++; sub_tree = proto_item_add_subtree(ti, ett_zip_network_list); - for (i= 1; i <= count; i++) { + for (i= 0; i < count; i++) { net = tvb_get_ntohs(tvb, offset); ti = proto_tree_add_text(zip_tree, tvb, offset , 2, "Zone for network : %u", net); net_tree = proto_item_add_subtree(ti, ett_zip_network_list); |