diff options
| author |   Alexander Aring <aahringo@redhat.com> | 2020-08-27 15:02:54 -0400 |
|---|---|---|
| committer |   David Teigland <teigland@redhat.com> | 2020-08-27 15:59:09 -0500 |
| commit | 7ae0451e2e6c29ff9fc17754b1129d9491177634 (patch) | |
| tree | 85eb5265eef1d434024e9859d88d400132cfaf0c /fs/dlm | |
| parent | fs: dlm: handle possible othercon writequeues (diff) | |
| download | linux-dev-7ae0451e2e6c29ff9fc17754b1129d9491177634.tar.xz linux-dev-7ae0451e2e6c29ff9fc17754b1129d9491177634.zip | |
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>
Diffstat (limited to 'fs/dlm')
| -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; } |