Add test case for large objects.

Add test case.

This tests a codepath in QR_next_tuple that had no coverage before.

Avoid conflicts of external symbols other than ODBC APis.

Change *ANSI* in the title of 32-bit Unicode driver setup dialog to *Unicode*.

Remove dead code

Remove option to disable geqo.

You can use the generic ConnSettings option for that.

Minor code simplification.

Any compiler worth its salt will turn "strlen(constant)" into a constant,
so let's get rid of the fancy logic to precalculate those.

Fix max identifier length stuff.

CC_get_max_idlen function didn't work correctly. It sent the "show
identifier_length" query to the backend, but assumed that the value comes
back in the command tag. It doesnt; the command tag is always "SHOW".

We fell back to the compiled in NAMEDATALEN_V73 default. But that's
off-by-one; the actual max identifier length is NAMEDATALEN - 1.

Remove support for receiving 'P' or 'B' responses from server.

Server versions >= 7.4 don't send these anymore.

Remove some unused #defines

Avoid unnecessary round-trip to server at connection startup.

When connecting with libpq, we issued an explicit query,
"SELECT pg_client_encoding()" at the connection startup to get the
effective client_encoding. That's not really required; the server reports
that automatically right after the authentication handshake. We just have
to ask libpq for the current setting.

Merge info30.c and info.c

We no longer support ODBC 3.0, so these don't need to be separate anymore.
This removes some duplicated code at the bottom of each GetInfoAPI.

Michael Paquier

Remove odbcapi25w.c

All the functions in that file are obsolete, as we no longer support old
ODBC versions.

Michael Paquier

Remove some remnants of ODBCVER < 3.51 support.

Remove hack for Windows 95 & 98.

Let's not pretend that we still support those.

Remove references to removed KSQO and protocol options from the docs.

And some other stuff that was only relevant to very old server versions.

Michael Paquier, with some tweaks by me.

Remove support for ODBC versions < 3.51.

Older versions have been obsolete for a very long time by now. Removing the
support lets us get rid of a lot of code that no-one has in practice been
even compiling for a long time.

Michael Paquier

Remove support for protocol versions older than 3.

PostgreSQL added support for protocol version 3 in server version 7.4.
There's no need to support servers older than that. This simplifies the
code a lot, removing a lot of corner cases that haven't been properly
regression tested for a long time anyway.

Original patch by me, with some tweaks by Michael Paquier.

Prep next release version 09.03.0400.

Oops, two files forget..

Prep next release Version 09.03.0400.

Add some files.

Prep next release version 09.03.0400.

Fix some differnces in regression test.
1. result-convrsion-test.c
   Explicitly specify an appropriate time zone.
   isprint() doesn't work as expected in my environment. It's safe
   to verify that the parameter is a single byte before calling
   isprint().

2. prepare-test.c
   Explicitly specify "postgres_verbose" intervalstyle.

3. diagnostics.c
   pg_terminate_backend() sometimes succeeds with no error message.

Enable -Wall compile option if it's available.

PG_BUILD_VERSION seems no longer needed.

Include pdb files in the contents of installers.

Removed RESET_CRYPTO_CALLBACKS preprocessor symbol definition.
This is no longer needed and rather dangerous.
The removal also make it possible to remove the dependency to libeay32.

Removed a duplicated description.

Suppress option button from the bootstrapper dialog using BalExtention.

Don't continue to build installers in case the compilation failed.

Use 1ST, 2ND or 3RD PAGE instead of PREV or NEXT PAGE for the setup
program.

Add a powershell script buildBootStrapper.ps1 which builds bootStrapper program
with some optional parameters. Quote the version number of bootstrapper from
the configuration xml file unless specified. Also change the name of 32bit MSI
in the bootstrapper according to changes of 32bit installer build system.
Change buildInstallers.ps1 build the bootstrapper together.
Change ALLUSERS from "2" to "1" of the 64bit installer.

Fix the wrong use of snprintf format in the previous commit.

Avoid the registry operation to set XADLL entry from wow64 process.

Cleanup the code for *test* button and add the functionality to test
MSDTC support.

Add a new dialog page to the setup program and allow the setting
of PREFERLIBPQ and XAOPT option.

Silence compilers warnings without _HANDLE_ENLIST_IN_DTC_ or USE_LIBPQ.

Compiler complained about unused variable.

Fix SQLTables column names to be ODBC version 3 compliant

In ODBC3, TABLE_QUALIFIER was renamed to TABLE_CAT, and TABLE_OWNER to
TABLE_SCHEM.

David Hedberg

Improve regression test for reading a bytea column to SQL_C_CHAR.

When converting a bytea to SQL_C_CHAR, the driver uses upper-case letters in
the hex output, while the backend uses lower-case letters. That leads to
different output when using bytea_output=hex than with bytea_output=escape.
Set bytea_output explicitly in the regression test, so that it works
regardless of the current bytea_output setting. And test both settings - the
code path taken in the driver is quite different.

