diff options
| author | Wen Gu <guwen@linux.alibaba.com> | 2022-01-13 11:36:41 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2022-01-13 16:14:53 +0300 |
| commit | ea89c6c0983c39702a4a52ccaa4702e0cb71179b (patch) | |
| tree | 9e8c99b82d0eeaee64d464df07eacc135608cfcf /net/smc/smc_cdc.c | |
| parent | 91341fa0003befd097e190ec2a4bf63ad957c49a (diff) | |
| download | linux-ea89c6c0983c39702a4a52ccaa4702e0cb71179b.tar.xz | |
net/smc: Introduce a new conn->lgr validity check helper
It is no longer suitable to identify whether a smc connection
is registered in a link group through checking if conn->lgr
is NULL, because conn->lgr won't be reset even the connection
is unregistered from a link group.
So this patch introduces a new helper smc_conn_lgr_valid() and
replaces all the check of conn->lgr in original implementation
with the new helper to judge if conn->lgr is valid to use.
Signed-off-by: Wen Gu <guwen@linux.alibaba.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/smc/smc_cdc.c')
| -rw-r--r-- | net/smc/smc_cdc.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/smc/smc_cdc.c b/net/smc/smc_cdc.c index 84c8a4374fdd..9d5a97168969 100644 --- a/net/smc/smc_cdc.c +++ b/net/smc/smc_cdc.c @@ -197,7 +197,8 @@ int smc_cdc_get_slot_and_msg_send(struct smc_connection *conn) { int rc; - if (!conn->lgr || (conn->lgr->is_smcd && conn->lgr->peer_shutdown)) + if (!smc_conn_lgr_valid(conn) || + (conn->lgr->is_smcd && conn->lgr->peer_shutdown)) return -EPIPE; if (conn->lgr->is_smcd) { |