commit 513305859675cbac9a50ef50ab3046359f166383
author Patrick Williams <patrick@stwcx.xyz> Wed Oct 06 12:48:56 2021 -0500
committer Patrick Williams <patrick@stwcx.xyz> Mon Oct 11 02:16:27 2021 +0000
tree 8f83647e82fec18b5b7ad461a2e9eb5be2d3d537
parent 60c186b92abb1ba7de2db3a2a882ee91b6c753ec

catch exceptions as const

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I6a2316d11f1dd70d8dc2368c2aa01d88cf1cdb26

common/utils.cpp

diff --git a/common/utils.cpp b/common/utils.cpp
index 7f97320..6f8a6eb 100644
--- a/common/utils.cpp
+++ b/common/utils.cpp
@@ -451,7 +451,7 @@
 
         msg.signal_send();
     }
-    catch (std::exception& e)
+    catch (const std::exception& e)
     {
         std::cerr << "Error emitting pldm event signal:"
                   << "ERROR=" << e.what() << "\n";

libpldmresponder/bios.cpp

diff --git a/libpldmresponder/bios.cpp b/libpldmresponder/bios.cpp
index 74000b8..07e6d40 100644
--- a/libpldmresponder/bios.cpp
+++ b/libpldmresponder/bios.cpp
@@ -183,7 +183,7 @@
                                 "uint64_t"};
         pldm::utils::DBusHandler().setDbusProperty(dbusMapping, value);
     }
-    catch (std::exception& e)
+    catch (const std::exception& e)
     {
 
         std::cerr << "Error Setting time,PATH=" << setTimePath

libpldmresponder/bios_config.cpp

diff --git a/libpldmresponder/bios_config.cpp b/libpldmresponder/bios_config.cpp
index 57e2dc9..e955f26 100644
--- a/libpldmresponder/bios_config.cpp
+++ b/libpldmresponder/bios_config.cpp
@@ -799,7 +799,7 @@
     {
         attrNameHdl = biosStringTable.findHandle(attrName);
     }
-    catch (std::invalid_argument& e)
+    catch (const std::invalid_argument& e)
     {
         std::cerr << "Could not find handle for BIOS string, ATTRIBUTE="
                   << attrName.c_str() << "\n";

libpldmresponder/bios_table.cpp

diff --git a/libpldmresponder/bios_table.cpp b/libpldmresponder/bios_table.cpp
index a618f52..0524614 100644
--- a/libpldmresponder/bios_table.cpp
+++ b/libpldmresponder/bios_table.cpp
@@ -23,7 +23,7 @@
     {
         empty = fs::is_empty(filePath);
     }
-    catch (fs::filesystem_error& e)
+    catch (const fs::filesystem_error& e)
     {
         return true;
     }

oem/ibm/libpldmresponder/file_io.cpp

diff --git a/oem/ibm/libpldmresponder/file_io.cpp b/oem/ibm/libpldmresponder/file_io.cpp
index 1192510..7d46a38 100644
--- a/oem/ibm/libpldmresponder/file_io.cpp
+++ b/oem/ibm/libpldmresponder/file_io.cpp
@@ -262,7 +262,7 @@
     {
         value = table.at(fileHandle);
     }
-    catch (std::exception& e)
+    catch (const std::exception& e)
     {
         std::cerr << "File handle does not exist in the file table, HANDLE="
                   << fileHandle << "\n";
@@ -354,7 +354,7 @@
     {
         value = table.at(fileHandle);
     }
-    catch (std::exception& e)
+    catch (const std::exception& e)
     {
         std::cerr << "File handle does not exist in the file table, HANDLE="
                   << fileHandle << "\n";
@@ -479,7 +479,7 @@
     {
         value = table.at(fileHandle);
     }
-    catch (std::exception& e)
+    catch (const std::exception& e)
     {
         std::cerr << "File handle does not exist in the file table, HANDLE="
                   << fileHandle << "\n";
@@ -560,7 +560,7 @@
     {
         value = table.at(fileHandle);
     }
-    catch (std::exception& e)
+    catch (const std::exception& e)
     {
         std::cerr << "File handle does not exist in the file table, HANDLE="
                   << fileHandle << "\n";