89 lines
3.0 KiB
Diff
89 lines
3.0 KiB
Diff
From 63aa90525f8f408526ee5e16c42dcc976245eca7 Mon Sep 17 00:00:00 2001
|
|
From: Mohit Agrawal <moagrawa@redhat.com>
|
|
Date: Thu, 22 Nov 2018 18:28:35 +0530
|
|
Subject: [PATCH 451/451] core: Resolve memory leak at the time of graph init
|
|
|
|
Problem: In the commit 751b14f2bfd40e08ad395ccd98c6eb0a41ac4e91
|
|
one code path is missed to avoid leak at the time
|
|
of calling graph init
|
|
|
|
Solution: Before destroying graph call xlator fini to avoid leak for
|
|
server-side xlators those call init during graph init
|
|
|
|
> Credit: Pranith Kumar Karampuri
|
|
> fixes: bz#1651431
|
|
|
|
> Change-Id: I6e7cff0d792ab9d954524b28667e94f2d9ec19a2
|
|
> Signed-off-by: Mohit Agrawal <moagrawal@redhat.com>
|
|
> (Cherry pick from commit 12285e76e8f93ef6f6aa2611869bd1f40955dc9e)
|
|
> (Reviewed on upstream link https://review.gluster.org/#/c/glusterfs/+/21695/)
|
|
|
|
Change-Id: Ie81635622552d43f41bbbaf810c5009a2c772a31
|
|
BUG: 1650138
|
|
Signed-off-by: Mohit Agrawal <moagrawa@redhat.com>
|
|
Reviewed-on: https://code.engineering.redhat.com/gerrit/156609
|
|
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
---
|
|
glusterfsd/src/glusterfsd.c | 36 +++++++++++++++++++++++++++++++++---
|
|
1 file changed, 33 insertions(+), 3 deletions(-)
|
|
|
|
diff --git a/glusterfsd/src/glusterfsd.c b/glusterfsd/src/glusterfsd.c
|
|
index 262a0c1..2e43cdb 100644
|
|
--- a/glusterfsd/src/glusterfsd.c
|
|
+++ b/glusterfsd/src/glusterfsd.c
|
|
@@ -2338,6 +2338,23 @@ out:
|
|
}
|
|
#endif
|
|
|
|
+int
|
|
+glusterfs_graph_fini(glusterfs_graph_t *graph)
|
|
+{
|
|
+ xlator_t *trav = NULL;
|
|
+
|
|
+ trav = graph->first;
|
|
+
|
|
+ while (trav) {
|
|
+ if (trav->init_succeeded) {
|
|
+ trav->fini(trav);
|
|
+ trav->init_succeeded = 0;
|
|
+ }
|
|
+ trav = trav->next;
|
|
+ }
|
|
+
|
|
+ return 0;
|
|
+}
|
|
|
|
int
|
|
glusterfs_process_volfp (glusterfs_ctx_t *ctx, FILE *fp)
|
|
@@ -2384,10 +2401,23 @@ out:
|
|
fclose (fp);
|
|
|
|
if (ret) {
|
|
- if (graph && (ctx && (ctx->active != graph)))
|
|
- glusterfs_graph_destroy (graph);
|
|
+ /* TODO This code makes to generic for all graphs
|
|
+ client as well as servers.For now it destroys
|
|
+ graph only for server-side xlators not for client-side
|
|
+ xlators, before destroying a graph call xlator fini for
|
|
+ xlators those call xlator_init to avoid leak
|
|
+ */
|
|
+ if (graph) {
|
|
+ xl = graph->first;
|
|
+ if ((ctx && (ctx->active != graph)) &&
|
|
+ (xl && !strcmp(xl->type, "protocol/server"))) {
|
|
+ glusterfs_graph_fini(graph);
|
|
+ glusterfs_graph_destroy(graph);
|
|
+ }
|
|
+ }
|
|
+
|
|
/* there is some error in setting up the first graph itself */
|
|
- if (!ctx->active) {
|
|
+ if (!ctx || !ctx->active) {
|
|
emancipate (ctx, ret);
|
|
cleanup_and_exit (ret);
|
|
}
|
|
--
|
|
1.8.3.1
|
|
|