diff options
| author |   Alexander Aring <aahringo@redhat.com> | 2022-08-15 15:43:21 -0400 |
|---|---|---|
| committer |   David Teigland <teigland@redhat.com> | 2022-08-23 14:53:59 -0500 |
| commit | f45307d395da7ab1aa537cf00e8fd7123a2be4e2 (patch) | |
| tree | f5c85cf1038b07b770366b43e10bebe40356a57e | |
| parent | fs: dlm: allow lockspaces have zero lvblen (diff) | |
| download | linux-dev-f45307d395da7ab1aa537cf00e8fd7123a2be4e2.tar.xz linux-dev-f45307d395da7ab1aa537cf00e8fd7123a2be4e2.zip | |
fs: dlm: handle rcom in else if branch
Currently we handle in dlm_receive_buffer() everything else than a
DLM_MSG type as DLM_RCOM message. Although a different message than
DLM_MSG should be a DLM_RCOM we should explicit check on DLM_RCOM and
drop a log_error() if we see something unexpected.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
| -rw-r--r-- | fs/dlm/lock.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/dlm/lock.c b/fs/dlm/lock.c index da95ba3c295e..c41aa8ab3230 100644 --- a/fs/dlm/lock.c +++ b/fs/dlm/lock.c @@ -5108,8 +5108,11 @@ void dlm_receive_buffer(union dlm_packet *p, int nodeid) down_read(&ls->ls_recv_active); if (hd->h_cmd == DLM_MSG) dlm_receive_message(ls, &p->message, nodeid); - else + else if (hd->h_cmd == DLM_RCOM) dlm_receive_rcom(ls, &p->rcom, nodeid); + else + log_error(ls, "invalid h_cmd %d from %d lockspace %x", + hd->h_cmd, nodeid, le32_to_cpu(hd->u.h_lockspace)); up_read(&ls->ls_recv_active); dlm_put_lockspace(ls); |