diff options
| author | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-08-02 13:09:01 +0000 |
|---|---|---|
| committer | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-08-02 13:09:01 +0000 |
| commit | de7824f86d308a2681c91f2c40a84f3f4f446b50 (patch) | |
| tree | 04f1080419d9ebbc824058e1b98fca361e8d0d66 /epan/dissectors/packet-iec104.c | |
| parent | 19b6078dea4594fb1a9bddc48abcffba58b53f05 (diff) | |
Fix cases where the type of a for loop variable is smaller than the type of the "end" variable.
In some cases the usage may have been benign since it can be seen by code inspection that the maximum value of the end variable can't exceed the maximum value of the loop variable.
However, on general principles, all the usages have been fixed.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@33692 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-iec104.c')
| -rw-r--r-- | epan/dissectors/packet-iec104.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/packet-iec104.c b/epan/dissectors/packet-iec104.c index 64b03d82ce..a500475714 100644 --- a/epan/dissectors/packet-iec104.c +++ b/epan/dissectors/packet-iec104.c @@ -1336,10 +1336,10 @@ static void dissect_iec104asdu(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tr /* Is is called twice: For 'Packet List' and for 'Packet Details' */ static void dissect_iec104apci(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) { - guint8 TcpLen = tvb_length(tvb); - guint16 Brossa = 0; + guint TcpLen = tvb_length(tvb); + guint Brossa = 0; guint8 Start; - guint8 Off; + guint Off; guint8 Byte1 = 0; struct apciheader * apcih; emem_strbuf_t * res; |