Call getaddrinfo() with AI_NUMERICSERV because the service is a port number.

socket: speedup also ipv6 connection

[Patch by Pavel Raiskup]
Call getaddrinfo() with AI_NUMERICHOST if it is ipv4/ipv6 address.
For that reason the inet_pton() is better than inet_addr().

Fix memset() call, meant to clear the whole struct.

Per compiler warning.

Make it possible to invoke several scripts from any working directory.

Stop dot-sourcing a script to avoid confusion of scope of variables. Also make it possible to specify the configuration file path as a parameter.

Add formatVersion attribute to configuration.xml so as to detect the format change. Unify configuration_template.xml to configuration.xml when the format change is detected.

Remove CRs from editConfiguration.ps1.

bufferoverflowu.lib seems no longer needed in recent versions of VC environment.
 So removed it from win64.mak. Speocify CUSTOMLINKLIBS=bufferoverflowu.lib from the command line when it is neccessary in old VC environment.

Add build_macros element to winbuild/configuration(_template).xml so that buildXxx.ps1 can specify macros to the makefiles. Also make some part of CL options of makefiles replacable.

Move -Wall -Wno-pointer-sign CFLAGS option specified in Makefile.am to configure.ac. Users can disable those options by giving CFLAGS option explicitly. Also verify the availability of those options.

Stop linking lib(i)odbc library because the library is unnecessary and rather harmful. Per report from Pavel Raiskup(postgresql.org/message-id/1769926.65KUyECjFr@nb.usersys.redhat.com).

Use snprintf() instead of sprintf() for safety.

I believe these instances were in fact safe, because a cursor name has
a maximum length, but better safe than sorry.

Add regression test for SQLTables() function.

Fix buffer overflow in handling of SQLTables params.

Found by Fortify static analysis tool.

Replace calls to my_strcat and my_strcat with snprintf_add.

my_strcat didn't check for buffer overflow, which Fortify static analysis
tool flagged as an issue. I'm not sure if that can actually happen, but
the code is clearer like this anyway.

Check return value of stdup() for out-of-memory.

There are a lot of little bugs like this throughout the code, but it's
start...

Flagged by Fortify static analysis tool.

Fix buffer overflow in interval parsing.

Includes a test case.

Flagged by Fortify static analysis tool.

Construct ctid string correctly for block no > 2^31.

Block number is unsigned. Other places where we construct ctid strings we
got this right, but not this one. I'm not sure how to trigger this codepath,
but I'm sure something funny would happen if you tried to use these functions
on large enough tables (> 16 TB).

Escape double-quotes in table name correctly.

When constructing the select/update/delete for current row in a rowset,
and the schema or table name contained double-quotes, they were not correctly
escaped in the constructed SQL statement. That lead to errors when doing
positioned updates.

Also, use snprintf and snprintf_add instead of sprintf and strcat in more
places. Makes these things look less like buffer overflows to static analysis
tools, and make for more readable code anyway.

Add test case for positioned updates (SQLSetPos).

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.