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

12 files changed, 43 insertions, 44 deletions

diff --git a/epan/dissectors/packet-ansi_a.c b/epan/dissectors/packet-ansi_a.c
index 77f1b7a3a1..3d56b01f65 100644
--- a/epan/dissectors/packet-ansi_a.c
+++ b/epan/dissectors/packet-ansi_a.c
@@ -5456,7 +5456,7 @@ elem_amps_hho_param(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree, gui
 static guint8
 elem_is2000_scr(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree, guint32 offset, guint len, gchar *add_string _U_, int string_len _U_)
 {
-    guint8      oct, num_con_rec, i;
+    guint8      oct, num_con_rec, ii;
     guint8      bit_mask, bit_offset;
     guint32     curr_offset, saved_offset;
     guint32     value;
@@ -5533,15 +5533,14 @@ elem_is2000_scr(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree, guint32
 
     curr_offset += 1;
 
-    for (i=1; i <= num_con_rec; i++)
+    for (ii=0; ii < num_con_rec; ii++)
     {
         oct = tvb_get_guint8(tvb, curr_offset);
 
         item =
             proto_tree_add_text(scr_subtree, tvb,
                 curr_offset, oct /* oct already includes the length octet itself */,
-                "Service option connection record [%u]",
-                i);
+                "Service option connection record [%u]", ii+1);
 
         subtree =
             proto_item_add_subtree(item, ett_srvc_con_rec);
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,
diff --git a/epan/dissectors/packet-dcerpc-epm.c b/epan/dissectors/packet-dcerpc-epm.c
index d6dd0a441a..d76cfcff99 100644
--- a/epan/dissectors/packet-dcerpc-epm.c
+++ b/epan/dissectors/packet-dcerpc-epm.c
@@ -320,7 +320,7 @@ epm_dissect_tower_data (tvbuff_t *tvb, int offset,
                              packet_info *pinfo, proto_tree *tree,
                              guint8 *drep _U_)
 {
-    guint16 num_floors, i;
+    guint16 num_floors, ii;
     dcerpc_info *di;
     const char *uuid_name;
     guint8   u8little_endian = DREP_LITTLE_ENDIAN;
@@ -334,7 +334,7 @@ epm_dissect_tower_data (tvbuff_t *tvb, int offset,
     proto_tree_add_uint(tree, hf_epm_tower_num_floors, tvb, offset, 2, num_floors);
     offset += 2;
 
-    for(i=1;i<=num_floors;i++){
+    for(ii=0;ii<num_floors;ii++){
         proto_item *it = NULL;
         proto_tree *tr = NULL;
         int old_offset = offset;
@@ -343,7 +343,7 @@ epm_dissect_tower_data (tvbuff_t *tvb, int offset,
         e_uuid_t uuid;
         proto_item *pi;
 
-        it = proto_tree_add_text(tree, tvb, offset, 0, "Floor %d ", i);
+        it = proto_tree_add_text(tree, tvb, offset, 0, "Floor %d ", ii+1);
         tr = proto_item_add_subtree(it, ett_epm_tower_floor);
 
         len = tvb_get_letohs(tvb, offset);
diff --git a/epan/dissectors/packet-iua.c b/epan/dissectors/packet-iua.c
index 6dad04f475..9fa59dc8d6 100644
--- a/epan/dissectors/packet-iua.c
+++ b/epan/dissectors/packet-iua.c
@@ -111,9 +111,9 @@ dissect_int_interface_identifier_parameter(tvbuff_t *parameter_tvb, proto_tree *
   offset = PARAMETER_VALUE_OFFSET;
 
   proto_item_append_text(parameter_item, " (");
-  for (id_number = 1; id_number <= number_of_ids; id_number++) {
+  for (id_number = 0; id_number < number_of_ids; id_number++) {
     proto_tree_add_item(parameter_tree, hf_int_interface_id, parameter_tvb, offset, INT_INTERFACE_ID_LENGTH, ENC_BIG_ENDIAN);
-    proto_item_append_text(parameter_item, (id_number > 1) ? ", %d" : "%d", tvb_get_ntohl(parameter_tvb, offset));
+    proto_item_append_text(parameter_item, (id_number > 0) ? ", %d" : "%d", tvb_get_ntohl(parameter_tvb, offset));
     offset += INT_INTERFACE_ID_LENGTH;
   }
   proto_item_append_text(parameter_item, ")");
@@ -221,10 +221,10 @@ dissect_integer_range_interface_identifier_parameter(tvbuff_t *parameter_tvb, pr
   number_of_ranges = (tvb_get_ntohs(parameter_tvb, PARAMETER_LENGTH_OFFSET) - PARAMETER_HEADER_LENGTH) / INTERVAL_LENGTH;
   offset = PARAMETER_VALUE_OFFSET;
   proto_item_append_text(parameter_item, " (");
-  for (range_number = 1; range_number <= number_of_ranges; range_number++) {
+  for (range_number = 0; range_number < number_of_ranges; range_number++) {
     proto_tree_add_item(parameter_tree, hf_interface_range_start, parameter_tvb, offset + START_OFFSET, START_LENGTH, ENC_BIG_ENDIAN);
     proto_tree_add_item(parameter_tree, hf_interface_range_end,   parameter_tvb, offset + END_OFFSET,   END_LENGTH,   ENC_BIG_ENDIAN);
-    proto_item_append_text(parameter_item, (range_number > 1) ? ", %d-%d" : "%d-%d",
+    proto_item_append_text(parameter_item, (range_number > 0) ? ", %d-%d" : "%d-%d",
                            tvb_get_ntohl(parameter_tvb, offset + START_OFFSET), tvb_get_ntohl(parameter_tvb, offset + END_OFFSET));
     offset += INTERVAL_LENGTH;
   }
diff --git a/epan/dissectors/packet-m2ua.c b/epan/dissectors/packet-m2ua.c
index 731c5f01b0..ccad7a5cde 100644
--- a/epan/dissectors/packet-m2ua.c
+++ b/epan/dissectors/packet-m2ua.c
@@ -329,7 +329,7 @@ dissect_interface_identifier_range_parameter(tvbuff_t *parameter_tvb, proto_tree
 
   number_of_ranges = (tvb_get_ntohs(parameter_tvb, PARAMETER_LENGTH_OFFSET) - PARAMETER_HEADER_LENGTH) / INTERVAL_LENGTH;
   offset = PARAMETER_VALUE_OFFSET;
-  for(range_number = 1; range_number <= number_of_ranges; range_number++) {
+  for(range_number = 0; range_number < number_of_ranges; range_number++) {
     proto_tree_add_item(parameter_tree, hf_interface_id_start, parameter_tvb, offset + START_OFFSET, START_LENGTH, ENC_BIG_ENDIAN);
     proto_tree_add_item(parameter_tree, hf_interface_id_stop,  parameter_tvb, offset + END_OFFSET,   END_LENGTH,   ENC_BIG_ENDIAN);
     offset += INTERVAL_LENGTH;
diff --git a/epan/dissectors/packet-mbtcp.c b/epan/dissectors/packet-mbtcp.c
index a78f1a6c71..1b6130273f 100644
--- a/epan/dissectors/packet-mbtcp.c
+++ b/epan/dissectors/packet-mbtcp.c
@@ -453,7 +453,7 @@ dissect_modbus(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _
     int           offset, group_offset, packet_type, temp_data;
     const char    *func_string = "";
     gint          payload_start, payload_len, event_index,
-                  i, byte_cnt, len, num_objects, object_index,
+                  ii, byte_cnt, len, num_objects, object_index,
                   object_len;
     guint32       group_byte_cnt, group_word_cnt;
     guint8        function_code, exception_code, mei_code, event_code, object_type;
@@ -802,9 +802,9 @@ dissect_modbus(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _
 
                     /* add subtrees to describe each group of packet */
                     group_offset = payload_start + 1;
-                    for (i = 0; i < byte_cnt / 7; i++) {
+                    for (ii = 0; ii < byte_cnt / 7; ii++) {
                         mi = proto_tree_add_text( modbus_tree, tvb, group_offset, 7,
-                                "Group %u", i);
+                                "Group %u", ii);
                         group_tree = proto_item_add_subtree(mi, ett_group_hdr);
                         proto_tree_add_item(group_tree, hf_modbus_reftype, tvb, group_offset, 1, ENC_BIG_ENDIAN);
                         proto_tree_add_item(group_tree, hf_modbus_lreference, tvb, group_offset + 1, 4, ENC_BIG_ENDIAN);
@@ -819,11 +819,11 @@ dissect_modbus(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _
 
                     /* add subtrees to describe each group of packet */
                     group_offset = payload_start + 1;
-                    i = 0;
+                    ii = 0;
                     while (byte_cnt > 0) {
                         group_byte_cnt = (guint32)tvb_get_guint8(tvb, group_offset);
                         mi = proto_tree_add_text( modbus_tree, tvb, group_offset, group_byte_cnt + 1,
-                                "Group %u", i);
+                                "Group %u", ii);
                         group_tree = proto_item_add_subtree(mi, ett_group_hdr);
                         proto_tree_add_uint(group_tree, hf_modbus_bytecnt, tvb, group_offset, 1,
                                 group_byte_cnt);
@@ -831,7 +831,7 @@ dissect_modbus(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _
                         dissect_mbtcp_data(tvb, pinfo, group_tree, function_code, group_offset + 2, group_byte_cnt - 1);
                         group_offset += (group_byte_cnt + 1);
                         byte_cnt -= (group_byte_cnt + 1);
-                        i++;
+                        ii++;
                     }
                 }
                 break;
@@ -844,12 +844,12 @@ dissect_modbus(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _
 
                     /* add subtrees to describe each group of packet */
                     group_offset = payload_start + 1;
-                    i = 0;
+                    ii = 0;
                     while (byte_cnt > 0) {
                         group_word_cnt = tvb_get_ntohs(tvb, group_offset + 5);
                         group_byte_cnt = (2 * group_word_cnt) + 7;
                         mi = proto_tree_add_text( modbus_tree, tvb, group_offset,
-                                group_byte_cnt, "Group %u", i);
+                                group_byte_cnt, "Group %u", ii);
                         group_tree = proto_item_add_subtree(mi, ett_group_hdr);
                         proto_tree_add_item(group_tree, hf_modbus_reftype, tvb, group_offset, 1, ENC_BIG_ENDIAN);
                         proto_tree_add_item(group_tree, hf_modbus_lreference, tvb, group_offset + 1, 4, ENC_BIG_ENDIAN);
@@ -858,7 +858,7 @@ dissect_modbus(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _
                         dissect_mbtcp_data(tvb, pinfo, group_tree, function_code, group_offset + 7, group_byte_cnt - 7);
                         group_offset += group_byte_cnt;
                         byte_cnt -= group_byte_cnt;
-                        i++;
+                        ii++;
                     }
                 }
                 break;
@@ -936,7 +936,7 @@ dissect_modbus(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _
                             doe = proto_tree_add_text(modbus_tree, tvb, payload_start+6, payload_len-6, "Objects");
 
                             object_index = 0;
-                            for (i = 1; i <= num_objects; i++)
+                            for (ii = 0; ii < num_objects; ii++)
                             {
                                 device_objects_tree = proto_item_add_subtree(doe, ett_device_id_objects);
 
@@ -946,7 +946,7 @@ dissect_modbus(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _
                                 object_type = tvb_get_guint8(tvb, payload_start+6+object_index);
                                 object_len = tvb_get_guint8(tvb, payload_start+6+object_index+1);
 
-                                doie = proto_tree_add_text(device_objects_tree, tvb, payload_start+6+object_index, 2+object_len, "Object #%d", i);
+                                doie = proto_tree_add_text(device_objects_tree, tvb, payload_start+6+object_index, 2+object_len, "Object #%d", ii+1);
                                 device_objects_item_tree = proto_item_add_subtree(doie, ett_device_id_object_items);
 
                                 proto_tree_add_item(device_objects_item_tree, hf_modbus_object_id, tvb, payload_start+6+object_index, 1, ENC_BIG_ENDIAN);
diff --git a/epan/dissectors/packet-ospf.c b/epan/dissectors/packet-ospf.c
index 4309ce4605..89fb121423 100644
--- a/epan/dissectors/packet-ospf.c
+++ b/epan/dissectors/packet-ospf.c
@@ -2655,7 +2655,7 @@ dissect_ospf_v2_lsa(tvbuff_t *tvb, int offset, proto_tree *tree,
         /* nr_links links follow
          * maybe we should put each of the links into its own subtree ???
          */
-        for (link_counter = 1; link_counter <= nr_links; link_counter++) {
+        for (link_counter = 0; link_counter < nr_links; link_counter++) {
             proto_tree *ospf_lsa_router_link_tree;
             proto_item *ti_local;
 
@@ -2727,7 +2727,7 @@ dissect_ospf_v2_lsa(tvbuff_t *tvb, int offset, proto_tree *tree,
              * According to RFC4915 the TOS metrics was never deployed and was subsequently deprecated,
              * but decoding still present because MT-ID use the same structure.
              */
-            for (metric_counter = 1; metric_counter <= nr_metric; metric_counter++) {
+            for (metric_counter = 0; metric_counter < nr_metric; metric_counter++) {
                 proto_tree_add_text(ospf_lsa_router_link_tree, tvb, offset, 4, "%s: %u, Metric: %u",
                                     metric_type_str,
                                     tvb_get_guint8(tvb, offset),
diff --git a/epan/dissectors/packet-pagp.c b/epan/dissectors/packet-pagp.c
index d6da392b98..a36c453e6e 100644
--- a/epan/dissectors/packet-pagp.c
+++ b/epan/dissectors/packet-pagp.c
@@ -159,7 +159,7 @@ dissect_pagp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
       guint16 num_tlvs;
       guint16 tlv;
       guint16 len;
-      guint16 i;
+      guint16 ii;
       guint16 offset = PAGP_FIRST_TLV;
       guint8  raw_octet;
 
@@ -334,7 +334,7 @@ dissect_pagp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 
 	    /* dump TLV entries */
 
-	    for ( i = 1; i <= num_tlvs; i++ ) {
+	    for ( ii = 0; ii < num_tlvs; ii++ ) {
 
 		tlv = tvb_get_ntohs(tvb, offset);
 		len = tvb_get_ntohs(tvb, offset + 2);
@@ -345,7 +345,7 @@ dissect_pagp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 		}
 
 		tlv_item = proto_tree_add_text (pagp_tree, tvb, offset, len,
-			   "TLV Entry #%d", i);
+			   "TLV Entry #%d", ii+1);
 
 		tlv_tree = proto_item_add_subtree (tlv_item, ett_pagp_tlvs);
 		proto_tree_add_uint_format (tlv_tree, hf_pagp_tlv, tvb,
diff --git a/epan/dissectors/packet-sua.c b/epan/dissectors/packet-sua.c
index 06acd70127..ed690ac694 100644
--- a/epan/dissectors/packet-sua.c
+++ b/epan/dissectors/packet-sua.c
@@ -565,7 +565,7 @@ dissect_routing_context_parameter(tvbuff_t *parameter_tvb, proto_tree *parameter
 
   number_of_contexts = (tvb_get_ntohs(parameter_tvb, PARAMETER_LENGTH_OFFSET) - PARAMETER_HEADER_LENGTH) / 4;
   context_offset = PARAMETER_VALUE_OFFSET;
-  for(context_number=1; context_number <= number_of_contexts; context_number++) {
+  for(context_number=0; context_number < number_of_contexts; context_number++) {
     proto_tree_add_item(parameter_tree, hf_sua_routing_context, parameter_tvb, context_offset, ROUTING_CONTEXT_LENGTH, ENC_BIG_ENDIAN);
     context_offset += ROUTING_CONTEXT_LENGTH;
   };
@@ -747,7 +747,7 @@ dissect_affected_destinations_parameter(tvbuff_t *parameter_tvb, proto_tree *par
 
   number_of_destinations= (tvb_get_ntohs(parameter_tvb, PARAMETER_LENGTH_OFFSET) - PARAMETER_HEADER_LENGTH) / 4;
   destination_offset = PARAMETER_VALUE_OFFSET;
-  for(destination_number=1; destination_number <= number_of_destinations; destination_number++) {
+  for(destination_number=0; destination_number < number_of_destinations; destination_number++) {
     proto_tree_add_item(parameter_tree, hf_sua_mask, parameter_tvb, destination_offset + AFFECTED_MASK_OFFSET, AFFECTED_MASK_LENGTH, ENC_BIG_ENDIAN);
     dpc_item = proto_tree_add_item(parameter_tree, hf_sua_dpc,  parameter_tvb, destination_offset + AFFECTED_DPC_OFFSET,  AFFECTED_DPC_LENGTH,  ENC_BIG_ENDIAN);
     if (mtp3_pc_structured())
diff --git a/epan/dissectors/packet-v5ua.c b/epan/dissectors/packet-v5ua.c
index 17aad6e12c..5acf7bc27d 100644
--- a/epan/dissectors/packet-v5ua.c
+++ b/epan/dissectors/packet-v5ua.c
@@ -163,10 +163,10 @@ dissect_int_interface_identifier_parameter(tvbuff_t *parameter_tvb, proto_tree *
   offset = INT_IF_ID_LINK_OFFSET;
   identifier = tvb_get_ntohl(parameter_tvb,INT_IF_ID_LINK_OFFSET)>>5;
   proto_item_append_text(parameter_item, "(");
-  for (id_number = 1; id_number <= number_of_ids; id_number++) {
+  for (id_number = 0; id_number < number_of_ids; id_number++) {
     proto_tree_add_item(parameter_tree, hf_link_id, parameter_tvb, offset, INT_IF_ID_LINK_LENGTH, ENC_BIG_ENDIAN);
     identifier = tvb_get_ntohl(parameter_tvb,offset)>>5;
-    if (id_number < 2) {
+    if (id_number < 1) {
 	proto_item_append_text(parameter_item, "L:%d",identifier);
     } else {
 	proto_item_append_text(parameter_item, " | L:%d",identifier);
@@ -612,7 +612,7 @@ dissect_integer_range_interface_identifier_parameter(tvbuff_t *parameter_tvb, pr
 
   number_of_ranges = (tvb_get_ntohs(parameter_tvb, INT_RANGE_LENGTH_OFFSET) - INT_RANGE_HEADER_LENGTH) / IF_ID_INTERVAL_LENGTH;
   offset = INT_RANGE_START_OFFSET;
-  for(range_number = 1; range_number <= number_of_ranges; range_number++) {
+  for(range_number = 0; range_number < number_of_ranges; range_number++) {
     proto_tree_add_item(parameter_tree, hf_if_range_start, parameter_tvb, offset + IF_ID_START_OFFSET, IF_ID_START_LENGTH, ENC_BIG_ENDIAN);
     proto_tree_add_item(parameter_tree, hf_if_range_end,   parameter_tvb, offset + IF_ID_END_OFFSET,   IF_ID_END_LENGTH,   ENC_BIG_ENDIAN);
     offset += IF_ID_INTERVAL_LENGTH;
diff --git a/epan/dissectors/packet-vuze-dht.c b/epan/dissectors/packet-vuze-dht.c
index 9f89301ec9..a8857485bd 100644
--- a/epan/dissectors/packet-vuze-dht.c
+++ b/epan/dissectors/packet-vuze-dht.c
@@ -358,7 +358,7 @@ dissect_vuze_dht_contacts(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, i
 
   ti = proto_tree_add_none_format( tree, hf_vuze_dht_contacts, tvb, offset, 0, "%d contacts", contacts_count );
   sub_tree = proto_item_add_subtree(ti, ett_vuze_dht_contacts);
-  for( i=1; i<=contacts_count; i++ )
+  for( i=0; i<contacts_count; i++ )
     offset = dissect_vuze_dht_contact( tvb, pinfo, sub_tree, offset );
 
   return offset;
@@ -400,7 +400,7 @@ dissect_vuze_dht_keys(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, int o
 
   ti = proto_tree_add_none_format( tree, hf_vuze_dht_keys, tvb, offset, 0, "%d keys", keys_count );
   sub_tree = proto_item_add_subtree(ti, ett_vuze_dht_keys);
-  for( i=1; i<=keys_count; i++ )
+  for( i=0; i<keys_count; i++ )
     offset = dissect_vuze_dht_key( tvb, pinfo, sub_tree, offset );
 
   return offset;
@@ -506,7 +506,7 @@ dissect_vuze_dht_value_group(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree
   proto_tree_add_item( sub_tree, hf_vuze_dht_values_count, tvb, offset, TL_SHORT, ENC_BIG_ENDIAN );
   offset += TL_SHORT;
 
-  for( i=1; i<=values_count; i++ )
+  for( i=0; i<values_count; i++ )
     offset = dissect_vuze_dht_value( tvb, pinfo, sub_tree, offset, ver );
 
   return offset;
@@ -524,7 +524,7 @@ dissect_vuze_dht_value_groups(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tre
 
   ti = proto_tree_add_none_format( tree, hf_vuze_dht_value_groups, tvb, offset, 0, "%d value groups", value_groups_count );
   sub_tree = proto_item_add_subtree(ti, ett_vuze_dht_value_groups);
-  for( i=1; i<=value_groups_count; i++ )
+  for( i=0; i<value_groups_count; i++ )
     offset = dissect_vuze_dht_value_group( tvb, pinfo, sub_tree, offset, ver );
 
   return offset;
@@ -582,7 +582,7 @@ dissect_vuze_dht_network_coordinates(tvbuff_t *tvb, packet_info *pinfo, proto_tr
 
     ti = proto_tree_add_none_format( tree, hf_vuze_dht_network_coordinates, tvb, offset, 0, "%d network coordinates", network_coordinates_count );
     sub_tree = proto_item_add_subtree(ti, ett_vuze_dht_network_coordinates);
-    for( i=1; i<=network_coordinates_count; i++ )
+    for( i=0; i<network_coordinates_count; i++ )
       offset = dissect_vuze_dht_network_coordinate( tvb, pinfo, sub_tree, offset );
   }
   return offset;
diff --git a/epan/dissectors/packet-wow.c b/epan/dissectors/packet-wow.c
index 48c4e42b94..cca8b476fe 100644
--- a/epan/dissectors/packet-wow.c
+++ b/epan/dissectors/packet-wow.c
@@ -197,7 +197,7 @@ dissect_wow_pdu(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 	guint8 cmd, srp_i_len, srp_g_len, srp_n_len;
 	guint16 num_realms;
 	guint32 offset = 0;
-	gint len, i;
+	gint len, ii;
 
 	col_set_str(pinfo->cinfo, COL_PROTOCOL, "WOW");
 
@@ -383,7 +383,7 @@ dissect_wow_pdu(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 				num_realms = tvb_get_letohs(tvb, offset);
 				offset += 2;
 
-				for(i = 1; i <= num_realms; i++) {
+				for(ii = 0; ii < num_realms; ii++) {
 					realm_name = tvb_get_ephemeral_stringz(tvb,
 								     offset + 3,
 								     &len);