Add the libpq use parameter to the dtc connection string.

Additonal improvement on MSDTC enlistment.

Unfortunately the current pgxalib.dll can't recover transactions which use
sslmode verify-[ca|full] or whose authentication is SSPI, certificate or
ident.
When MSDTC.exe tries to connect to PostgreSQL it does so under Windows user
NETWORKSERVICE, which won't match the username the original user connected
to PostgreSQL as. So PostgreSQL will reject the connection.

In addition XARMcreate() doesn't detect it on enlistment because invoked
xa_open() doesn't connect to the specified datasource. Though it has no
problem unless crash occurs between PREAPARE and COMMIT, the recovery of
the transaction is impossible when such a crash occurs.

Now 3 diffrent behaviors can be speicified using the xaopt connection
string option.

 1: The current default behavior.
 2: Same as above except it simply examines if the recovery for the specified
    datasource is unavailable before calling XARMcreate(). Currently it only
    examines whether the connection uses sslmode verify-[ca|full]. I can't find
    the way to detect SSPI, certicate or ident authentication.
 3: Invoked xa_open() tries to connect to the specified datasource immediately
    as user Network Service.

Register the XA DLL on installation.
MSDTC runs in 64bit mode on 64bit machines and 32bit mode on 32bit machines.
We had better register the XA DLL on installation.

Fix the regkey search bug when XARMcreate() causes an error at enlistment
in DTC.

Add '#ifdef USE_LIBPQ' around the code which use PreferLibpq option
introduced in the previous commit.

Add an option to explicitly force/avoid the use of libpq library.

Improve pgxalib.dll(MSDTC support).
1. Add an option which lets xa_open() try to connect to the database
   immediately. This would cause an error at enlistment in dtc when
   the authentication method depends not only on the db user but also
   on the OS user, which means the automatic transaction recovery is
   unavailable.
2. Log the error message in case of SQLDriverConnect() error.
3. Improve the logging.

Fix the bug reported by Phillippe Champignon.
The function SC_setInsertedTable() prepares an info for subsequent
'select @@IDENTITY' command. Take table name like catalog.schema.table
into account in the function.

Add msvc runtime dlls which are needed by the drivers to the installers.

Add a switch type parameter to BuildAll Powershell script which makes
it possible to build the installers after building drivers.

Fix a crash bug.

Fix buffer overflow in SQLGetData.

If a PostgreSQL timestamp column is fetched using SQLGetData, into a
SQL_C_CHAR output variable, with size of 20-22 bytes, the driver might
overrun the buffer by a few bytes with unusual timestamp values, with
year < 0 or > 10000.

