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/H5VLpassthru.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/H5VLpassthru.c')
| -rw-r--r-- | src/H5VLpassthru.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/src/H5VLpassthru.c b/src/H5VLpassthru.c index 6b4c59f..6eda875 100644 --- a/src/H5VLpassthru.c +++ b/src/H5VLpassthru.c @@ -641,16 +641,13 @@ H5VL_pass_through_info_to_str(const void *_info, char **str) under_vol_str_len = strlen(under_vol_string); /* Allocate space for our info */ - *str = (char *)H5allocate_memory(32 + under_vol_str_len, (hbool_t)0); + size_t strSize = 32 + under_vol_str_len; + *str = (char *)H5allocate_memory(strSize, (hbool_t)0); assert(*str); - /* Encode our info - * Normally we'd use snprintf() here for a little extra safety, but that - * call had problems on Windows until recently. So, to be as platform-independent - * as we can, we're using sprintf() instead. - */ - sprintf(*str, "under_vol=%u;under_info={%s}", (unsigned)under_value, - (under_vol_string ? under_vol_string : "")); + /* Encode our info */ + snprintf(*str, strSize, "under_vol=%u;under_info={%s}", (unsigned)under_value, + (under_vol_string ? under_vol_string : "")); return 0; } /* end H5VL_pass_through_info_to_str() */ |