|
 |
887953 |
From 64cfb0126d88bd6b841777fefb63e413b587a7b1 Mon Sep 17 00:00:00 2001
|
|
|
887953 |
From: N Balachandran <nbalacha@redhat.com>
|
|
|
887953 |
Date: Tue, 18 Dec 2018 14:08:04 +0530
|
|
|
887953 |
Subject: [PATCH 486/493] performance/rda: Fixed dict_t memory leak
|
|
|
887953 |
|
|
|
887953 |
Removed all references to dict_t xdata_from_req which is
|
|
|
887953 |
allocated but not used anywhere. It is also not cleaned up
|
|
|
887953 |
and hence causes a memory leak.
|
|
|
887953 |
|
|
|
887953 |
upstream patch: https://review.gluster.org/#/c/glusterfs/+/21859/
|
|
|
887953 |
|
|
|
887953 |
> Change-Id: I2edb857696191e872ad12a12efc36999626bacc7
|
|
|
887953 |
> fixes: bz#1659432
|
|
|
887953 |
> Signed-off-by: N Balachandran <nbalacha@redhat.com>
|
|
|
887953 |
|
|
|
887953 |
Change-Id: Ic07ecbefef3140aeb24a2afaf97c80ee65768e7e
|
|
|
887953 |
BUG: 1659439
|
|
|
887953 |
Signed-off-by: N Balachandran <nbalacha@redhat.com>
|
|
|
887953 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/158915
|
|
|
887953 |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
887953 |
Reviewed-by: Raghavendra Gowdappa <rgowdapp@redhat.com>
|
|
|
887953 |
---
|
|
|
887953 |
xlators/performance/readdir-ahead/src/readdir-ahead.c | 8 --------
|
|
|
887953 |
1 file changed, 8 deletions(-)
|
|
|
887953 |
|
|
|
887953 |
diff --git a/xlators/performance/readdir-ahead/src/readdir-ahead.c b/xlators/performance/readdir-ahead/src/readdir-ahead.c
|
|
|
887953 |
index 6501a6b..3e0951c 100644
|
|
|
887953 |
--- a/xlators/performance/readdir-ahead/src/readdir-ahead.c
|
|
|
887953 |
+++ b/xlators/performance/readdir-ahead/src/readdir-ahead.c
|
|
|
887953 |
@@ -505,18 +505,10 @@ rda_opendir(call_frame_t *frame, xlator_t *this, loc_t *loc, fd_t *fd,
|
|
|
887953 |
{
|
|
|
887953 |
int op_errno = 0;
|
|
|
887953 |
struct rda_local *local = NULL;
|
|
|
887953 |
- dict_t *xdata_from_req = NULL;
|
|
|
887953 |
|
|
|
887953 |
if (xdata) {
|
|
|
887953 |
- xdata_from_req = dict_new();
|
|
|
887953 |
- if (!xdata_from_req) {
|
|
|
887953 |
- op_errno = ENOMEM;
|
|
|
887953 |
- goto unwind;
|
|
|
887953 |
- }
|
|
|
887953 |
-
|
|
|
887953 |
local = mem_get0(this->local_pool);
|
|
|
887953 |
if (!local) {
|
|
|
887953 |
- dict_unref(xdata_from_req);
|
|
|
887953 |
op_errno = ENOMEM;
|
|
|
887953 |
goto unwind;
|
|
|
887953 |
}
|
|
|
887953 |
--
|
|
|
887953 |
1.8.3.1
|
|
|
887953 |
|