From d3970eb90caa538c88e44609130d9e159e01d60f Mon Sep 17 00:00:00 2001
From: Huynh Tran <nthuynh75@gmail.com>
Date: Tue, 21 Jun 2005 19:35:52 +0000
Subject: Made sure that pointers are initalized and reset to null after
 deallocation and some reformatting.

---
 src/dal/pqdataprovider.cpp | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

(limited to 'src/dal/pqdataprovider.cpp')

diff --git a/src/dal/pqdataprovider.cpp b/src/dal/pqdataprovider.cpp
index 3fa8db45..dd91f276 100644
--- a/src/dal/pqdataprovider.cpp
+++ b/src/dal/pqdataprovider.cpp
@@ -20,30 +20,39 @@
  *  $Id$
  */
 
+
 #include "pqdataprovider.h"
 
+
 namespace tmwserv
 {
 namespace dal
 {
 
+
 /**
  * Constructor
  */
 PqDataProvider::PqDataProvider(void)
     throw()
+        : mDb(0)
 {
+    // NOOP
 }
 
+
 /**
  * Destructor
  */
 PqDataProvider::~PqDataProvider(void)
     throw()
 {
-    disconnect();
+    if (mIsConnected) {
+        disconnect();
+    }
 }
 
+
 /**
  * Get the database backend name.
  */
@@ -54,6 +63,7 @@ PqDataProvider::getDbBackend(void) const
     return DB_BKEND_POSTGRESQL;
 }
 
+
 /**
  * Create a connection to the database.
  */
@@ -82,6 +92,7 @@ PqDataProvider::connect(const std::string& dbName,
     mIsConnected = true;
 }
 
+
 /**
  * Execute a SQL query.
  */
@@ -131,6 +142,7 @@ PqDataProvider::execSql(const std::string& sql,
     }
 }
 
+
 /**
  * Close connection to database.
  */
@@ -141,11 +153,13 @@ PqDataProvider::disconnect(void)
         return;
     }
 
-    // finish up with Postgre
+    // finish up with Postgre.
     PQfinish(mDb);
 
+    mDb = 0;
     mIsConnected = false;
 }
 
+
 } // namespace dal
 } // namespace tmwserv
-- 
cgit v1.2.3-60-g2f50