vt: keyboard, simplify vt_kdgkbsent
authorJiri Slaby <jslaby@suse.cz>
Mon, 19 Oct 2020 08:55:16 +0000 (10:55 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 28 Oct 2020 12:41:02 +0000 (13:41 +0100)
commit6ca03f90527e499dd5e32d6522909e2ad390896b
tree8e8e540c9d6383a80b8ebfca3ad2fe07af47bf83
parentc97f2a6fb3dfbfbbc88edc8ea62ef2b944e18849
vt: keyboard, simplify vt_kdgkbsent

Use 'strlen' of the string, add one for NUL terminator and simply do
'copy_to_user' instead of the explicit 'for' loop. This makes the
KDGKBSENT case more compact.

The only thing we need to take care about is NULL 'func_table[i]'. Use
an empty string in that case.

The original check for overflow could never trigger as the func_buf
strings are always shorter or equal to 'struct kbsentry's.

Cc: <stable@vger.kernel.org>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Link: https://lore.kernel.org/r/20201019085517.10176-1-jslaby@suse.cz
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/vt/keyboard.c