commit 8e0b9db6eb9e190da89b1e031be1ecde8f0ae4d1
author PavanKumarIntel <pavanx.kumar.martha@intel.com> Mon Oct 30 16:50:27 2023 +0000
committer pavanx.kumar.martha <pavanx.kumar.martha@intel.com> Wed Nov 15 16:34:53 2023 +0000
tree ffb88c62c46ca75a2a6bdb8d4a7ef5227d5661e7
parent e4d159b799a6597e7b980cf51fb86cacf6bc5eed

Fix the bugs found in static analysis

Not restoring ostream format:
    cerr format not restoring to default, can lead to printing
    values in incorrect format.

Change-Id: Ie529b9a8261f651c76d2fc232d89d4f711cb05b5
Signed-off-by: PavanKumarIntel <pavanx.kumar.martha@intel.com>

callback-manager/src/callback_manager.cpp

diff --git a/callback-manager/src/callback_manager.cpp b/callback-manager/src/callback_manager.cpp
index d6a5dee..aaa5ffc 100644
--- a/callback-manager/src/callback_manager.cpp
+++ b/callback-manager/src/callback_manager.cpp
@@ -108,6 +108,7 @@
         ledsToSet.push_back(std::make_pair(okLedPath, false));
         for (const auto& ledPair : ledsToSet)
         {
+            std::ios_base::fmtflags originalFlags = std::cerr.flags();
             conn->async_method_call(
                 [ledPair](const boost::system::error_code ec) {
                     if (ec)
@@ -115,12 +116,14 @@
                         std::cerr << "Cannot set " << ledPair.first << " to "
                                   << std::boolalpha
                                   << std::get<bool>(ledPair.second) << "\n";
+                        std::cerr.flags(originalFlags);
                     }
                     if constexpr (debug)
                     {
                         std::cerr << "Set " << ledPair.first << " to "
                                   << std::boolalpha
                                   << std::get<bool>(ledPair.second) << "\n";
+                        std::cerr.flags(originalFlags);
                     }
                 },
                 ledManagerBusname, ledPair.first,