|
 |
d2787b |
From 025718f1734655c411475ea338cee1659d96763e Mon Sep 17 00:00:00 2001
|
|
|
d2787b |
From: nik-redhat <nladha@redhat.com>
|
|
|
d2787b |
Date: Thu, 3 Sep 2020 15:42:45 +0530
|
|
|
d2787b |
Subject: [PATCH 595/610] glusterd: use after free (coverity issue)
|
|
|
d2787b |
|
|
|
d2787b |
Issue:
|
|
|
d2787b |
dict_unref is called on the same dict again,
|
|
|
d2787b |
in the out label of the code, which causes the
|
|
|
d2787b |
use after free issue.
|
|
|
d2787b |
|
|
|
d2787b |
Fix:
|
|
|
d2787b |
Set the dict to NULL after unref, to avoid
|
|
|
d2787b |
use after free issue.
|
|
|
d2787b |
|
|
|
d2787b |
CID: 1430127
|
|
|
d2787b |
|
|
|
d2787b |
>Updates: #1060
|
|
|
d2787b |
|
|
|
d2787b |
>Change-Id: Ide9a5cbc5f496705c671e72b0260da6d4c06f16d
|
|
|
d2787b |
>Signed-off-by: nik-redhat <nladha@redhat.com>
|
|
|
d2787b |
|
|
|
d2787b |
Upstream link: https://review.gluster.org/c/glusterfs/+/24946
|
|
|
d2787b |
BUG: 1997447
|
|
|
d2787b |
|
|
|
d2787b |
Change-Id: Id1e58cd6226b9329ad49bd5b75ee96a3a5ec5ab7
|
|
|
d2787b |
Signed-off-by: nik-redhat <nladha@redhat.com>
|
|
|
d2787b |
Reviewed-on: https://code.engineering.redhat.com/gerrit/c/rhs-glusterfs/+/280067
|
|
|
d2787b |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
d2787b |
---
|
|
|
d2787b |
xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c | 5 +++--
|
|
|
d2787b |
1 file changed, 3 insertions(+), 2 deletions(-)
|
|
|
d2787b |
|
|
|
d2787b |
diff --git a/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c b/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
|
|
|
d2787b |
index 386eed2..b0fa490 100644
|
|
|
d2787b |
--- a/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
|
|
|
d2787b |
+++ b/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
|
|
|
d2787b |
@@ -2039,8 +2039,9 @@ glusterd_update_snaps_synctask(void *opaque)
|
|
|
d2787b |
"Failed to remove snap %s", snap->snapname);
|
|
|
d2787b |
goto out;
|
|
|
d2787b |
}
|
|
|
d2787b |
- if (dict)
|
|
|
d2787b |
- dict_unref(dict);
|
|
|
d2787b |
+
|
|
|
d2787b |
+ dict_unref(dict);
|
|
|
d2787b |
+ dict = NULL;
|
|
|
d2787b |
}
|
|
|
d2787b |
snprintf(buf, sizeof(buf), "%s.accept_peer_data", prefix);
|
|
|
d2787b |
ret = dict_get_int32(peer_data, buf, &val;;
|
|
|
d2787b |
--
|
|
|
d2787b |
1.8.3.1
|
|
|
d2787b |
|