Avoid double-free() bug.

Spotted by Fortify static analysis tool.

Improve the help message of configure script.

Suppress compiler warnings.

Use with_xxxxx (xxxxx is the package name) variables directly instead of the withval variable.

winres.h instead of afxres.h. There are some cases (VS 2012 express etc) when afxres.h doesn't exist.

Rename the name of elements in configuration(_template).xml from
setenv to setvcvars.

When nmake macros are given as command line parameters, they can't be changed.

VOID is not defined on all systems. Use void insted.

Christoph Berg

Change conifuration_template.xml so that its 64bit default is use_sspi=yes,
 use_gss=no. Also add the treatment of libpq default version.

Add support for verify-ca/full sslmode using Windows Schannel Security
Service Provider.
Root CAs must be installed into Windows Root certificate store beforehand.

Fix the newly-introduced CORR_STRCPY and CORR_VALCPY macros.

They were broken, did not compile.

Michael Paquier

Fix a bug in CC_copy_conninfo() that free()s password item of input source
unexpectedly.
Change CC_copy_conninfo() and copy_globals() so that they copy each item
one by one.

OK I found the way to handle private keys of PEM form using CryptoAPI.
Certificates of PFX form are no longer needed for SSL client certificate
authntication.

Concentrate the settings of Windows build system to an xml file and
build binaries or installers with reference to it.

(configuration.xml)
xml file to specify the setting of the build environment.
It is automatically generated in winbuild folder as a copy
of winbuild/configuration_template.xml when you invoke
editConfiguration script firstly.

winbuild/configuration.ps1 modules to handle the configuration file
winbuild/editConfiguration.ps1 edit configuration file (GUI)
winbuild/editConfiguration.bat same as above with a minimized console window

buildx86.ps1 build 32bit binary
buildx64.ps1 build 64bit binary

installer/buildx86-installer.ps1 build 32bit installer
installer/buildx64-installer.ps1 build 64bit installer

Change Windows installer build system for the next release.

Change to specify ProductCode for each version so as not to forget to change
ProductCode in case of major version up.
Remove VC runtime merge modules from the installer(runtime dlls will be
installed via psqlodbc setup program).

[64bit version only]
Use libpq by default
Remove GSSAPI support by default

Handle SSL client certificate authentication in Windows Schannel security
support provider.

You have to place the certificate file postgresql.pfx (PFX or PKCS12 format)
in %APPDATA%\postgresql folder instead of postgresql.crt and postgresql.key
(I wasn't able to find the way to handle PEM format using Cryptography API
unfortunately).
You can create the file using the following command.
 openssl pkcs12 -export -in postgresql.crt -inkey postgresql.key
 -out postgresql.pfx (with empty password).

When LF->CR+LF conversion causes an buffer truncation, supress the conversion (in case of unicode).

Don't issue a BEGIN when running VACUUM in auto-commit mode.

Normally in auto-commit mode the driver begins a new transaction
implicitly at the first statement, by sending a BEGIN statement. However,
some commands, like VACUUM, cannot be run in a transaction block, and you
will get an error like "VACUUM cannot run inside a transaction block" from
the server. In UseServerSidePrepare=0 mode, the code looks at the first word
of the query to determine if the statement is one of the special ones, and
if so, didn't begin a new transaction even when auto-commit mode is
disabled. However, in UseServerSidePrepare=1 mode, when using SQLPrepare/
SQLExecute to run the VACUUM, that check was not made. Fix that.

There was one more related inconsistency between UseServerSidePrepare modes.
Without server-side-prepares, if you issued an explicit BEGIN in auto-commit
mode, the implicit BEGIN was ont sent. But without server-side prepares, it
was. It seems best to send the implicit BEGIN in both cases, because then
you get a warning from the backend about the second BEGIN. That's a good
thing, because a sane ODBC application should be using the ODBC function
SQLEndTran() for transaction control, not explicit BEGIN/COMMIT.

Also add a test case for executing VACUUM, with and without autocommit.

