Fixed problem binding string params for procedures

Parameters for stored procedures were being bound as the type
of the parameter regardless of the type being passed in.
Passing a string containing "150" into a parameter expecting
an int should result in the value 150 being passed, but the
value was being corrupted. This has been fixed.

Change-Id: I3a867549df623bc5c5876e6a9ff178c527ba3451

Author: gperrow-SAP

package.json

@@ -2,7 +2,7 @@
   "author": "SQL ANYWHERE",
   "name": "sqlanywhere",
   "description": "SQL Anywhere JavaScript Driver.",
-  "version": "1.0.20",
+  "version": "1.0.21",
   "repository": {
     "url": "https://github.com/sqlanywhere/node-sqlanywhere"
   },

src/sqlanywhere.cpp

@@ -133,24 +133,9 @@ void executeWork( uv_work_t *req )
 	    getErrorMsg( baton->obj->conn, baton->error_msg );
 	    return;
 	}
-	
-	if( param.value.type == A_INVALID_TYPE &&
-	    ( baton->params[i].value.is_null == NULL ||
-	      !(*(baton->params[i].value.is_null)) ) ) {
-	    param.value.type = baton->params[i].value.type;
-	}
-	param.value.buffer = baton->params[i].value.buffer;
-	param.value.is_address = baton->params[i].value.is_address;
-	
-	if( param.value.type == A_STRING || param.value.type == A_BINARY ) {
-	    param.value.length = baton->params[i].value.length;
-	    param.value.buffer_size = baton->params[i].value.buffer_size;
-	}
-	
-	if( baton->params[i].value.is_null != NULL ) {
-	    param.value.is_null = baton->params[i].value.is_null;
-	}
-	
+
+        memcpy( &param.value, &baton->params[i].value, sizeof( param.value ) );
+
 	if( !api.sqlany_bind_param( sqlany_stmt, i, &param ) ) {
 	    baton->err = true;
 	    getErrorMsg( baton->obj->conn, baton->error_msg );

src/sqlanywhere_v0_10.cpp

@@ -126,22 +126,7 @@ void executeWork( uv_work_t *req )
 	    return;
 	}
 
-	if( param.value.type == A_INVALID_TYPE &&
-	    ( baton->params[i].value.is_null == NULL ||
-	      !(*(baton->params[i].value.is_null)) ) ) {
-	    param.value.type = baton->params[i].value.type;
-	}
-	param.value.buffer = baton->params[i].value.buffer;
-	param.value.is_address = baton->params[i].value.is_address;
-	
-	if( param.value.type == A_STRING || param.value.type == A_BINARY ) {
-	    param.value.length = baton->params[i].value.length;
-	    param.value.buffer_size = baton->params[i].value.buffer_size;
-	}
-	
-	if( baton->params[i].value.is_null != NULL ) {
-	    param.value.is_null = baton->params[i].value.is_null;
-	}
+        memcpy( &param.value, &baton->params[i].value, sizeof( param.value ) );
 
 	if( !api.sqlany_bind_param( sqlany_stmt, i, &param ) ) {
 	    baton->err = true;

src/utils.cpp

@@ -371,7 +371,7 @@ bool getBindParameters( std::vector<ExecuteData *>		&execData,
 	    param.value.buffer_size = sizeof( param_char );
 
 	} else if( bind_params->Get(i)->IsNull() ) {
-	    param.value.type = A_VAL32;
+	    param.value.type = A_STRING;
 	    sacapi_bool *is_null = new sacapi_bool;
 	    param.value.is_null = is_null;
 	    ex->addNull( is_null );

src/utils_v0_10.cpp

@@ -340,7 +340,7 @@ bool getBindParameters( std::vector<ExecuteData *>		&execData
 	    param.value.buffer_size = sizeof( param_char );
 
 	} else if( bind_params->Get(i)->IsNull() ) {
-	    param.value.type = A_VAL32;
+	    param.value.type = A_STRING;
 	    sacapi_bool *is_null = new sacapi_bool;
 	    param.value.is_null = is_null;
 	    ex->addNull( is_null );