|
 |
d1681e |
From 09c73d2c7793d2316c62837a80b41904a2a5b15f Mon Sep 17 00:00:00 2001
|
|
|
d1681e |
From: Xavier Hernandez <jahernan@redhat.com>
|
|
|
d1681e |
Date: Mon, 20 Nov 2017 10:51:09 +0100
|
|
|
d1681e |
Subject: [PATCH 103/128] features/locks: Fix memory leaks
|
|
|
d1681e |
|
|
|
d1681e |
> Upstream patch: https://review.gluster.org/18812
|
|
|
d1681e |
|
|
|
d1681e |
Change-Id: Ic1d2e17a7d14389b6734d1b88bd28c0a2907bbd6
|
|
|
d1681e |
BUG: 1507361
|
|
|
d1681e |
Signed-off-by: Xavier Hernandez <jahernan@redhat.com>
|
|
|
d1681e |
Reviewed-on: https://code.engineering.redhat.com/gerrit/125955
|
|
|
d1681e |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
d1681e |
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
|
|
|
d1681e |
---
|
|
|
d1681e |
xlators/features/locks/src/clear.c | 2 +-
|
|
|
d1681e |
xlators/features/locks/src/common.c | 1 +
|
|
|
d1681e |
xlators/features/locks/src/entrylk.c | 3 ++-
|
|
|
d1681e |
xlators/features/locks/src/inodelk.c | 7 ++++---
|
|
|
d1681e |
xlators/features/locks/src/posix.c | 3 +++
|
|
|
d1681e |
5 files changed, 11 insertions(+), 5 deletions(-)
|
|
|
d1681e |
|
|
|
d1681e |
diff --git a/xlators/features/locks/src/clear.c b/xlators/features/locks/src/clear.c
|
|
|
d1681e |
index d7c210f..22c03b5 100644
|
|
|
d1681e |
--- a/xlators/features/locks/src/clear.c
|
|
|
d1681e |
+++ b/xlators/features/locks/src/clear.c
|
|
|
d1681e |
@@ -184,7 +184,7 @@ clrlk_clear_posixlk (xlator_t *this, pl_inode_t *pl_inode, clrlk_args *args,
|
|
|
d1681e |
} else {
|
|
|
d1681e |
gcount++;
|
|
|
d1681e |
}
|
|
|
d1681e |
- GF_FREE (plock);
|
|
|
d1681e |
+ __destroy_lock(plock);
|
|
|
d1681e |
}
|
|
|
d1681e |
}
|
|
|
d1681e |
pthread_mutex_unlock (&pl_inode->mutex);
|
|
|
d1681e |
diff --git a/xlators/features/locks/src/common.c b/xlators/features/locks/src/common.c
|
|
|
d1681e |
index 4b63c43..7311126 100644
|
|
|
d1681e |
--- a/xlators/features/locks/src/common.c
|
|
|
d1681e |
+++ b/xlators/features/locks/src/common.c
|
|
|
d1681e |
@@ -438,6 +438,7 @@ pl_inode_get (xlator_t *this, inode_t *inode)
|
|
|
d1681e |
|
|
|
d1681e |
ret = __inode_ctx_put (inode, this, (uint64_t)(long)(pl_inode));
|
|
|
d1681e |
if (ret) {
|
|
|
d1681e |
+ pthread_mutex_destroy (&pl_inode->mutex);
|
|
|
d1681e |
GF_FREE (pl_inode);
|
|
|
d1681e |
pl_inode = NULL;
|
|
|
d1681e |
goto unlock;
|
|
|
d1681e |
diff --git a/xlators/features/locks/src/entrylk.c b/xlators/features/locks/src/entrylk.c
|
|
|
d1681e |
index 4231d76..8e9008b 100644
|
|
|
d1681e |
--- a/xlators/features/locks/src/entrylk.c
|
|
|
d1681e |
+++ b/xlators/features/locks/src/entrylk.c
|
|
|
d1681e |
@@ -784,7 +784,7 @@ pl_common_entrylk (call_frame_t *frame, xlator_t *this,
|
|
|
d1681e |
break;
|
|
|
d1681e |
|
|
|
d1681e |
default:
|
|
|
d1681e |
- inode_unref (pinode->inode);
|
|
|
d1681e |
+ need_inode_unref = _gf_true;
|
|
|
d1681e |
gf_log (this->name, GF_LOG_ERROR,
|
|
|
d1681e |
"Unexpected case in entrylk (cmd=%d). Please file"
|
|
|
d1681e |
"a bug report at http://bugs.gluster.com", cmd);
|
|
|
d1681e |
@@ -960,6 +960,7 @@ pl_entrylk_client_cleanup (xlator_t *this, pl_ctx_t *ctx)
|
|
|
d1681e |
__pl_entrylk_unref (l);
|
|
|
d1681e |
}
|
|
|
d1681e |
pthread_mutex_unlock (&pinode->mutex);
|
|
|
d1681e |
+
|
|
|
d1681e |
inode_unref (pinode->inode);
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
diff --git a/xlators/features/locks/src/inodelk.c b/xlators/features/locks/src/inodelk.c
|
|
|
d1681e |
index e1702c7..4aebac6 100644
|
|
|
d1681e |
--- a/xlators/features/locks/src/inodelk.c
|
|
|
d1681e |
+++ b/xlators/features/locks/src/inodelk.c
|
|
|
d1681e |
@@ -721,9 +721,6 @@ out:
|
|
|
d1681e |
if (ctx)
|
|
|
d1681e |
pthread_mutex_unlock (&ctx->lock);
|
|
|
d1681e |
|
|
|
d1681e |
- if (need_inode_unref)
|
|
|
d1681e |
- inode_unref (pl_inode->inode);
|
|
|
d1681e |
-
|
|
|
d1681e |
/* The following (extra) unref corresponds to the ref that
|
|
|
d1681e |
* was done at the time the lock was granted.
|
|
|
d1681e |
*/
|
|
|
d1681e |
@@ -732,6 +729,10 @@ out:
|
|
|
d1681e |
grant_blocked_inode_locks (this, pl_inode, dom);
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
+ if (need_inode_unref) {
|
|
|
d1681e |
+ inode_unref (pl_inode->inode);
|
|
|
d1681e |
+ }
|
|
|
d1681e |
+
|
|
|
d1681e |
return ret;
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
diff --git a/xlators/features/locks/src/posix.c b/xlators/features/locks/src/posix.c
|
|
|
d1681e |
index ff9a3da..a158227 100644
|
|
|
d1681e |
--- a/xlators/features/locks/src/posix.c
|
|
|
d1681e |
+++ b/xlators/features/locks/src/posix.c
|
|
|
d1681e |
@@ -2472,6 +2472,7 @@ pl_forget (xlator_t *this,
|
|
|
d1681e |
list) {
|
|
|
d1681e |
|
|
|
d1681e |
list_del (&rw_req->list);
|
|
|
d1681e |
+ call_stub_destroy(rw_req->stub);
|
|
|
d1681e |
GF_FREE (rw_req);
|
|
|
d1681e |
}
|
|
|
d1681e |
}
|
|
|
d1681e |
@@ -2555,6 +2556,8 @@ pl_forget (xlator_t *this,
|
|
|
d1681e |
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
+ pthread_mutex_destroy(&pl_inode->mutex);
|
|
|
d1681e |
+
|
|
|
d1681e |
GF_FREE (pl_inode);
|
|
|
d1681e |
|
|
|
d1681e |
return 0;
|
|
|
d1681e |
--
|
|
|
d1681e |
1.8.3.1
|
|
|
d1681e |
|