diff options
| author |  Justin Stitt <justinstitt@google.com>
| 2023-08-11 21:56:15 +0000 |
|---|---|---|
| committer |  Heiko Carstens <hca@linux.ibm.com>
| 2023-08-18 15:08:12 +0200 |
| commit | cfd012107f11ec4af010f11eca341edc831abf6c (patch) | |
| tree | 23e77634348c056f784a92f53e0cb1ce836fe7f9 /arch/s390/kernel/ipl.c | |
| parent | s390/ipl: fix virtual vs physical address confusion (diff) | |
| download | linux-cfd012107f11ec4af010f11eca341edc831abf6c.tar.gz linux-cfd012107f11ec4af010f11eca341edc831abf6c.tar.bz2 linux-cfd012107f11ec4af010f11eca341edc831abf6c.zip | |
s390/ipl: refactor deprecated strncpy
`strncpy` is deprecated for use on NUL-terminated destination strings [1].
Use `strscpy` which has the same behavior as `strncpy` here with the
extra safeguard of guaranteeing NUL-termination of destination
strings. In it's current form, this may result in silent truncation
if the src string has the same size as the destination string.
[hca@linux.ibm.com: use strscpy() instead of strscpy_pad()]
Link: www.kernel.org/doc/html/latest/process/deprecated.html#strncpy-on-nul-terminated-strings[1]
Link: https://github.com/KSPP/linux/issues/90
Cc: linux-hardening@vger.kernel.org
Signed-off-by: Justin Stitt <justinstitt@google.com>
Link: https://lore.kernel.org/r/20230811-arch-s390-kernel-v1-1-7edbeeab3809@google.com
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
Diffstat (limited to 'arch/s390/kernel/ipl.c')
| -rw-r--r-- | arch/s390/kernel/ipl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/s390/kernel/ipl.c b/arch/s390/kernel/ipl.c index 7f3a84e414a4..05e51666db03 100644 --- a/arch/s390/kernel/ipl.c +++ b/arch/s390/kernel/ipl.c @@ -266,7 +266,7 @@ static ssize_t sys_##_prefix##_##_name##_store(struct kobject *kobj, \ struct kobj_attribute *attr, \ const char *buf, size_t len) \ { \ - strncpy(_value, buf, sizeof(_value) - 1); \ + strscpy(_value, buf, sizeof(_value)); \ strim(_value); \ return len; \ } \ |