diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2010-02-02 07:43:44 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-02-03 23:16:46 -0500 |
commit | e23e11792a77c7d54f363841cd5e33074fb0c8c9 (patch) | |
tree | 0d3cf838f24f20ac1e370f904af11b88c2e23a47 | |
parent | 4bbd1a1903639f826215d76af74f4901efc34daa (diff) |
hisax: timeout off by one in waitrecmsg()
With `while (timeout++ < maxdelay)' timeout reaches maxdelay + 1 after the
loop This is probably unlikely a problem in practice.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Cc: Karsten Keil <isdn@linux-pingi.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/isdn/hisax/isar.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/isdn/hisax/isar.c b/drivers/isdn/hisax/isar.c index bfeb9b6aa043..6bde16c00fb5 100644 --- a/drivers/isdn/hisax/isar.c +++ b/drivers/isdn/hisax/isar.c | |||
@@ -138,7 +138,7 @@ waitrecmsg(struct IsdnCardState *cs, u_char *len, | |||
138 | while((!(cs->BC_Read_Reg(cs, 0, ISAR_IRQBIT) & ISAR_IRQSTA)) && | 138 | while((!(cs->BC_Read_Reg(cs, 0, ISAR_IRQBIT) & ISAR_IRQSTA)) && |
139 | (timeout++ < maxdelay)) | 139 | (timeout++ < maxdelay)) |
140 | udelay(1); | 140 | udelay(1); |
141 | if (timeout >= maxdelay) { | 141 | if (timeout > maxdelay) { |
142 | printk(KERN_WARNING"isar recmsg IRQSTA timeout\n"); | 142 | printk(KERN_WARNING"isar recmsg IRQSTA timeout\n"); |
143 | return(0); | 143 | return(0); |
144 | } | 144 | } |