[PATCH] drivers/input/keyboard/atkbd.c: fix off by one errors
This patch fixes two possible off by one errors found by the Coverity checker (look at the period[i] and delay[j] in the two first unchanged lines). Signed-off-by: Adrian Bunk <bunk@stusta.de> Cc: Vojtech Pavlik <vojtech@suse.cz> Cc: Dmitry Torokhov <dtor_core@ameritech.net> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
7a019225c7
commit
53b21689ba
1 changed files with 4 additions and 2 deletions
|
@ -465,8 +465,10 @@ static int atkbd_event(struct input_dev *dev, unsigned int type, unsigned int co
|
||||||
if (atkbd->softrepeat) return 0;
|
if (atkbd->softrepeat) return 0;
|
||||||
|
|
||||||
i = j = 0;
|
i = j = 0;
|
||||||
while (i < 32 && period[i] < dev->rep[REP_PERIOD]) i++;
|
while (i < 31 && period[i] < dev->rep[REP_PERIOD])
|
||||||
while (j < 4 && delay[j] < dev->rep[REP_DELAY]) j++;
|
i++;
|
||||||
|
while (j < 3 && delay[j] < dev->rep[REP_DELAY])
|
||||||
|
j++;
|
||||||
dev->rep[REP_PERIOD] = period[i];
|
dev->rep[REP_PERIOD] = period[i];
|
||||||
dev->rep[REP_DELAY] = delay[j];
|
dev->rep[REP_DELAY] = delay[j];
|
||||||
param[0] = i | (j << 5);
|
param[0] = i | (j << 5);
|
||||||
|
|
Loading…
Reference in a new issue