Add test case for interleaving operations on several prepared statements.

Add test case for SQLDescribeParam

Remove useless 'sync' parameter from prepareParameters() function.

It was always passed as TRUE.

Add preliminary release notes for upcoming 9.3 release.

This doesn't cover everything yet, but it's a start.

Change the default for UseServerSidePrepare to 1.

The docs have recommended UseServerSidePrepare=1 for server versions 7.4
onwards, so it seems prudent to change the default so that people don't need
to remember to specify it manually.

With UseServerSidePrepare=1, the "insertreturning" regression test case
behaves better, ie. SQLNumResultCols() correctly returns the number of
columns for an INSERT RETURNING statement, even when called before
SQLExecute(). Hence change the expected output. I'm not 100% if the behavior
with UseServerSidePrepare=0 was even correct, so I'm not adding an
alternative expected output file for it.

Fix bug when a statement is executed twice with data-at-execution params.

When a statement with array-bound data-at-execution parameters is executed
twice, the second result set was not returned when UseServerSidePrepare=1.
This was demonstrated by the data-at-execution regression test case.

Add test case for testing the effect of COMMIT/ROLLBACK on cursors.

The behavior differs depending on UseDeclareFetch, so there are two expected
outputs for this.

Add test case for multi-statements.

Remove common.o on "make clean"

Handle turning standard_conforming_strings to off in mid-session.

We already watched for a ParameterStatus response indicating that
standard_conforming_strings was turned on, and acted accordingly, but
if it was turned off, we did nothing. Fix that.

Also add a test case for that.

Add test case for using SQLBindParameter with SQLExecDirect.

Remove some dead code.

Oops, used wrong #ifdef for CC_Copy.

CC_Copy is needed if _HANDLE_ENLIST_IN_DTC_ is used, regardless of
CLEANUP_CONN_BEFORE_ISOLATION. That caused compilation on Windows to fail.

Silence misc compiler warnings.

Also a few comment typo and whitespace fixes.

Configure is checking SIZEOF_LONG not SIZEOF_LONG_INT.

The first cut of psqlodbc setup project. It builds a setup program which
can't be done by a single MSI. It would install VC++ redistributable, 32
bit psqlodbc driver and 64 bit psqlodbc driver (on 64 bit windows).

Change configure.ac so that ODBC_CONFIG is set when neither unixODBC nor
iODBC is explicitly specified.

Ignore automatically-generated files in source code with .gitignore.
This is useful to prevent accidental commit of files that are not wanted
in the remote repository. The original patch was provided by Michael
Paquier. I also added files genearated by VC build tools or WIX tools
to .gitignore.

Revise MSDTC support.
1. Remove pointlessly complicated AsyncThreads stuff. Instead use
   _beginthread() to clean up threads.
2. Make pgenlist.dll from the structure change of ConnectionClass.
   The driver dlls exports the functions described in connexp.h
   which are used by pgenlist.dll.

3. Isolate the current communication path if necessary.
   While an IAsyncPG object is alive, a ConnectionClass object (hereinafter
   refered to as conn-obj) is assigned to it. The assignment has to be
   changed in the following cases.

   a) SQLDisconnect() is called for the current connection handle which
      is assigned to an IAsyncPG object.
Allocate another conn-obj and move the current communication
path (*sock* member of the current conn-obj) to the new conn-obj.
The communicaation path is lost from the current conn-obj and
the new conn-obj is assigned to the IAsyncPG object.

   b) Another (global) transaction is about to begin but the current
      global transaction is not PREPARED yet.
Same as case a) but will open a new communication path for
the current conn-obj for the subsequent ODBC API calls.

   c) Another (global) transaction is about to begin and the current
      global transaction is already PREPARED.
Allocate another conn-obj and open a new communication path for
the conn-obj. The new conn-obj is assigned to the IAsyncPG
object only to issue COMMIT/ROLLBACK PREPARED command.

   communication pass (*sock* member of the current ConnectionClass
   object) to the new object and change the state of the current
   object NOT CONNECTED. The IAsyncPG object uses new object instead
   of the current object. In case b) the current object will open a
   new communication path.

