[Skiboot] [PATCH] libffs: fix gcc8 stringop-truncation warnings
Brad Bishop
bradleyb at fuzziesquirrel.com
Thu Nov 22 01:27:53 AEDT 2018
Copy the null byte too in this copy operation to avoid the following
warning:
| libflash/libffs.c: In function 'ffs_part_info':
| libflash/libffs.c:525:3: error: 'strncpy' output may be truncated copying 15 bytes from a string of length 15 [-Werror=stringop-truncation]
| strncpy(n, ent->name, FFS_PART_NAME_MAX);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Signed-off-by: Brad Bishop <bradleyb at fuzziesquirrel.com>
---
libflash/libffs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libflash/libffs.c b/libflash/libffs.c
index 221c2b02..816153d3 100644
--- a/libflash/libffs.c
+++ b/libflash/libffs.c
@@ -522,7 +522,7 @@ int ffs_part_info(struct ffs_handle *ffs, uint32_t part_idx,
n = calloc(1, FFS_PART_NAME_MAX + 1);
if (!n)
return FLASH_ERR_MALLOC_FAILED;
- strncpy(n, ent->name, FFS_PART_NAME_MAX);
+ strncpy(n, ent->name, FFS_PART_NAME_MAX +1);
*name = n;
}
return 0;
--
2.19.1
More information about the Skiboot
mailing list