diff options
author | Dmitry Antipov <dmantipov@yandex.ru> | 2020-05-06 08:29:48 +0300 |
---|---|---|
committer | Xavi Hernandez <xhernandez@redhat.com> | 2020-05-18 07:17:40 +0000 |
commit | 8e550380f7098275452fc26c46489739aa8bdae6 (patch) | |
tree | d608c63cbca44d42ca627e644b0403568a314be3 | |
parent | dc1d549d0e776ecd4c8381577eeed990250a7f6a (diff) | |
download | glusterfs-8e550380f7098275452fc26c46489739aa8bdae6.tar.gz glusterfs-8e550380f7098275452fc26c46489739aa8bdae6.tar.xz glusterfs-8e550380f7098275452fc26c46489739aa8bdae6.zip |
features/bit-rot: invalid snprintf() buffer size
Found with clang-10 -Wfortify-source:
bit-rot-scrub.c:1802:15: warning: 'snprintf' size argument is too large;
destination buffer has size 32, but size argument is 4096 [-Wfortify-source]
len = snprintf(key, PATH_MAX, "quarantine-%d", j);
^
bit-rot-scrub.c:1813:9: warning: 'snprintf' size argument is too large;
destination buffer has size 32, but size argument is 4096 [-Wfortify-source]
snprintf(main_key, PATH_MAX, "quarantine-%d", tmp_count);
Change-Id: I9b9c09ef2223ed181d81215154345de976b82f13
Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru>
Fixes: #1221
-rw-r--r-- | xlators/features/bit-rot/src/bitd/bit-rot-scrub.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xlators/features/bit-rot/src/bitd/bit-rot-scrub.c b/xlators/features/bit-rot/src/bitd/bit-rot-scrub.c index d20ecc7cdb..10c0478f2e 100644 --- a/xlators/features/bit-rot/src/bitd/bit-rot-scrub.c +++ b/xlators/features/bit-rot/src/bitd/bit-rot-scrub.c @@ -1799,7 +1799,7 @@ br_collect_bad_objects_of_child(xlator_t *this, br_child_t *child, dict_t *dict, tmp_count = total_count; for (j = 0; j < count; j++) { - len = snprintf(key, PATH_MAX, "quarantine-%d", j); + len = snprintf(key, sizeof(key), "quarantine-%d", j); ret = dict_get_strn(child_dict, key, len, &entry); if (ret) continue; @@ -1810,7 +1810,7 @@ br_collect_bad_objects_of_child(xlator_t *this, br_child_t *child, dict_t *dict, if ((len < 0) || (len >= PATH_MAX)) { continue; } - snprintf(main_key, PATH_MAX, "quarantine-%d", tmp_count); + snprintf(main_key, sizeof(main_key), "quarantine-%d", tmp_count); ret = dict_set_dynstr_with_alloc(dict, main_key, tmp); if (!ret) |