Reduce the memory usage of ConnectinClass objects by changing their large
 fixed length text fields to variable ones. Because changes are applied to
 percent-encoded fields this time, password field is also a target of this
 change.

Change SQL_ATTR_PARAMS_PROCESSED_PTR attribute which is set by SQLSetStmtAttr() from (SQLUINTEGER *) to (SQLULEN *). This fixes the bug reported by Christopf Berg. Also verify similar attributes which were changed from SQL(U)INTEGER (*) to SQL(U)LEN (*) when 64bit ODBC was introduced.

SQL_C_SLONG stands for SQLINTEGER not long.

Fix uninitialized use of 'allocbuf' variable, in case of out-of-memory.

Compiler warned about this. If ENLARGE_NEWSTATEMENT macro ran out of memory,
it would jump to cleanup routine. The cleanup would check if allocbuf is
NULL, and try to free() it if not. allocbuf needs to be initialized to NULL
before the first ENLARGE_NEWSTATEMENT macro invocation.

Change release date.

add change history.

Make sure accesses to a connection from multiple threads are exclusive. This change fixes the problem *UseDeclareFetch=1, Fetch=100, UseServerSidePrepare=1 causes Windows client to intermittently hang* reported by Jack Wilson. I hope it would also fix the problem *Segmentation Fault in Postgres server when using psqlODBC* reported by Joshua Berry.

include VS2010 Merge Modules.

VS2010 module support at postgresql 9.2. per report jack(ljwilson).

add test modules.

add change history.

Fix the *cursor XXXXXXXX already exists* error when handling *with cte* statements reported by Joe Conway.

Optimal some modification.

Apply blank date <-> null conversion functionality to timestamp type as well.

Avoid a conflict between winsock2.h and winsock.h? in socket.c.

Allow special characters in the password value of a connection string
by enclosing the value by braces({}).

Don't #include errno.h on Windows. VC10 or later intentionally changed
the values of error numbers like EINTR, EWOULDBLOCK.

change lib name by 9.2.

change lib name by 9.2.

Prep release version 09.02.0100

add the change history.

Merge branch 'snapshot'

Fix a problem reported by Jack Wilson(ljwilson@digitalav.com).
When DeUseDeclareFetch=1 and UseServerSidePrepare=1, issue COMMIT commands properly in case of AUTOCOMMIT on mode,

commit check

Add history prep release.

Make the MSI build process of 32bit drivers available in 64bit mode also.

Avoid a gpf on Windows reported by Jack Wilson(ljwilson@digitalav.com).
It occurs only when DeUseDeclareFetch=1 and UseServerSidePrepare=Parse=DisallowPremature=0. This change corrects the behavior of automatcally closed non-holdable cursors.

Fix bug with VARCHAR(5) parameter and UseServerSidePrepare=0

If you passed a VARCHAR parameter with column_size 5 to SQLPrepare, and you
had UseServerSideprepare=0 and BoolsAsChar=1, the code describes the
parameters before running the query, so that it can check if the parameter
is a boolean. That produced a bogus extra empty result set to the caller in
subsequent SQLExecute().

This fixes the two regression test failures.

Patch by Hiroshi Inoue, with minor copy-editing by me.

Add regression test suite.

Two tests are currently failing, because they hit a real bug in the driver.
After the bug is fixed, the tests should pass.

This doesn't work on Windows yet.

Teach installer that the FAQs and HOWTOs are not in this repository anymore.

Silence some compiler warnings, when building on Windows.

Add items to release notes, for things I've fixed recently.

Improve formatting of release notes.

Move release notes for 7.3 and below to separate files.

That's ancient history by now, so seems reasonable to make it less
prominent.

Fix "make dist" target, now that some of the documentation files are gone.

Fix typo in docs.

Remove documentation files that were moved to the new psqlodbc-www repository.

