diff options
author | Eric Dumazet <edumazet@google.com> | 2018-10-04 20:41:37 -0400 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2018-10-04 20:42:26 -0400 |
commit | f74c371fe72a4f820d287db8067683fb533e4ede (patch) | |
tree | f33df632923f8e573967ea1e68106bc74a29f55c | |
parent | 684bec1092b6991ff2a7751e8a763898576eb5c2 (diff) |
Input: mousedev - add a schedule point in mousedev_write()
syzbot was able to trigger rcu stalls by calling write()
with large number of bytes.
Add a cond_resched() in the loop to avoid this.
Link: https://lkml.org/lkml/2018/8/23/1106
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reported-by: syzbot+9436b02171ac0894d33e@syzkaller.appspotmail.com
Reviewed-by: Paul E. McKenney <paulmck@linux.ibm.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
-rw-r--r-- | drivers/input/mousedev.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/input/mousedev.c b/drivers/input/mousedev.c index e08228061bcd..412fa71245af 100644 --- a/drivers/input/mousedev.c +++ b/drivers/input/mousedev.c | |||
@@ -707,6 +707,7 @@ static ssize_t mousedev_write(struct file *file, const char __user *buffer, | |||
707 | mousedev_generate_response(client, c); | 707 | mousedev_generate_response(client, c); |
708 | 708 | ||
709 | spin_unlock_irq(&client->packet_lock); | 709 | spin_unlock_irq(&client->packet_lock); |
710 | cond_resched(); | ||
710 | } | 711 | } |
711 | 712 | ||
712 | kill_fasync(&client->fasync, SIGIO, POLL_IN); | 713 | kill_fasync(&client->fasync, SIGIO, POLL_IN); |