diff options
author | Vasily Gorbik <gor@linux.ibm.com> | 2018-06-18 10:50:42 -0400 |
---|---|---|
committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2018-07-02 05:24:53 -0400 |
commit | f6ea4d25e95972f482e14cdcefaa0789e6158d35 (patch) | |
tree | 9692fea7a08d23e8b66291ef55356ce37abd6578 /arch/s390/appldata | |
parent | 4e0f5e916f7f4623b312e4fcf9f9fe45c4bea290 (diff) |
s390: fix gcc 8 stringop-truncation warnings in proc handlers
arch/s390/kernel/topology.c:591:3: warning: 'strncpy' output truncated
before terminating nul copying 2 bytes from a string of the same length
[-Wstringop-truncation]
strncpy(buf, topology_is_enabled() ? "1\n" : "0\n", ARRAY_SIZE(buf));
arch/s390/appldata/appldata_base.c:326:3: warning: 'strncpy' output truncated
before terminating nul copying 2 bytes from a string of the same length
[-Wstringop-truncation]
strncpy(buf, ops->active ? "1\n" : "0\n", ARRAY_SIZE(buf));
arch/s390/appldata/appldata_base.c:217:3: warning: 'strncpy' output truncated
before terminating nul copying 2 bytes from a string of the same length
[-Wstringop-truncation]
strncpy(buf, appldata_timer_active ? "1\n" : "0\n", ARRAY_SIZE(buf));
To avoid the warning, just reuse memcpy.
Reviewed-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'arch/s390/appldata')
-rw-r--r-- | arch/s390/appldata/appldata_base.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/arch/s390/appldata/appldata_base.c b/arch/s390/appldata/appldata_base.c index ee6a9c387c87..25c1399b0787 100644 --- a/arch/s390/appldata/appldata_base.c +++ b/arch/s390/appldata/appldata_base.c | |||
@@ -214,8 +214,7 @@ appldata_timer_handler(struct ctl_table *ctl, int write, | |||
214 | return 0; | 214 | return 0; |
215 | } | 215 | } |
216 | if (!write) { | 216 | if (!write) { |
217 | strncpy(buf, appldata_timer_active ? "1\n" : "0\n", | 217 | memcpy(buf, appldata_timer_active ? "1\n" : "0\n", ARRAY_SIZE(buf)); |
218 | ARRAY_SIZE(buf)); | ||
219 | len = strnlen(buf, ARRAY_SIZE(buf)); | 218 | len = strnlen(buf, ARRAY_SIZE(buf)); |
220 | if (len > *lenp) | 219 | if (len > *lenp) |
221 | len = *lenp; | 220 | len = *lenp; |
@@ -323,7 +322,7 @@ appldata_generic_handler(struct ctl_table *ctl, int write, | |||
323 | return 0; | 322 | return 0; |
324 | } | 323 | } |
325 | if (!write) { | 324 | if (!write) { |
326 | strncpy(buf, ops->active ? "1\n" : "0\n", ARRAY_SIZE(buf)); | 325 | memcpy(buf, ops->active ? "1\n" : "0\n", ARRAY_SIZE(buf)); |
327 | len = strnlen(buf, ARRAY_SIZE(buf)); | 326 | len = strnlen(buf, ARRAY_SIZE(buf)); |
328 | if (len > *lenp) | 327 | if (len > *lenp) |
329 | len = *lenp; | 328 | len = *lenp; |