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-iua.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-iua.c')
-rw-r--r-- | epan/dissectors/packet-iua.c | 8 |
1 files changed, 4 insertions, 4 deletions
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; } |