diff options
| author |   Alexander Aring <aahringo@redhat.com> | 2024-08-02 13:26:40 -0400 |
|---|---|---|
| committer |   David Teigland <teigland@redhat.com> | 2024-08-08 15:15:07 -0500 |
| commit | 90ad918e371fcb7ea4cb6c0a391acc6bba51662e (patch) | |
| tree | 4afedd7bdb66231377d3a9db7e1fc6bed8b467c3 | |
| parent | dlm: remove unnecessary refcounts (diff) | |
| download | wireguard-linux-90ad918e371fcb7ea4cb6c0a391acc6bba51662e.tar.xz wireguard-linux-90ad918e371fcb7ea4cb6c0a391acc6bba51662e.zip | |
dlm: never return invalid nodeid by dlm_our_nodeid()
This patch will remote the return of an invalid nodeid value when
local_comm is not set. This case should never happen as the DLM stack
tries to compare valid nodeids with an invalid nodeid returned by
dlm_our_nodeid(). Instead we let it crash to getting at least recognized
if we running into such state.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to '')
| -rw-r--r-- | fs/dlm/config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/dlm/config.c b/fs/dlm/config.c index 99952234799e..eac96f1c1d74 100644 --- a/fs/dlm/config.c +++ b/fs/dlm/config.c @@ -928,7 +928,7 @@ int dlm_comm_seq(int nodeid, uint32_t *seq) int dlm_our_nodeid(void) { - return local_comm ? local_comm->nodeid : 0; + return local_comm->nodeid; } /* num 0 is first addr, num 1 is second addr */ |