While at it, fix the truncation behavior for date/time/timestamp values
so that if the output buffer is too small, we still write what fits and
truncate, instead of writing nothing. This is something that one might
do on purpose, e.g. if you only want to fetch the year part of a date,
you might call SQLGetData() on a date column, with only 4 bytes long
output buffer. (although we haven't heard any complaints from the field).

Also, if the output buffer is of type SQL_C_WCHAR, but the buffer length
is odd, and we have to truncate, we didn't NULL-terminate the string.
Using an odd-sized buffer with SQL_C_WCHAR is a pretty unlikely thing to
do, but nevertheless I think this is more correct behavior.

Add test cases for the bugs fixed.

Add a test case for data type conversions for results coming from server

Minor cleanup.

Remove unused or bogus PG_TYPE macros.

Some of these allegedly built-in types didn't exist in PostgreSQL source code
(maybe they did in some ancient version?), some were just unused in
psqlodbc code.

Remove unused function.

Silences a compiler warning.

Fix bug in building an abbreviated connection string.

Introduced by the keepalive-patch, which accidentally removed a line from
sprintf format string. Caught by a compiler warning.

Silence compiler warnings about unused variables.

Fix race condition in getting the current decimal separator.

The code to get the current decimal separator was not thread-safe. If two
threads concurrently saw that the locale had changed, they might both try
to free() the same string.

This isn't a perfect fix, there's still a race condition if a thread changes
the locale while another thread is running the psqlodbc code. But at least
this eliminates the race condition when the locale was changed earlier, not
concurrently, and even if it is changed concurrently, this is less likely
to crash, depending on the way localeconv() is implemented in libc.

Clear unused stuff from PgDtc_isolate() function. Also add CC_initialize_pg_version() call after CC_copy_conninfo() so as to avoid the trouble caused by early PG_VERSION_xx() calls.

Get rid of copy and paste code in installer/ and unify them.

The powershell scripts had lots of duplication. Make the unified
code instead.
Original patch was provided by Craig Ringer.

Add a check for NULL dereference pointed out by Pluto Cobain.
It's for safety though I don't think it would occur.

Remove duplicated line in snprintf() which causes a crash introduced by
the commit 1177281049e8bb3df5992da5dc1ca31ad892082e.

Remove unsed dialog which was added by mistake.

Revise descriptor stuff.
descriptor.c(h) was broken by the commit
2da1668f4c7d7871a7212db10af606a7e646abd6
and it now causes a crash bug.
Instead of reverting the change revising the stuff completely.

Fix a memory leak pointed out by pluto.cobain.

Remove the duplicate winbuild readme

Remove obsolete .bat installer creators

Document enabling MSDTC tracing

Document build opts for SDK vs VS better

Add KeepaliveTime and KeepaliveInterval options to specify time or
interval for keepalive.

Remove PgDtc_set_property(.., prepareRequestded, 0) which causes a crash
in case PREPARE TRANSACTION error. The property prepareRequested isn't set
anywhere.

Fix driver name mismatch between 32-bit ODBC app and 64-bit MSDTC host

If the 32-bit driver runs on a 64-bit host within a 32-bit app
running under SysWow64 then XA transactions will fail to recover.

That's because recovery is done by pgxalib.dll which runs within
MSDTC.exe - a 64-bit application. It connects to a DSN supplied
by psqlODBC in EnlistInDtc_1pipe during DTC enlistment by the
XA resource manager. This DSN uses the driver name of the client's
ODBC driver - which is fine so long as the client and MSDTC
architectures match. If they don't then MSDTC will fail to connect
to Pg, so it'll never resolve transactions.

Traces of MSDTC will show RM_COMMIT_DELIVERY_FAILED_DUE_TO_CONNECTION_DOWN
events.

Traces of pgxalib.dll will show xa_recover(..) being entered, followed by
"SQLDriverConnect return=-1".

To fix this, we detect if we're running under SysWow64 by examining the
PROCESSOR_ARCHITEW6432 environment variable and supply the 64-bit driver name
if so.

See mailing list thread:

  http://www.postgresql.org/message-id/53A45B59.70303@2ndquadrant.com

for details.

Craig Ringer

Fix bug with UseDeclareFetch=1 when a transaction is committed before fetch.

If a server cursor is closed before the application has fetched any rows
from the result set, the "base" of the result set's cached rowset was
off-by-one.

Also add a regression test for the same.

This fixes the bug reported by Jan-Peter Seifert.

Remove unused function.

Previous commit added a void return to a function with char * return value,
which generated a warning. On further inspection, the whole function is
used nowhere, so just remove it rather than try fixing the warning.

This patch fixes a NULL pointer dereference occurring when connecting
using SSPI authentication (without kerberos, a local Windows user trying
to connect to postgres instance on same host). Patch originally by
Nikhil R Deshpande, 1 line added for safety by me.

Fix compilation on Windows, broken by my previous commit.

Michael Paquier

Refactor SQLCancel() and add comments, for readability.

Fix bug in SQLCancel().

If one thread was busy executing a savepoint statement, while another thread
calls SQLCancel() on the statement, SQLCancel() would try to release the
locks held by the other thread. That's extremely racy; if the other thread
tries to increment/decrement the lock_CC_for_rb counter at the same time,
we could end up releasing the lock to many times or too few times. Also,
releasing a pthread mutex from a different thread than the one who acquired
it is undefined behavior.

AFAICS the DiscardStatementSvp() call in SQLCancel() can be safely removed.
It's outright wrong and dangerous if the statement is busy executing in
a different thread, and the other codepaths in PGAPI_Cancel() already call
DiscardStatementSvp().

Bug report by Jade Koskela.

Fix broken buildX86-installer.ps1 due to the change to use psqlodbcm_cpu.wxs instead of psqlodbcm.wxs.

Don't use PG_VERSION_xx() macros in original_CC_connect() because
it's not obtained from the server yet. This may fix another cause
of the problem Craig Ringer reported.

Merge remote-tracking branch 'ringerc/fix-febe-protocol-desync-emptyquery'

Fix use of server-queried version before connection

psqlODBC attempted to use the version that's queried from the server before it
was obtained from the server during the connection setup. Because the server
version is initialized to 0.0, tests for "server version >= 7.4" or "server
version >= 6.4" using the PG_VERSION_LT macro were always returning false.

Instead, use PROTOCOL_64 and PROTOCOL_74 tests against conninfo. These don't
offer any "greater than" form, so just test both. There's little point adding
anything prettier when it can all be removed when v1/v2 protocol support is
removed soon anyway.

The main symptom of this issue was that psqlODBC was sending an invalid
protocol message for the v3 protocol, "Q \0", causing the server to wait for
the second half of the length word forever. At connection close, the server
would emit:

    unexpected EOF within message length word

This issue was observed while tracking down problems with XA transactions in
MSDTC failing to commit after a successful prepare because an isolated tx
created via getLockedXAConn() got stuck here, but is not confined to that
problem.

Fix the problems using SQLFetch on prepared INSERT with RETURNING clause reported by John Smith.

Clear errors for prepared statement for subsequent exections.

Increase digits used to convert from REAL/DOUBLEs to strings so that the reverse
 conversions can recover original values. Patch originally by John Smith, slightly changed by me.

SQL_NUMERIC_STRUCT was not available until ODBC 3.0

Also add a prototype for ResolveNumericParam for readability.

Fix access to unallocated memory in IAsyncPG

IAsyncPG was deleting its self, then returning an instance variable
by value. This is probably going to be safe pretty much all the time,
but it's still a memory error, and DrMemory (a Windows valgrind-alike)
complains about it:

Error #4: UNADDRESSABLE ACCESS: reading 0x00b1b2b0-0x00b1b2b4 4 byte(s)
# 0 pgenlista.dll!IAsyncPG::Release   [c:\users\administrator\desktop\psqlodbc\msdtc_enlist.cpp:352]
# 1 pgenlista.dll!CloseCommitThread   [c:\users\administrator\desktop\psqlodbc\msdtc_enlist.cpp:971]
# 2 MSVCR120D.dll!_callthreadstart    [f:\dd\vctools\crt\crtw32\startup\thread.c:255]
# 3 MSVCR120D.dll!_threadstart        [f:\dd\vctools\crt\crtw32\startup\thread.c:237]
# 4 KERNEL32.dll!BaseThreadInitThunk +0x11     (0x7768338a <KERNEL32.dll+0x1338a>)
Note: @0:00:57.879 in thread 4516
Note: next higher malloc: 0x00b1b420-0x00b1b440
Note: 0x00b1b2b0-0x00b1b2b4 overlaps memory 0x00b1b2a0-0x00b1b390 that was freed here:
Note: # 0 replace_operator_delete                              [d:\drmemory_package\common\alloc_replace.c:2524]
Note: # 1 pgenlista.dll!IAsyncPG::`scalar deleting destructor'
Note: # 2 pgenlista.dll!IAsyncPG::Release                      [c:\users\administrator\desktop\psqlodbc\msdtc_enlist.cpp:341]
Note: # 3 pgenlista.dll!CloseCommitThread                      [c:\users\administrator\desktop\psqlodbc\msdtc_enlist.cpp:971]
Note: # 4 MSVCR120D.dll!_callthreadstart                       [f:\dd\vctools\crt\crtw32\startup\thread.c:255]
Note: # 5 MSVCR120D.dll!_threadstart                           [f:\dd\vctools\crt\crtw32\startup\thread.c:237]
Note: # 6 KERNEL32.dll!BaseThreadInitThunk                    +0x11     (0x7768338a <KERNEL32.dll+0x1338a>)
Note: instruction: mov    0x10(%eax) -> %eax

This patch fixes the issue.

ctype_length() is missing the length for SQL_C_BIGINT and SQL_C_NUMERIC(patch by John Smith <nukemd@hotmail.com>).

Discuss issues with SDK support and installation

Document build outputs

The build didn't explain what went where, or what to do about dependencies.
Improve the docs to cover the topic.

Fix the warning MSB8028: The intermediate directory (..\x86_Unicode_Release\) contains files shared from another project (pgxalib.vcxproj).  This can lead to incorrect clean and rebuild behavior.

It's safe to add this change in addition to the previous commit.
d0c993d3f6f0a4496a20486f2ec19b14591baba7
Author: U-Lenovo-PC\hiroshi <hiroshi@Lenovo-PC.(none)>
Date:   Sat Jun 14 21:53:57 2014 +0900

    Fix potentially uninitialized local pointer access which causes
    fatal compilation errors on vs2012/2013 reported by Craig Ringer.

Fix potentially uninitialized local pointer access which causes
fatal compilation errors on vs2012/2013 reported by Craig Ringer.

Add top-level Windows makefile targets for building installers.

You can now do "nmake /f win64.mak world" to build both x86 and x64
versions of the installers in one command.

Make sure that the psqlodbc ANSI driver is ODBC3.5.

Changes by a patch and a request from Craig Ringer <craig@2ndquadrant.com>.
Applied a patch [ODBC] PATCH: WiX has moved.
Register 64bit XA DLL on installation per discussion with him.

Fix a PowerShell syntax error.
Also handle the case when GSSBINDIR parameter is given as an empty string in psqodbcm_cpu.wxs.

Support running regression tests on Windows.

Some tests fail, and numeric test crashes for me, but at least it's
relatively easy to run the suite on Windows now.

Michael Paquier and me

Fix ODBC function escape for SPACE function.

Also add a regression test case for SPACE and a few other ODBC function
escapes.