diff options
| author | Alexander Aring <aahringo@redhat.com> | 2020-08-27 22:02:54 +0300 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2020-08-27 23:59:09 +0300 |
| commit | 7ae0451e2e6c29ff9fc17754b1129d9491177634 (patch) | |
| tree | 85eb5265eef1d434024e9859d88d400132cfaf0c | |
| parent | 948c47e9bcb6a42229cb1da1cc350c887a33ebb8 (diff) | |
| download | linux-7ae0451e2e6c29ff9fc17754b1129d9491177634.tar.xz | |
fs: dlm: use free_con to free connection
This patch use free_con() functionality to free the listen connection if
listen fails. It also fixes an issue that a freed resource is still part
of the connection_hash as hlist_del() is not called in this case. The
only difference is that free_con() handles othercon as well, but this is
never been set for the listen connection.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
| -rw-r--r-- | fs/dlm/lowcomms.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c index 794216eb728c..1bf1808bfa6b 100644 --- a/fs/dlm/lowcomms.c +++ b/fs/dlm/lowcomms.c @@ -1704,10 +1704,8 @@ int dlm_lowcomms_start(void) fail_unlisten: dlm_allow_conn = 0; con = nodeid2con(0,0); - if (con) { - close_connection(con, false, true, true); - kfree_rcu(con, rcu); - } + if (con) + free_conn(con); fail: return error; } |