Fixing issue where connect would throw SEGFAULT on some OSs

Author: mariano

CHANGELOG

@@ -2,6 +2,10 @@ For more in depth changelog, check the Github Commits:
 
 https://github.com/mariano/node-db-mysql/commits/master
 
+0.6.6
+-----
+* Fixed issue where new connections would throw SEGFAULT on some OSs
+
 0.6.5
 -----
 * Fixed issue where conversion of result fields to JS Date object could be truncated on some systems

package.json

@@ -2,7 +2,7 @@
 , "description" : "MySQL database bindings for Node.JS"
 , "keywords": ["database","db","native","binding","library","plugin","client","mysql","libmysql"]
 , "homepage" : "http://nodejsdb.org/db-mysql"
-, "version" : "0.6.5"
+, "version" : "0.6.6"
 , "engines" : { "node" : ">=0.4.1" }
 , "maintainers" :
   [ { "name": "Mariano Iglesias"

src/connection.cc

@@ -9,18 +9,10 @@ node_db_mysql::Connection::Connection()
       timeout(0),
       writeTimeout(0),
       connection(NULL) {
-    this->connection = new MYSQL();
-    if (this->connection == NULL) {
-        throw node_db::Exception("Cannot create MYSQL handle");
-    }
-    mysql_init(this->connection);
 }
 
 node_db_mysql::Connection::~Connection() {
     this->close();
-    if (this->connection != NULL) {
-        delete this->connection;
-    }
 }
 
 void node_db_mysql::Connection::setCharset(const std::string& charset) throw() {
@@ -69,6 +61,11 @@ bool node_db_mysql::Connection::isAlive(bool ping) throw() {
 void node_db_mysql::Connection::open() throw(node_db::Exception&) {
     this->close();
 
+    this->connection = mysql_init(NULL);
+    if (this->connection == NULL) {
+        throw node_db::Exception("Cannot create MYSQL handle");
+    }
+
     if (!this->charset.empty()) {
         mysql_options(this->connection, MYSQL_SET_CHARSET_NAME, this->charset.c_str());
     }

wscript

@@ -11,7 +11,7 @@ from os.path import exists
 
 srcdir = "."
 blddir = "build"
-VERSION = "0.6.5"
+VERSION = "0.6.6"
 
 def set_options(opt):
   opt.tool_options("compiler_cxx")