diff options
author | Justin Stitt <[email protected]> | 2024-03-20 23:22:02 +0000 |
---|---|---|
committer | Helge Deller <[email protected]> | 2024-04-25 11:58:59 +0200 |
commit | 5317797e9cd07ff48132a36d545c25c1687ee676 (patch) | |
tree | ff65b249ebba8c72adbec556d529e3f688b5f4db /lib/memory-notifier-error-inject.c | |
parent | 91bcea421ecece579eb283eb811d9eb197693772 (diff) |
video: hdmi: prefer length specifier in format over string copying
strncpy() is deprecated for use on NUL-terminated destination strings
[1] and as such we should prefer more robust and less ambiguous string
interfaces.
It looks like the main use of strncpy() here is to limit the amount of
bytes printed from hdmi_log() by using a tmp buffer and limiting the
number of bytes copied. Really, we should use the %.<len>s format
qualifier to achieve this.
Link: https://www.kernel.org/doc/html/latest/process/deprecated.html#strncpy-on-nul-terminated-strings [1]
Link: https://manpages.debian.org/testing/linux-manual-4.8/strscpy.9.en.html
Link: https://github.com/KSPP/linux/issues/90
Cc: [email protected]
Signed-off-by: Justin Stitt <[email protected]>
Reviewed-by: Kees Cook <[email protected]>
Signed-off-by: Helge Deller <[email protected]>
Diffstat (limited to 'lib/memory-notifier-error-inject.c')
0 files changed, 0 insertions, 0 deletions