diff options
| author | Dmitry Torokhov <dtor@insightbb.com> | 2006-07-06 23:54:48 -0400 |
|---|---|---|
| committer | Dmitry Torokhov <dtor@insightbb.com> | 2006-07-06 23:54:48 -0400 |
| commit | 95349fe8144b7d18f04bdca1c2d3fb85789de4fb (patch) | |
| tree | 90c603c34b9ae521342467b29ba0cdeba60cef5f /drivers/input | |
| parent | 699756199d65700e8deed59ae250439ca8684686 (diff) | |
| download | op-kernel-dev-95349fe8144b7d18f04bdca1c2d3fb85789de4fb.zip op-kernel-dev-95349fe8144b7d18f04bdca1c2d3fb85789de4fb.tar.gz | |
Input: libps2 - warn instead of oopsing when passed bad arguments
This is more user-friendly and also fixes Coverity #id 249
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Diffstat (limited to 'drivers/input')
| -rw-r--r-- | drivers/input/serio/libps2.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/input/serio/libps2.c b/drivers/input/serio/libps2.c index 61a6f97..ed202f2 100644 --- a/drivers/input/serio/libps2.c +++ b/drivers/input/serio/libps2.c @@ -177,6 +177,11 @@ int ps2_command(struct ps2dev *ps2dev, unsigned char *param, int command) return -1; } + if (send && !param) { + WARN_ON(1); + return -1; + } + mutex_lock_nested(&ps2dev->cmd_mutex, SINGLE_DEPTH_NESTING); serio_pause_rx(ps2dev->serio); |
