|
 |
f338ef |
From a7a7d497af4230430f8a0cc54d8b49cfea260039 Mon Sep 17 00:00:00 2001
|
|
|
f338ef |
From: Xavi Hernandez <xhernandez@redhat.com>
|
|
|
f338ef |
Date: Tue, 25 Jun 2019 18:00:06 +0200
|
|
|
f338ef |
Subject: [PATCH 207/221] glusterd: fix use-after-free of a dict_t
|
|
|
f338ef |
|
|
|
f338ef |
A dict was passed to a function that calls dict_unref() without taking
|
|
|
f338ef |
any additional reference. Given that the same dict is also used after
|
|
|
f338ef |
the function returns, this was causing a use-after-free situation.
|
|
|
f338ef |
|
|
|
f338ef |
To fix the issue, we simply take an additional reference before calling
|
|
|
f338ef |
the function.
|
|
|
f338ef |
|
|
|
f338ef |
Upstream patch:
|
|
|
f338ef |
> BUG: 1723890
|
|
|
f338ef |
> Upstream patch link: https://review.gluster.org/c/glusterfs/+/22943
|
|
|
f338ef |
> Change-Id: I98c6b76b08fe3fa6224edf281a26e9ba1ffe3017
|
|
|
f338ef |
> Signed-off-by: Xavi Hernandez <xhernandez@redhat.com>
|
|
|
f338ef |
|
|
|
f338ef |
Change-Id: I98c6b76b08fe3fa6224edf281a26e9ba1ffe3017
|
|
|
f338ef |
Updates: bz#1722801
|
|
|
f338ef |
Signed-off-by: Xavi Hernandez <xhernandez@redhat.com>
|
|
|
f338ef |
Reviewed-on: https://code.engineering.redhat.com/gerrit/174656
|
|
|
f338ef |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
f338ef |
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
|
|
|
f338ef |
---
|
|
|
f338ef |
xlators/mgmt/glusterd/src/glusterd-utils.c | 2 +-
|
|
|
f338ef |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
f338ef |
|
|
|
f338ef |
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c
|
|
|
f338ef |
index c6e9bb0..4c487d0 100644
|
|
|
f338ef |
--- a/xlators/mgmt/glusterd/src/glusterd-utils.c
|
|
|
f338ef |
+++ b/xlators/mgmt/glusterd/src/glusterd-utils.c
|
|
|
f338ef |
@@ -3697,7 +3697,7 @@ glusterd_add_volumes_to_export_dict(dict_t *peer_data, char **buf,
|
|
|
f338ef |
if (totthread) {
|
|
|
f338ef |
gf_log(this->name, GF_LOG_INFO,
|
|
|
f338ef |
"Finished merger of all dictionraies into single one");
|
|
|
f338ef |
- dict_arr[totthread++] = peer_data;
|
|
|
f338ef |
+ dict_arr[totthread++] = dict_ref(peer_data);
|
|
|
f338ef |
ret = glusterd_dict_arr_serialize(dict_arr, totthread, buf, length);
|
|
|
f338ef |
gf_log(this->name, GF_LOG_INFO,
|
|
|
f338ef |
"Serialize dictionary data return is %d", ret);
|
|
|
f338ef |
--
|
|
|
f338ef |
1.8.3.1
|
|
|
f338ef |
|