kdb: Eliminate strncpy() warnings by replacing with strscpy()
authorDaniel Thompson <daniel.thompson@linaro.org>
Thu, 13 Feb 2020 09:57:24 +0000 (09:57 +0000)
committerDaniel Thompson <daniel.thompson@linaro.org>
Wed, 1 Apr 2020 15:59:02 +0000 (16:59 +0100)
commitd228bee8201a7ea77c414f1298b2f572f42c6113
tree32d72ed695779f3ce53c5a566588857319bc81fa
parent7111951b8d4973bda27ff663f2cf18b663d15b48
kdb: Eliminate strncpy() warnings by replacing with strscpy()

Currently the code to manage the kdb history buffer uses strncpy() to
copy strings to/and from the history and exhibits the classic "but
nobody ever told me that strncpy() doesn't always terminate strings"
bug. Modern gcc compilers recognise this bug and issue a warning.

In reality these calls will only abridge the copied string if kdb_read()
has *already* overflowed the command buffer. Thus the use of counted
copies here is only used to reduce the secondary effects of a bug
elsewhere in the code.

Therefore transitioning these calls into strscpy() (without checking
the return code) is appropriate.

Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
kernel/debug/kdb/kdb_main.c