The memory for sql->bound_values is now freed by mdb_sql_reset, so these values no longer should be freed in dump_results_pp. Also, this fixes a memory leak that was occurring when results were not being pretty printed.
SQL engine was being reset in both dump_results functions. Rather than having several different functions responsible for resetting the engine depending on the circumstances, the reset now only occurs in run_query, the same function where the query is executed.
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.
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.