diff options
author | Sean McBride <sean@rogue-research.com> | 2022-01-28 16:34:57 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-01-28 16:34:57 (GMT) |
commit | b5eed1b56324fc07154c2e2d8251d2b87505ca23 (patch) | |
tree | b3e3840a886966ae07e50a69ab7492ebe4df876c /src/H5FDmulti.c | |
parent | 44de59b642c4d5ccf505a1072d10b63dc9fe0628 (diff) | |
download | hdf5-b5eed1b56324fc07154c2e2d8251d2b87505ca23.zip hdf5-b5eed1b56324fc07154c2e2d8251d2b87505ca23.tar.gz hdf5-b5eed1b56324fc07154c2e2d8251d2b87505ca23.tar.bz2 |
Replaced several uses of sprintf with safer snprintf (#1383)
* Replaced several uses of sprintf with safer snprintf
* Committing clang-format changes
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Diffstat (limited to 'src/H5FDmulti.c')
-rw-r--r-- | src/H5FDmulti.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/H5FDmulti.c b/src/H5FDmulti.c index 3dcfa37..d9a6ce9 100644 --- a/src/H5FDmulti.c +++ b/src/H5FDmulti.c @@ -517,7 +517,7 @@ H5FD_split_populate_config(const char *meta_ext, hid_t meta_plist_id, const char meta_name_g[sizeof(meta_name_g) - 1] = '\0'; } else - sprintf(meta_name_g, "%%s%s", meta_ext); + snprintf(meta_name_g, sizeof(meta_name_g), "%%s%s", meta_ext); } else { strncpy(meta_name_g, "%s.meta", sizeof(meta_name_g)); @@ -535,7 +535,7 @@ H5FD_split_populate_config(const char *meta_ext, hid_t meta_plist_id, const char raw_name_g[sizeof(raw_name_g) - 1] = '\0'; } else - sprintf(raw_name_g, "%%s%s", raw_ext); + snprintf(raw_name_g, sizeof(raw_name_g), "%%s%s", raw_ext); } else { strncpy(raw_name_g, "%s.raw", sizeof(raw_name_g)); @@ -634,7 +634,7 @@ H5FD_multi_populate_config(const H5FD_mem_t *memb_map, const hid_t *memb_fapl, c if (!memb_name) { assert(strlen(letters) == H5FD_MEM_NTYPES); for (mt = H5FD_MEM_DEFAULT; mt < H5FD_MEM_NTYPES; mt = (H5FD_mem_t)(mt + 1)) { - sprintf(_memb_name_g[mt], "%%s-%c.h5", letters[mt]); + snprintf(_memb_name_g[mt], 16, "%%s-%c.h5", letters[mt]); _memb_name_ptrs[mt] = _memb_name_g[mt]; } memb_name = _memb_name_ptrs; |