diff options
author | Patrick Caulfield <pcaulfie@redhat.com> | 2007-07-11 08:39:43 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2007-08-14 05:28:05 -0400 |
commit | 25720c2d73058f4f929f16093f60817ed52a285c (patch) | |
tree | 09b14797656f31596c3b061608eeba7b8ded3701 /fs/dlm/lowcomms.c | |
parent | 39d3520c92cf7a28c07229ca00cc35a1e8026c77 (diff) |
[DLM] Clear othercon pointers when a connection is closed
This patch clears the othercon pointer and frees the memory when a connnection
is closed. This could cause a small memory leak when nodes leave the cluster.
Signed-Off-By: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/dlm/lowcomms.c')
-rw-r--r-- | fs/dlm/lowcomms.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c index dd362739d291..d15fd5f9f5c5 100644 --- a/fs/dlm/lowcomms.c +++ b/fs/dlm/lowcomms.c | |||
@@ -327,6 +327,8 @@ static void close_connection(struct connection *con, bool and_other) | |||
327 | if (con->othercon && and_other) { | 327 | if (con->othercon && and_other) { |
328 | /* Will only re-enter once. */ | 328 | /* Will only re-enter once. */ |
329 | close_connection(con->othercon, false); | 329 | close_connection(con->othercon, false); |
330 | kmem_cache_free(con_cache, con->othercon); | ||
331 | con->othercon = NULL; | ||
330 | } | 332 | } |
331 | if (con->rx_page) { | 333 | if (con->rx_page) { |
332 | __free_page(con->rx_page); | 334 | __free_page(con->rx_page); |