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 /src | |
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 'src')
-rw-r--r-- | src/webserver_main.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/webserver_main.cpp b/src/webserver_main.cpp index b613008360..b05161d316 100644 --- a/src/webserver_main.cpp +++ b/src/webserver_main.cpp @@ -40,7 +40,7 @@ inline void setupSocket(crow::App& app) { BMCWEB_LOG_INFO << "attempting systemd socket activation"; if (sd_is_socket_inet(SD_LISTEN_FDS_START, AF_UNSPEC, SOCK_STREAM, 1, - 0)) + 0) != 0) { BMCWEB_LOG_INFO << "Starting webserver on socket handle " << SD_LISTEN_FDS_START; @@ -113,7 +113,7 @@ int run() crow::google_api::requestRoutes(app); #endif - if (bmcwebInsecureDisableXssPrevention) + if (bmcwebInsecureDisableXssPrevention != 0) { cors_preflight::requestRoutes(app); } @@ -128,7 +128,7 @@ int run() #ifndef BMCWEB_ENABLE_REDFISH_DBUS_LOG_ENTRIES int rc = redfish::EventServiceManager::startEventLogMonitor(*io); - if (rc) + if (rc != 0) { BMCWEB_LOG_ERROR << "Redfish event handler setup failed..."; return rc; |