Browse Source

[3080] A few additional minor review corrections.

PgSqlLeaseMgr::openDatabase now sets conn_ to NULL after calling PQfinish.
PgSqlLeaseMgr constructor, moved member inits to the initializer list
Fixed exception text.
Thomas Markwalder 11 years ago
parent
commit
c0a4d644ec
1 changed files with 6 additions and 7 deletions
  1. 6 7
      src/lib/dhcpsrv/pgsql_lease_mgr.cc

+ 6 - 7
src/lib/dhcpsrv/pgsql_lease_mgr.cc

@@ -280,9 +280,9 @@ public:
         if (!lease_->hwaddr_.empty()) {
             if (lease->hwaddr_.size() > HWAddr::MAX_HWADDR_LEN) {
                 isc_throw(DbOperationError,
-                          "Attempted to store Hardware address longer ("
+                          "Hardware address length : "
                           << lease->hwaddr_.size()
-                          << " than allowed maximum of "
+                          << " exceeds maximum allowed of: "
                           << HWAddr::MAX_HWADDR_LEN);
             }
 
@@ -569,13 +569,10 @@ private:
 };
 
 PgSqlLeaseMgr::PgSqlLeaseMgr(const LeaseMgr::ParameterMap& parameters)
-    : LeaseMgr(parameters) {
-    conn_ = NULL;
+    : LeaseMgr(parameters), exchange4_(new PgSqlLease4Exchange()),
+    exchange6_(new PgSqlLease6Exchange()), conn_(NULL) {
     openDatabase();
     prepareStatements();
-
-    exchange4_.reset(new PgSqlLease4Exchange());
-    exchange6_.reset(new PgSqlLease6Exchange());
 }
 
 PgSqlLeaseMgr::~PgSqlLeaseMgr() {
@@ -590,6 +587,7 @@ PgSqlLeaseMgr::~PgSqlLeaseMgr() {
 
         PQclear(r);
         PQfinish(conn_);
+        conn_ = NULL;
     }
 }
 
@@ -665,6 +663,7 @@ PgSqlLeaseMgr::openDatabase() {
         // to release it, but grab the error message first.
         std::string error_message = PQerrorMessage(conn_);
         PQfinish(conn_);
+        conn_ = NULL;
         isc_throw(DbOpenError, error_message);
     }
 }