auth: Apply Coverity's suggestions to move objects whenever possible

diff --git a/ext/lmdb-safe/lmdb-typed.hh b/ext/lmdb-safe/lmdb-typed.hh
index 45da5e00b5e3943ac267ea3725a30fbd5fc53c2c..3fcdcc74127b3365e44145df0a1645e5c520466f 100644 (file)
--- a/ext/lmdb-safe/lmdb-typed.hh
+++ b/ext/lmdb-safe/lmdb-typed.hh
@@ -242,7 +242,7 @@ class TypedDBI
 {
 public:
   TypedDBI(std::shared_ptr<MDBEnv> env, string_view name)
-    : d_env(env), d_name(name)
+    : d_env(std::move(env)), d_name(name)
   {
     d_main = d_env->openDB(name, MDB_CREATE);
 

diff --git a/modules/geoipbackend/geoipbackend.cc b/modules/geoipbackend/geoipbackend.cc
index 7c57b85b0d86f0aac813c2535e5823f498e9fa23..4ca8ed0a4766bc512af288102e836617dbb28451 100644 (file)
--- a/modules/geoipbackend/geoipbackend.cc
+++ b/modules/geoipbackend/geoipbackend.cc
@@ -194,7 +194,7 @@ void GeoIPBackend::initialize()
             string attr = iter->first.as<string>();
             if (attr == "content") {
               string content = iter->second.as<string>();
-              rr.content = content;
+              rr.content = std::move(content);
             }
             else if (attr == "weight") {
               rr.weight = iter->second.as<int>();
@@ -215,7 +215,7 @@ void GeoIPBackend::initialize()
         }
         else {
           string content = rec->second.as<string>();
-          rr.content = content;
+          rr.content = std::move(content);
           rr.weight = 100;
         }
         rr.auth = 1;

diff --git a/modules/lmdbbackend/lmdbbackend.cc b/modules/lmdbbackend/lmdbbackend.cc
index a1b9594eecb24e7fef799a04cd9f0ff9b7649804..d455cb7ca7f22d03d4e4c736796cc5ff0e78b6bb 100644 (file)
--- a/modules/lmdbbackend/lmdbbackend.cc
+++ b/modules/lmdbbackend/lmdbbackend.cc
@@ -2440,7 +2440,7 @@ bool LMDBBackend::updateDNSSECOrderNameAndAuth(uint32_t domain_id, const DNSName
           lrr.ordername = hasOrderName;
           changed = true;
         }
-        newRRs.push_back(lrr);
+        newRRs.push_back(std::move(lrr));
       }
       if (changed) {
         cursor.put(key, serToString(newRRs));

diff --git a/pdns/dnswasher.cc b/pdns/dnswasher.cc
index 0d5dd1c86c137335729b59805b85666eaf551e2a..f370ce36aee41105e8c83bb7d0779f7a6b2294f4 100644 (file)
--- a/pdns/dnswasher.cc
+++ b/pdns/dnswasher.cc
@@ -224,12 +224,12 @@ try
       cerr<<"Invalidly encoded base64 key provided"<<endl;
       exit(EXIT_FAILURE);
     }
-    ipo = IPCipherObfuscator::make(key, doDecrypt);
+    ipo = IPCipherObfuscator::make(std::move(key), doDecrypt);
   }
   else if(!g_vm.count("key") && g_vm.count("passphrase")) {
     string key = makeIPCipherKey(g_vm["passphrase"].as<string>());
 
-    ipo = IPCipherObfuscator::make(key, doDecrypt);
+    ipo = IPCipherObfuscator::make(std::move(key), doDecrypt);
   }
   else {
     cerr<<"Can't specify both 'key' and 'passphrase'"<<endl;

diff --git a/pdns/dynlistener.cc b/pdns/dynlistener.cc
index 456d62ba59d59007d5c871cadb40eb91e56d3058..d2e7b06096299e8224520ac4bd847ed3a00f47d1 100644 (file)
--- a/pdns/dynlistener.cc
+++ b/pdns/dynlistener.cc
@@ -322,7 +322,7 @@ void DynListener::sendlines(const string &l)
 void DynListener::registerFunc(const string &name, g_funk_t *gf, const string &usage, const string &args)
 {
   g_funkwithusage_t e = {gf, args, usage};
-  s_funcdb[name]=e;
+  s_funcdb[name] = std::move(e);
 }
 
 void DynListener::registerRestFunc(g_funk_t *gf)

diff --git a/pdns/pdnsutil.cc b/pdns/pdnsutil.cc
index bbb840cbdf88707d5b8c64fb42c09a2442f27ffc..ac46da046f8250eae433ba188a4da98f888891cc 100644 (file)
--- a/pdns/pdnsutil.cc
+++ b/pdns/pdnsutil.cc
@@ -889,10 +889,11 @@ static int checkAllZones(DNSSECKeeper &dk, bool exitOnError)
       errors++;
     }
 
