diff options
author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2013-07-16 03:34:04 +0000 |
---|---|---|
committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2013-07-16 03:34:04 +0000 |
commit | 44e4c933de0fb566d1652c3ff170fd36af726b91 (patch) | |
tree | cc641f8b862aef751216acf8251c88a64b4d0df2 /epan/dissectors/packet-ipmi-picmg.c | |
parent | 86147047f5281825e7abd6b643914b4c35d891f4 (diff) |
Fix warning: cannot optimize possibly infinite loops. There was no actual infinite loop, but the intended loop iteration count could have been wrong due to the implicit "mod 256" when previously assigning a value to a guint8.
svn path=/trunk/; revision=50661
Diffstat (limited to 'epan/dissectors/packet-ipmi-picmg.c')
-rw-r--r-- | epan/dissectors/packet-ipmi-picmg.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-ipmi-picmg.c b/epan/dissectors/packet-ipmi-picmg.c index c9d6cd3b59..3ee4d27e59 100644 --- a/epan/dissectors/packet-ipmi-picmg.c +++ b/epan/dissectors/packet-ipmi-picmg.c @@ -908,8 +908,8 @@ static void rs12(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree) { static const int *byte1[] = { &hf_ipmi_picmg_12_dynamic, &hf_ipmi_picmg_12_pwr_lvl, NULL }; - guint8 v, v2, i, max; - guint32 tmp; + guint8 v, v2, i; + guint32 tmp, max; proto_tree_add_bitmask_text(tree, tvb, 0, 1, NULL, NULL, ett_ipmi_picmg_12_byte1, byte1, ENC_LITTLE_ENDIAN, BMT_NO_FALSE); proto_tree_add_item(tree, hf_ipmi_picmg_12_delay, tvb, 1, 1, ENC_LITTLE_ENDIAN); |