summaryrefslogtreecommitdiff
path: root/include/vm_websocket.hpp
diff options
context:
space:
mode:
authorEd Tanous <edtanous@google.com>2022-01-25 21:39:19 +0300
committerEd Tanous <ed@tanous.net>2022-02-15 03:35:55 +0300
commite662eae819f545ef07193f216b42e161b7ff7a46 (patch)
tree2b13d88ae009fa7af3396252c8787992ff2cc6c8 /include/vm_websocket.hpp
parentc5ba4c27ddb28c7844c0ea3078df4114f531dd25 (diff)
downloadbmcweb-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.hpp2
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;
}