-    seenInfos.insert(di);
+    seenInfos.insert(std::move(di));
 
-    if(errors && exitOnError)
+    if (errors && exitOnError) {
       return EXIT_FAILURE;
+    }
   }
   cout<<"Checked "<<domainInfo.size()<<" zones, "<<errors<<" had errors."<<endl;
   if(!errors)

diff --git a/pdns/webserver.cc b/pdns/webserver.cc
index 34577e70e5770a8bb301b723cff45f020bc1a155..d221d37e16d2e3f5fdffd1b71350d9526ad2459e 100644 (file)
--- a/pdns/webserver.cc
+++ b/pdns/webserver.cc
@@ -143,7 +143,7 @@ static void bareHandlerWrapper(const WebServer::HandlerFunction& handler, YaHTTP
 void WebServer::registerBareHandler(const string& url, const HandlerFunction& handler)
 {
   YaHTTP::THandlerFunction f = [=](YaHTTP::Request* req, YaHTTP::Response* resp){return bareHandlerWrapper(handler, req, resp);};
-  YaHTTP::Router::Any(url, f);
+  YaHTTP::Router::Any(url, std::move(f));
 }
 
 static bool optionsHandler(HttpRequest* req, HttpResponse* resp) {

diff --git a/pdns/ws-api.cc b/pdns/ws-api.cc
index f3b71104be3751cf23104c38ff89cbe41d74e9c0..f479336ef1b41de6de9b33eea59f01e5a5545f61 100644 (file)
--- a/pdns/ws-api.cc
+++ b/pdns/ws-api.cc
@@ -117,7 +117,7 @@ void apiDiscovery(HttpRequest* req, HttpResponse* resp) {
     { "version", 1 },
     { "url", "/api/v1" }
   };
-  Json doc = Json::array { version1 };
+  Json doc = Json::array { std::move(version1) };
 
   resp->setJsonBody(doc);
 }
@@ -130,7 +130,7 @@ void apiDiscoveryV1(HttpRequest* req, HttpResponse* resp) {
     { "server_url", "/api/v1/servers{/server}" },
     { "api_features", Json::array {} }
   };
-  Json doc = Json::array { version1 };
+  Json doc = Json::array { std::move(version1) };
 
   resp->setJsonBody(doc);
 

diff --git a/pdns/zoneparser-tng.hh b/pdns/zoneparser-tng.hh
index 1b7562766a275b6b5643baa678d702805e12528e..b50e86ec6658eb96141af70e220c8c3cbcc80fe3 100644 (file)
--- a/pdns/zoneparser-tng.hh
+++ b/pdns/zoneparser-tng.hh
@@ -60,7 +60,7 @@ private:
   unsigned makeTTLFromZone(const std::string& str);
 
   struct filestate {
-    filestate(FILE* fp, string filename) : d_fp(fp), d_filename(filename), d_lineno(0){}
+    filestate(FILE* fp, string filename) : d_fp(fp), d_filename(std::move(filename)), d_lineno(0){}
     FILE *d_fp;
     string d_filename;
     int d_lineno;