diff options
author | Ed Tanous <edtanous@google.com> | 2022-01-25 21:39:19 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2022-02-15 03:35:55 +0300 |
commit | e662eae819f545ef07193f216b42e161b7ff7a46 (patch) | |
tree | 2b13d88ae009fa7af3396252c8787992ff2cc6c8 /include/vm_websocket.hpp | |
parent | c5ba4c27ddb28c7844c0ea3078df4114f531dd25 (diff) | |
download | bmcweb-e662eae819f545ef07193f216b42e161b7ff7a46.tar.xz |
Enable readability-implicit-bool-conversion checks
These checks ensure that we're not implicitly converting ints or
pointers into bools, which makes the code easier to read.
Tested:
Ran series through redfish service validator. No changes observed.
UUID failing in Qemu both before and after.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I1ca0be980d136bd4e5474341f4fd62f2f6bbdbae
Diffstat (limited to 'include/vm_websocket.hpp')
-rw-r--r-- | include/vm_websocket.hpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/vm_websocket.hpp b/include/vm_websocket.hpp index 2a0353be32..6b7d9e7a9f 100644 --- a/include/vm_websocket.hpp +++ b/include/vm_websocket.hpp @@ -42,7 +42,7 @@ class Handler : public std::enable_shared_from_this<Handler> // boost::process::child::terminate uses SIGKILL, need to send SIGTERM // to allow the proxy to stop nbd-client and the USB device gadget. int rc = kill(proxy.id(), SIGTERM); - if (rc) + if (rc != 0) { return; } |