Fixed regression with empty parameter list

If conn.exec or stmt.exec was called with an explicitly empty
parameter list (i.e. passing [] rather than not passing
parameters at all), the driver would crash. This has been fixed.

Change-Id: I6181954e41f3cc4eaa32f782c6e0d16c3f11f6f4

Author: gperrow-SAP

package.json

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

src/utils.cpp

@@ -195,6 +195,13 @@ static bool getWideBindParameters( std::vector<ExecuteData *>		&execData,
     unsigned		num_cols = row0->Length();
     unsigned		c;
 
+    if( num_cols == 0 ) {
+	// if an empty array was passed in, we still need ExecuteData
+	ExecuteData *ex = new ExecuteData;
+	execData.push_back( ex );
+	return true;
+    }
+
     // Make sure that each array in the list has the same number and types
     // of values
     for( unsigned int r = 1; r < num_rows; r++ ) {
@@ -298,6 +305,9 @@ bool getBindParameters( std::vector<ExecuteData *>		&execData,
     Handle<Array>		bind_params = Handle<Array>::Cast( arg );
 
     if( bind_params->Length() == 0 ) {
+	// if an empty array was passed in, we still need ExecuteData
+	ExecuteData *ex = new ExecuteData;
+	execData.push_back( ex );
 	return true;
     }
 

src/utils_v0_10.cpp

@@ -162,6 +162,13 @@ static bool getWideBindParameters( std::vector<ExecuteData *>		&execData,
     Handle<Array>	row0 = Handle<Array>::Cast( rows->Get(0) );
     unsigned		num_cols = row0->Length();
     unsigned		c;
+    
+    if( num_cols == 0 ) {
+	// if an empty array was passed in, we still need ExecuteData
+	ExecuteData *ex = new ExecuteData;
+	execData.push_back( ex );
+	return true;
+    }
 
     // Make sure that each array in the list has the same number and types
     // of values
@@ -267,6 +274,9 @@ bool getBindParameters( std::vector<ExecuteData *>		&execData
     Handle<Array>		bind_params = Handle<Array>::Cast( arg );
 
     if( bind_params->Length() == 0 ) {
+	// if an empty array was passed in, we still need ExecuteData
+	ExecuteData *ex = new ExecuteData;
+	execData.push_back( ex );
 	return true;
     }