Fix cppcheck warnings

uweseimet · Dec 30, 2024 · f324038 · f324038
1 parent d5d11d7
commit f324038
Show file tree

Hide file tree

Showing 10 changed files with 14 additions and 18 deletions.
diff --git a/cpp/base/device_factory.h b/cpp/base/device_factory.h
@@ -33,7 +33,7 @@ class DeviceFactory
     shared_ptr<PrimaryDevice> CreateDevice(PbDeviceType, int, const string&) const;
     PbDeviceType GetTypeForFile(const string&) const;
 
-    auto GetExtensionMapping() const
+    const auto& GetExtensionMapping() const
     {
         return mapping;
     }

diff --git a/cpp/base/property_handler.cpp b/cpp/base/property_handler.cpp
@@ -93,7 +93,7 @@ property_map PropertyHandler::GetProperties(const string &filter) const
     return filtered_properties;
 }
 
-property_map PropertyHandler::GetUnknownProperties() const
+const property_map& PropertyHandler::GetUnknownProperties() const
 {
     return unknown_properties;
 }

diff --git a/cpp/base/property_handler.h b/cpp/base/property_handler.h
@@ -55,7 +55,7 @@ class PropertyHandler
     void Init(const string&, const property_map&, bool);
 
     property_map GetProperties(const string& = "") const;
-    property_map GetUnknownProperties() const;
+    const property_map& GetUnknownProperties() const;
     const string& RemoveProperty(const string&, const string& = "");
     void AddProperty(const string&, string_view);
     void RemoveProperties(const string&);

diff --git a/cpp/command/command_image_support.h b/cpp/command/command_image_support.h
@@ -36,7 +36,7 @@ class CommandImageSupport
     {
         return depth;
     }
-    string GetDefaultFolder() const
+    const string& GetDefaultFolder() const
     {
         return default_folder;
     }

diff --git a/cpp/devices/scsi_generic.h b/cpp/devices/scsi_generic.h
@@ -30,7 +30,7 @@ class ScsiGeneric : public PrimaryDevice
         return device + " (" + GetPaddedName() + ")";
     }
 
-    string GetDevice() const
+    const string& GetDevice() const
     {
         return device;
     }

diff --git a/cpp/devices/storage_device.h b/cpp/devices/storage_device.h
@@ -49,7 +49,7 @@ class StorageDevice : public PrimaryDevice
     {
         filename = filesystem::path(file);
     }
-    string GetLastFilename() const
+    const string& GetLastFilename() const
     {
         return last_filename;
     }
@@ -93,7 +93,7 @@ class StorageDevice : public PrimaryDevice
         medium_changed = b;
     }
 
-    static auto GetReservedFiles()
+    static const auto& GetReservedFiles()
     {
         return reserved_files;
     }

diff --git a/cpp/devices/tap_driver.cpp b/cpp/devices/tap_driver.cpp
@@ -221,7 +221,7 @@ pair<string, string> TapDriver::ExtractAddressAndMask(logger &logger) const
         address = components[0];
 
         const int m = ParseAsUnsignedInt(components[1]);
-        if (m == -1 || m < 8 || m > 32) {
+        if (m < 8 || m > 32) {
             logger.error("Invalid CIDR netmask notation '{}'", components[1]);
             return {"", ""};
         }

diff --git a/cpp/devices/tape.cpp b/cpp/devices/tape.cpp
@@ -537,7 +537,7 @@ bool Tape::Locate(bool locate_16)
         throw ScsiException(SenseKey::ILLEGAL_REQUEST, Asc::INVALID_FIELD_IN_CDB);
     }
 
-    auto identifier = static_cast<uint32_t>(locate_16 ? GetCdbInt64(4) : GetCdbInt32(3));
+    auto identifier = locate_16 ? GetCdbInt64(4) : GetCdbInt32(3);
     const bool bt = GetCdbByte(1) & 0x04;
 
     if (tar_file) {

diff --git a/cpp/s2pexec/s2pexec_core.cpp b/cpp/s2pexec/s2pexec_core.cpp
@@ -330,9 +330,7 @@ bool S2pExec::ParseArguments(span<char*> args, bool in_process)
         if (buffer_size = ParseAsUnsignedInt(buf); buffer_size <= 0) {
             throw ParserException("Invalid receive buffer size: '" + buf + "'");
         }
-        else {
-            buffer.resize(buffer_size);
-        }
+        buffer.resize(buffer_size);
     }
 
     return true;

diff --git a/cpp/shared/s2p_util.cpp b/cpp/shared/s2p_util.cpp
@@ -176,12 +176,10 @@ string s2p_util::ParseIdAndLun(const string &id_spec, int &id, int &lun)
     }
 
     if (const auto &components = Split(id_spec, COMPONENT_SEPARATOR, 2); !components.empty()) {
-        if (components.size() >= 1) {
-            id = ParseAsUnsignedInt(components[0]);
-            if (id == -1 || id > 7) {
-                id = -1;
-                return "Invalid device ID: '" + components[0] + "' (0-7)";
-            }
+        id = ParseAsUnsignedInt(components[0]);
+        if (id == -1 || id > 7) {
+            id = -1;
+            return "Invalid device ID: '" + components[0] + "' (0-7)";
         }
 
         if (components.size() >= 2) {