diff options
author | Ed Tanous <edtanous@google.com> | 2022-01-07 20:30:41 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2022-01-12 22:00:37 +0300 |
commit | d3a9e08452986b28c8024de2b0fdb3cadbf4411b (patch) | |
tree | f9dfbc45cfdf3c58a9e03e5c8d98cc199c417243 | |
parent | ca45aa3c416e55027b8d17078bac6493e81d564f (diff) | |
download | bmcweb-d3a9e08452986b28c8024de2b0fdb3cadbf4411b.tar.xz |
Enforce variable init
There were a few places we weren't initting our variables per cpp core
guidelines. Fix all of them, and enable checks for this.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Iba09924beb9fb26f597ff94d1cecbd6d6b1af912
-rw-r--r-- | .clang-tidy | 2 | ||||
-rw-r--r-- | include/multipart_parser.hpp | 6 | ||||
-rw-r--r-- | include/sessions.hpp | 2 | ||||
-rw-r--r-- | redfish-core/include/event_service_manager.hpp | 9 | ||||
-rw-r--r-- | redfish-core/include/utils/systemd_utils.hpp | 2 | ||||
-rw-r--r-- | redfish-core/lib/network_protocol.hpp | 2 |
6 files changed, 13 insertions, 10 deletions
diff --git a/.clang-tidy b/.clang-tidy index eae3c722cb..311617baa6 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -202,7 +202,9 @@ clang-analyzer-valist.ValistBase, clang-analyzer-webkit.NoUncountedMemberChecker, clang-analyzer-webkit.RefCntblBaseVirtualDtor, cppcoreguidelines-init-variables, +cppcoreguidelines-interfaces-global-init, cppcoreguidelines-pro-bounds-pointer-arithmetic, +cppcoreguidelines-pro-type-member-init, cppcoreguidelines-pro-type-reinterpret-cast, cppcoreguidelines-special-member-functions, misc-misplaced-const, diff --git a/include/multipart_parser.hpp b/include/multipart_parser.hpp index e385558037..ee028f1d3e 100644 --- a/include/multipart_parser.hpp +++ b/include/multipart_parser.hpp @@ -337,10 +337,10 @@ class MultipartParser static constexpr char hyphen = '-'; static constexpr char colon = ':'; - std::array<bool, 256> boundaryIndex; + std::array<bool, 256> boundaryIndex{}; std::string lookbehind; - State state; - Boundary flags; + State state{State::START}; + Boundary flags{Boundary::NON_BOUNDARY}; size_t index = 0; size_t partDataMark = 0; size_t headerFieldMark = 0; diff --git a/include/sessions.hpp b/include/sessions.hpp index 79e71ced9a..b445915e34 100644 --- a/include/sessions.hpp +++ b/include/sessions.hpp @@ -46,7 +46,7 @@ struct UserSession std::string clientId; std::string clientIp; std::chrono::time_point<std::chrono::steady_clock> lastUpdated; - PersistenceType persistence; + PersistenceType persistence{PersistenceType::TIMEOUT}; bool cookieAuth = false; bool isConfigureSelfOnly = false; diff --git a/redfish-core/include/event_service_manager.hpp b/redfish-core/include/event_service_manager.hpp index 7cb5cc7cb7..ee402b61ee 100644 --- a/redfish-core/include/event_service_manager.hpp +++ b/redfish-core/include/event_service_manager.hpp @@ -564,9 +564,9 @@ class Subscription : public persistent_data::UserSubscription class EventServiceManager { private: - bool serviceEnabled; - uint32_t retryAttempts; - uint32_t retryTimeoutInterval; + bool serviceEnabled = false; + uint32_t retryAttempts = 0; + uint32_t retryTimeoutInterval = 0; EventServiceManager() { @@ -1189,7 +1189,8 @@ class EventServiceManager std::size_t index = 0; while ((index + iEventSize) <= bytesTransferred) { - struct inotify_event event; + struct inotify_event event + {}; std::memcpy(&event, &readBuffer[index], iEventSize); if (event.wd == dirWatchDesc) { diff --git a/redfish-core/include/utils/systemd_utils.hpp b/redfish-core/include/utils/systemd_utils.hpp index 75bbe35806..ad157dc793 100644 --- a/redfish-core/include/utils/systemd_utils.hpp +++ b/redfish-core/include/utils/systemd_utils.hpp @@ -39,7 +39,7 @@ inline std::string getUuid() if (sd_id128_get_machine_app_specific(appId, &machineId) == 0) { - std::array<char, SD_ID128_STRING_MAX> str; + std::array<char, SD_ID128_STRING_MAX> str{}; ret = sd_id128_to_string(machineId, str.data()); ret.insert(8, 1, '-'); ret.insert(13, 1, '-'); diff --git a/redfish-core/lib/network_protocol.hpp b/redfish-core/lib/network_protocol.hpp index 41b14f83c3..61c3c68a07 100644 --- a/redfish-core/lib/network_protocol.hpp +++ b/redfish-core/lib/network_protocol.hpp @@ -358,7 +358,7 @@ inline std::string getHostName() { std::string hostName; - std::array<char, HOST_NAME_MAX> hostNameCStr; + std::array<char, HOST_NAME_MAX> hostNameCStr{}; if (gethostname(hostNameCStr.data(), hostNameCStr.size()) == 0) { hostName = hostNameCStr.data(); |