The version-specific files are kept in this repository, so that they are
kept in sync with the code.

Improve the handing of installation errors.

Fix a DataSource registration crash bug on 64bit Windows.

Fix column size info of MACADDR, CIDR or INET type.

Fix SQLPrepare() -> SQLDescibeCol() -> SQLPrepare() without SQLExecute() case.

Add some NULL checks to avoid crash.

Fix the bug reported by Bernhard Goebel. When fetching a row "behind" the rowset by "fetch absolute" only the first ones of the result set can be fetched. "First ones" means: row 1 to row X are readable furthermore, where X is the value used with the FETCH parameter in SQLDriverConnect.

Fix the bug about empty bytea values reported by Nils Gsche.

Add missing comparison

Change permissions on 'bootstrap' to make it executable.

Avoid multiple evaluation in some macros.

There was at least one real bug caused by the multiple evaluation of 2nd
argument to SC_set_result(). SC_pre_execute() passed QR_Constructor() call
as the 2nd argument, so we ended up creating extra result sets that were
leaked.

Change the way chained result sets are free'd, to avoid recursion.

If you have a lot of chained results, e.g. if you array bind a lot of
parameters, recursing through the whole chain might make you run out of
stack space. Convert recursion to iteration.

Also, when we're freeing all the results in the chain, we only need to
call QR_set_cursor(self, NULL) once, on the first result in the chain.
QR_set_cursor itself loops through the whole chain, resetting the cursor
name of every result. By only calling it for the first result, we avoid
O(n^2) behavior, significantly speeding up SQLFreeHandle() on a statement
with a large number of chained results.

Format if-statement with commented out "then"-block differently.

To silence a Coverity warning that the semicolon might be misplaced.

Silence compiler warning that bmp variable might be used uninitialized.

It wasn't actually true, but apparently not all compiler versions were smart
enough to realize it.

Silence warning about sscanf format vs var type mismatch on 32-bit systems.

Mark these two structs as const.

It's a good habit to mark constants as such. But I'm also hoping that this
would silence Coverity from complaining that ver2str and ver3str might
not fit in 6 bytes.

Restructure < 7.4 protocol code a bit.

This avoids accessing unitialized bitmap[0]. It was harmless, as the
uninitialize value that was read was not used for anything, but Coverity
complained about it.

Initialize field to silence a Coverity false positive.

Coverity doesn't like the fact that the struct is passed to
SC_set_rowset_start without initializing rowset_start field. It's OK,
SC_set_rowset_start doesn't actually use the 'incr' value calculated from
the field if the 3rd arg is FALSE, so this is a false positive, but seems
like a good idea to be tidy here anyway.

Remove spurious 'break'.

Found by Coverity scan.

Add NULL-check, to silence a Coverity problem.

The problem is that if the loop above doesn't find a result set with
command tag "FETCH ...", res is left to point to an already-freed result
set, and we'll go ahead and call QR_set_withhold on the already-freed
struct. That can't happen in practice, because the backend does always
return a result set with "FETCH ..." command tag when a FETCH statement is
issued, but better safe than sorry.

Remove useless null-check.

We reference the same pointer without null-check elsewhere in the function,
so if it's ever NULL, we'll crash anyway. (but looking at the callers, it
never is NULL)

Found by Coverity scan.

Don't crash if getpwuid() returns NULL.

Found by Coverity scan.

Handle realloc failures gracefully.

If realloc fails and returns NULL, don't forget the old array. Also check
for overflow of num_Stmts, which is only a 16-bit int

Remove unused variable, to silence compiler warning.

Silence compiler warnings about unused msg_truncated variables.

BTW, the error and notice handling could be made much smarter wrt.
truncation. The caller of handle_notice/error_message knows the message size
in advance, so it could easily pass it down and we could allocate a buffer
of the right size, instead of using a fixed size buffer that can be too
small.

Add parens, per compiler warning's suggestion.

Silence warning about losing const in "const char *" to "char *" assignment.