diff options
author | Tim Gardner <[email protected]> | 2021-09-27 09:45:02 -0600 |
---|---|---|
committer | Helge Deller <[email protected]> | 2022-01-29 22:24:25 +0100 |
commit | 37a1a2e6eeeb101285cd34e12e48a881524701aa (patch) | |
tree | 0f05978f72370bdd2eaf032311c02338daf6f600 /lib/test_overflow.c | |
parent | 1e3c3e6193d45afd5879da5a82b2c92997b0ebe9 (diff) |
video: fbdev: nvidiafb: Use strscpy() to prevent buffer overflow
Coverity complains of a possible buffer overflow. However,
given the 'static' scope of nvidia_setup_i2c_bus() it looks
like that can't happen after examiniing the call sites.
CID 19036 (#1 of 1): Copy into fixed size buffer (STRING_OVERFLOW)
1. fixed_size_dest: You might overrun the 48-character fixed-size string
chan->adapter.name by copying name without checking the length.
2. parameter_as_source: Note: This defect has an elevated risk because the
source argument is a parameter of the current function.
89 strcpy(chan->adapter.name, name);
Fix this warning by using strscpy() which will silence the warning and
prevent any future buffer overflows should the names used to identify the
channel become much longer.
Cc: Antonino Daplas <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Signed-off-by: Tim Gardner <[email protected]>
Signed-off-by: Helge Deller <[email protected]>
Diffstat (limited to 'lib/test_overflow.c')
0 files changed, 0 insertions, 0 deletions