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.

Silence compiler warnings from regression tests.

Three categories of warnings:

1. Missing return at end of main() function
2. Unused variables (copy-pasted from another test)
3. Data type mismatch in printfs, e.g. passing SQLULEN param to %d

Oops, forgot to git add some last-minute changes to the new regression test.

Rewrite the conversion functions between strings and SQL_NUMERIC_STRUCT.

Aside from making the functions more readable and faster, this fixes
a number of bugs:

1. When converting a large number from decimal to binary, the least
significant byte might be wrong (reported by Walter Couto)

2. In binary->decimal conversion, a number with small precision but large
scale caused a "Floating point exception".

3. In binary->decimal conversion, building the final string was wrong, picking
up "digits" from outside the reserved stack space, if scale was larger than 40

4. The param_string buffer allocated for the result of binary->decimal
conversion was too small.

Also add a test cases for all of these bugs.

Minor code cleanup.

The main change is to move the responsibility of adding the "length" header
to the QueryBuild from ResolveOneParam to its caller, when constructing the
parameters to send over to the server in a Bind message. ResolveOneParam
is complicated enough without that, so it seems more clear this way.

The rest is just fixes to the macros to make them deal with a "&qb" argument,
by adding parens around the macro argument. And while at it, clean up the
whitespace in them.

Misc whitespace and comment fixes. Remove dead, commented-out code.

For sake of readability.

Fix crash if connection is closed while sending a query to the server.

This is a similar bug that was fixed in commit 22b151eb. Some functions
reset conn->sock to NULL on a socket error, so we must not keep a copy of
it in a local variable. Also, SOCK_get_id should check for NULL argument
like all the rest of the SOCK_get/put_* functions.

Remove unused NDEBUG and PSQLODBC_EXPORTS defines from makefiles.

Remove more unused stuff from win64.mak

Silence misc issues flagged up by Clang's static analyzer, scan-build

Using tolower() requires ctype.h with some compilers.

gcc and MSVC were happy without it, but clang warns.

Merge 32-bit and 64-bit .wxs files used to build the .MSM files on Windows

Unify the 32-bit and 64-bit installer build processes.

Use the same .wxs file for both builds. The MSM files is still generated
differently, but this is a step in the right direction.

Fix Debug-mode builds with win64.mak

It apparently hasn't worked for a very long time. Copy some tricks from
the already-deleted win32.mak to make it work: Use /MDd rather than /MD
when compiling in Debug mode, and replace /GZ with /RTC1, to silence a
deprecation warning from the compiler

Fix Windows build with MSDTC=no.

Rename Windows output folders, for clarity.

Remove obsolete Windows build files.

buildx86.ps1 and buildx64.ps1 were used to invoke the nmake makefiles, but
have been obsoleted by the scripts in winbuild/. win64.mak can be used to
build 32-bit and 64-bit binaries now, so win32.mak is no longer needed.

Simplify win32.mak and win64.mak a little bit.

Change the way we test whether a directory exists, so that we don't need
a version check for Windows NT. (although we probably don't need to support
compiling on NT anymore anyway)

Improve Windows compilation instructions in the docs.

Refactor win64.mak to allow passing options in local file.

Move all the default values for external library paths to
windows-defaults.mak. Add an include for file called windows-local.mak,
this allows you to put local config options in that file, which is much
easier than passing them on the command line every time.

Remove /incremental option from linker call on Windows / nmake

The linker defaults to what were passing, so should work fine without it.

Also remove an unused variable from win32.mak

Remove special check for nmake version 6.00.9782.0

Surely we don't need to support that old versions anymore. Besides, it
was bizarre to check for that exact version, and not any older or newer
versions of that era.

Remove unused cruft from Windows build files.

Add comments to modify_msi.vbs.

It took me quite a while to reverse-engineer what it does, and more
importantly, why. Add comment to save time for the next reader.

Clean up Windows makefiles

Get rid of the CPU and CPUTYPE variables in win64.mak. Instead, assume that
the TARGET_CPU variable has been set correctly in the environment.
TARGET_CPU is set by "setenv" script that comes with the Microsoft Visual
Studio C compiler package. It never worked to pass a "CPU=x86" and using a
x64 compiler, for example, so seems better to determine the correct target
from the environment, than fail with a cryptic error message if the CPU
argument and the environment don't match. To build 32-bit binaries with
win64.mak, simply do "setenv /x86" before running "nmake -f win64.mak".
Similarly, to build 64-bit binaries, use "setenv /x64".

It should now be possible to build both 32-bit and 64-bit binaries with
win64.mak. Ideally, we could now remove win32.mak altogether, but there are
small differences between win64.mak and win32.mak in the build options that
they use. I don't understand the differences enough to dare to remove
win32.mak yet.

The CPU=AMD64 alias for x64 is no longer supported (as the whole parameter
is now gone).

Also do whitespace & other cosmetic fixes to win32.mak and win64, to make
them more similar. This helps when comparing the two with "diff".

Remove remains of the long-gone "MULTIBYTE" setting (it was misspelled in
win64.mak, anyway)