59 lines
2.0 KiB
Diff
59 lines
2.0 KiB
Diff
From 76c9faf5c750428e5eb69462b82ee0c12cbdabc0 Mon Sep 17 00:00:00 2001
|
|
From: nik-redhat <nladha@redhat.com>
|
|
Date: Fri, 25 Sep 2020 18:39:51 +0530
|
|
Subject: [PATCH 593/610] dht: explicit null dereference
|
|
|
|
Added a null check for uuid_list_copy, to avoid
|
|
null dereference in strtok_r() in case of strdup()
|
|
failure.
|
|
|
|
CID: 1325612
|
|
CID: 1274223
|
|
|
|
>Updates: #1060
|
|
|
|
>Change-Id: I641a5068cd76d7b2ed92eccf39e7f97d6f7b2480
|
|
>Signed-off-by: nik-redhat <nladha@redhat.com>
|
|
|
|
Upstream link: https://review.gluster.org/c/glusterfs/+/25046
|
|
BUG: 1997447
|
|
|
|
Change-Id: I576b4ce610948bdb84eb30377a684c54df718bdc
|
|
Signed-off-by: nik-redhat <nladha@redhat.com>
|
|
Reviewed-on: https://code.engineering.redhat.com/gerrit/c/rhs-glusterfs/+/280063
|
|
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
---
|
|
xlators/cluster/dht/src/dht-common.c | 2 ++
|
|
xlators/cluster/dht/src/dht-shared.c | 2 ++
|
|
2 files changed, 4 insertions(+)
|
|
|
|
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c
|
|
index edfc6e7..e6a16ff 100644
|
|
--- a/xlators/cluster/dht/src/dht-common.c
|
|
+++ b/xlators/cluster/dht/src/dht-common.c
|
|
@@ -4296,6 +4296,8 @@ dht_find_local_subvol_cbk(call_frame_t *frame, void *cookie, xlator_t *this,
|
|
index = conf->local_subvols_cnt;
|
|
|
|
uuid_list_copy = gf_strdup(uuid_list);
|
|
+ if (!uuid_list_copy)
|
|
+ goto unlock;
|
|
|
|
for (uuid_str = strtok_r(uuid_list, " ", &saveptr); uuid_str;
|
|
uuid_str = next_uuid_str) {
|
|
diff --git a/xlators/cluster/dht/src/dht-shared.c b/xlators/cluster/dht/src/dht-shared.c
|
|
index 58e3339..cca272a 100644
|
|
--- a/xlators/cluster/dht/src/dht-shared.c
|
|
+++ b/xlators/cluster/dht/src/dht-shared.c
|
|
@@ -567,6 +567,8 @@ gf_defrag_pattern_list_fill(xlator_t *this, gf_defrag_info_t *defrag,
|
|
pattern_str = strtok_r(data, ",", &tmp_str);
|
|
while (pattern_str) {
|
|
dup_str = gf_strdup(pattern_str);
|
|
+ if (!dup_str)
|
|
+ goto out;
|
|
pattern_list = GF_CALLOC(1, sizeof(gf_defrag_pattern_list_t), 1);
|
|
if (!pattern_list) {
|
|
goto out;
|
|
--
|
|
1.8.3.1
|
|
|