diff options
| author | Ilya Dryomov <idryomov@gmail.com> | 2021-01-20 14:49:07 +0100 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@gmail.com> | 2021-01-21 16:49:59 +0100 |
| commit | 9d5ae6f3c50a6f718b6d4be3c7b0828966e01b05 (patch) | |
| tree | e22d6b87aa4ea94b2cb458aa9fc9b35d0e5a4e19 /net/ceph | |
| parent | 4972cf605f8a10784bb9ec9bdf3465892fb547c8 (diff) | |
| download | linux-9d5ae6f3c50a6f718b6d4be3c7b0828966e01b05.tar.gz | |
libceph: fix "Boolean result is used in bitwise operation" warning
This line dates back to 2013, but cppcheck complained because commit
2f713615ddd9 ("libceph: move msgr1 protocol implementation to its own
file") moved it. Add parenthesis to silence the warning.
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'net/ceph')
| -rw-r--r-- | net/ceph/messenger_v1.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ceph/messenger_v1.c b/net/ceph/messenger_v1.c index 04f653b3c89733..2cb5ffdf071af7 100644 --- a/net/ceph/messenger_v1.c +++ b/net/ceph/messenger_v1.c @@ -1100,7 +1100,7 @@ static int read_partial_message(struct ceph_connection *con) if (ret < 0) return ret; - BUG_ON(!con->in_msg ^ skip); + BUG_ON((!con->in_msg) ^ skip); if (skip) { /* skip this message */ dout("alloc_msg said skip message\n"); |