Add various moves as suggested by Coverity

Out of all the suggestions by Coverity I picked the ones that move non-Ptr objects into variables or attributes. Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
nikolalukovic · Jan 31, 2024 · eaf9147 · eaf9147
1 parent 853169a
commit eaf9147
Show file tree

Hide file tree

Showing 10 changed files with 12 additions and 12 deletions.
diff --git a/openvpn/client/cliopthelper.hpp b/openvpn/client/cliopthelper.hpp
@@ -284,7 +284,7 @@ class ParseClientConfig
                 if (options.exists("push-peer-info"))
                     pushPeerInfo_ = true;
                 if (pushPeerInfo_)
-                    peerInfoUV_ = peer_info_uv;
+                    peerInfoUV_ = std::move(peer_info_uv);
             }
 
             // dev name

diff --git a/openvpn/client/cliproto.hpp b/openvpn/client/cliproto.hpp
@@ -715,7 +715,7 @@ class Session : ProtoContext,
         else
         {
             fatal_ = Error::AUTH_FAILED;
-            fatal_reason_ = reason;
+            fatal_reason_ = std::move(reason);
             log_reason = "AUTH_FAILED";
         }
         if (notify_callback)

diff --git a/openvpn/common/jsonhelper.hpp b/openvpn/common/jsonhelper.hpp
@@ -164,7 +164,7 @@ inline void from_vector(Json::Value &root, const T &vec, const NAME &name)
     for (auto &e : vec)
         array.append(e.to_json());
     if (array.size())
-        root[name] = array;
+        root[name] = std::move(array);
 }
 
 template <typename TITLE>

diff --git a/openvpn/common/options.hpp b/openvpn/common/options.hpp
@@ -1049,7 +1049,7 @@ class OptionList : public std::vector<Option>, public RCCopyable<thread_unsafe_r
                             extraneous_err(line_num, "option", opt);
                         untag_open_tag(opt.ref(0));
                         opt.push_back("");
-                        multiline = opt;
+                        multiline = std::move(opt);
                         in_multiline = true;
                     }
                     else
@@ -1117,7 +1117,7 @@ class OptionList : public std::vector<Option>, public RCCopyable<thread_unsafe_r
                                 extraneous_err(line_num, "meta option", opt);
                             untag_open_meta_tag(opt.ref(0));
                             opt.push_back("");
-                            multiline = opt;
+                            multiline = std::move(opt);
                             in_multiline = true;
                         }
                         else

diff --git a/openvpn/crypto/static_key.hpp b/openvpn/crypto/static_key.hpp
@@ -165,7 +165,7 @@ class OpenVPNStaticKey
         }
         if (in_body || data.size() != KEY_SIZE)
             throw static_key_parse_error();
-        key_data_ = data;
+        key_data_ = std::move(data);
     }
 
     std::string render() const

diff --git a/openvpn/netconf/linux/gw.hpp b/openvpn/netconf/linux/gw.hpp
@@ -109,7 +109,7 @@ class LinuxGW
                 if (m < best_metric || best_metric == std::numeric_limits<int>::max())
                 {
                     best_metric = m;
-                    dev_ = d;
+                    dev_ = std::move(d);
                     addr_ = a;
                 }
             }

diff --git a/openvpn/options/merge.hpp b/openvpn/options/merge.hpp
@@ -268,7 +268,7 @@ class ProfileMerge
                     if (OptionList::is_open_tag(opt.ref(0)) && opt.size() == 1)
                     {
                         OptionList::untag_open_tag(opt.ref(0));
-                        multiline = opt;
+                        multiline = std::move(opt);
                         in_multiline = true;
                         unsigned int flags = 0; // not used
                         opaque_multiline = is_fileref_directive(multiline.ref(0), flags);

diff --git a/openvpn/ssl/psid_cookie_impl.hpp b/openvpn/ssl/psid_cookie_impl.hpp
@@ -131,7 +131,7 @@ class PsidCookieImpl : public PsidCookie
 
     virtual void provide_psid_cookie_transport(PsidCookieTransportBase::Ptr pctb) override
     {
-        pctb_ = pctb;
+        pctb_ = std::move(pctb);
     }
 
 #ifndef UNIT_TEST

diff --git a/openvpn/tun/linux/client/sitnl.hpp b/openvpn/tun/linux/client/sitnl.hpp
@@ -693,8 +693,8 @@ class SITNL
 
         if (!res->route.defined() && save.ifa_index == res->ifindex)
         {
-            res->route = route;
-            OPENVPN_LOG_RTNL(__func__ << ": MATCH " << route.to_string() << " ifindex=" << save.ifa_index);
+            res->route = std::move(route);
+            OPENVPN_LOG_RTNL(__func__ << ": MATCH " << res->route.to_string() << " ifindex=" << save.ifa_index);
         }
         return 0;
     }

diff --git a/test/unittests/test_helper.hpp b/test/unittests/test_helper.hpp
@@ -303,7 +303,7 @@ class FakeAsyncResolvable : public RESOLVABLE
             EndpointType ep(openvpn_io::ip::make_address(result.first), result.second);
             endpoints.push_back(ep);
         }
-        results_[host + ":" + service] = endpoints;
+        results_[host + ":" + service] = std::move(endpoints);
     }
 
     FakeAsyncResolvable(CTOR_ARGS... args)