From 40d1ecd11fad23ef9ce49cb05014766ea8420170 Mon Sep 17 00:00:00 2001 From: rogerswb Date: Thu, 23 May 2013 14:58:19 -0500 Subject: [PATCH] Update _SQLGetData - Was incorrectly attempting to validate input of an output only parameter According to the spec, pcbValue is designated as an output only variable, so there is no need to validate any incoming values. Some packages do not initialize this variable before passing it into the function call; this will randomly result in an improperly thrown error. Removed the input validation. --- src/odbc/odbc.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/odbc/odbc.c b/src/odbc/odbc.c index e0d965a..6e7852b 100644 --- a/src/odbc/odbc.c +++ b/src/odbc/odbc.c @@ -1538,10 +1538,6 @@ static SQLRETURN SQL_API _SQLGetData( strcpy(sqlState, "HY009"); return SQL_ERROR; } - if (pcbValue && *pcbValue<0) { - strcpy(sqlState, "HY090"); - return SQL_ERROR; - } if (col->col_type == MDB_BOOL) { // bool cannot be null