From 4ff96e11ea80d9a0d9dec2031c9cdd99d2afdc61 Mon Sep 17 00:00:00 2001

From: Sanju Rakonde <srakonde@redhat.com>

Date: Mon, 30 Apr 2018 11:12:34 +0530

Subject: [PATCH 246/260] glusterd: Fix for memory leak in get-state detail

>Fixes: bz#1573066

>Change-Id: I76fe3bdde7351736b32eb3d6c4cc5f8f276257ed

>Signed-off-by: Sanju Rakonde <srakonde@redhat.com>

upstream patch: https://review.gluster.org/#/c/19948/

BUG: 1567899

Change-Id: I76fe3bdde7351736b32eb3d6c4cc5f8f276257ed

Signed-off-by: Sanju Rakonde <srakonde@redhat.com>

Reviewed-on: https://code.engineering.redhat.com/gerrit/137469

Tested-by: RHGS Build Bot <nigelb@redhat.com>

Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>

---

 xlators/mgmt/glusterd/src/glusterd-handler.c | 9 ++++++++-

 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c

index 30adb99..962c87e 100644

--- a/xlators/mgmt/glusterd/src/glusterd-handler.c

+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c

@@ -5409,8 +5409,15 @@ out:

         if (pending_node)

                 GF_FREE (pending_node);

-        if (brick_req)

+        if (brick_req) {

+                if (brick_req->input.input_val)

+                        GF_FREE (brick_req->input.input_val);

                 GF_FREE (brick_req);

+        }

+        if (args.dict)

+                dict_unref (args.dict);

+        if (args.errstr)

+                GF_FREE (args.errstr);

         return ret;

 }

-- 

1.8.3.1