diff options
| author | Luc Van Oostenryck | 2005-06-29 05:44:44 +0200 |
|---|---|---|
| committer | Linus Torvalds | 2005-06-29 06:20:29 +0200 |
| commit | 45ae36cbce9c7f55615e2b5f8faf23c7a06d0bd2 (patch) | |
| tree | 8d8a6a7e7cb0bb5d991e9dad1fc602528db63c76 /drivers | |
| parent | [PATCH] Using msleep() instead of HZ (diff) | |
| download | kernel-qcow2-linux-45ae36cbce9c7f55615e2b5f8faf23c7a06d0bd2.tar.gz kernel-qcow2-linux-45ae36cbce9c7f55615e2b5f8faf23c7a06d0bd2.tar.xz kernel-qcow2-linux-45ae36cbce9c7f55615e2b5f8faf23c7a06d0bd2.zip | |
[PATCH] drivers/char/tipar.c: off by one array access
In the setup function, the delay variable is initialized with ints[2],
but ints is declared as:
int ints[2];
Since the module parameter should correspond to:
tipar=timeout,delay
I suppose that the following patch fix the problem.
Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@looxix.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/char/tipar.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/tipar.c b/drivers/char/tipar.c index 659335d80ee7..ec78d2f161f7 100644 --- a/drivers/char/tipar.c +++ b/drivers/char/tipar.c @@ -396,7 +396,7 @@ static struct file_operations tipar_fops = { static int __init tipar_setup(char *str) { - int ints[2]; + int ints[3]; str = get_options(str, ARRAY_SIZE(ints), ints); |