Coverity nits

Should fix CID 1168020, 1167490, 1167489, 1167488, 1126082.

diff --git a/pdns/ext/yahttp/yahttp/reqresp.cpp b/pdns/ext/yahttp/yahttp/reqresp.cpp
index 62117e31b2e2e149b15d41b2a3186ffa3d9b9f0d..03d59c149bad4d8d598e86ffe76d2d3ee1202fdd 100644 (file)
--- a/pdns/ext/yahttp/yahttp/reqresp.cpp
+++ b/pdns/ext/yahttp/yahttp/reqresp.cpp
@@ -35,7 +35,7 @@ namespace YaHTTP {
   };
   Request::~Request() {};
 
-  Response::Response() {};
+  Response::Response() : status(0) {};
   Response::Response(const Request &req) {
     headers["connection"] = "close";
     method = req.method;

diff --git a/pdns/session.cc b/pdns/session.cc
index c3ee0a2aa4b62600b0099742287e975373e16290..0634647ceb6b7e5444a5c2599e83cbf11d4869eb 100644 (file)
--- a/pdns/session.cc
+++ b/pdns/session.cc
@@ -31,13 +31,13 @@
 #include "misc.hh"
 #include "iputils.hh"
 
-Session::Session(int s, ComboAddress r) : d_good(true)
+Session::Session(int s, ComboAddress r) : d_timeout(10), d_good(true)
 {
   d_remote=r;
   d_socket=s;
 }
 
-Session::Session() : d_good(false)
+Session::Session() : d_socket(-1), d_timeout(10), d_good(false)
 {
 }
 

diff --git a/pdns/ws-recursor.cc b/pdns/ws-recursor.cc
index 85fcb49ddb210b11eb34709a52bb674edc87bdd3..256c1bcdc5c49ea15e672ad3aa11712e2328bcad 100644 (file)
--- a/pdns/ws-recursor.cc
+++ b/pdns/ws-recursor.cc
@@ -48,8 +48,10 @@ RecursorWebServer::RecursorWebServer(FDMultiplexer* fdm)
 {
   RecursorControlParser rcp; // inits
 
-  if(!arg().mustDo("experimental-webserver"))
+  if(!arg().mustDo("experimental-webserver")) {
+    d_ws = NULL;
     return;
+  }
 
   d_ws = new AsyncWebServer(fdm, arg()["experimental-webserver-address"], arg().asNum("experimental-webserver-port"), arg()["experimental-webserver-password"]);