The third stricmp() in writeDriverCommoninfo() is useless, because the second one causes the function to return.
SC_get_errormsg() definition is repeated twice in succession.
In info.c:SQLGetInfo(), "len = 0" is unnecessary because len is assigned at the end of the function.
In execute.c:PGAPI_ExecDirect(), flag doesn't need to be set as its value doesn't change.
PODBC_PER_STATEMENT_ROLLBACK is not used.
[patch by Tsunakawa, Takayuki]

SQLS(G)etDescRecW instead of SQLS(G)etDescRec for Unicode driver.

There's no msvcrxxx,dll in VC14.

In Windows the use of external variables timezone, tzname and daylight are deprecated. Use _timezone, _tzname and _daylight instead.

Fix a parameter unmatch of snprintf.

LINKMT is not used in the build process.  ADDL_INC should be ADD_INC.  Just a typo.  The CLEAN target doesn't delete .pdb files. [patch by Tsunakawa, Takayuki]

Fixed a typo.

The statement linked list is not properly chained, which throws away some statements. [patch by Tsunakawa, Takayuki]

The existing memory is not freed upon realloc() failure.
PGAPI_BindCol() doesn't check the failure of extend_getdata_info().
[patch by Tsunakawa, Takayuki]

setup.c:test_connection() calls EN_Constructor() twice, leaking small memory. [patch by tsunakawa, Takayuki]

Checking a flag bit mistakenly resets other flag bits. [patch by Tsunakawa, Takayuki]

Correctly judge the SQLSTATE 01S06(Attempt to fetch before the result set returned the first rowset) for SQL_FETCH_PRIOR operations of SQFetchScroll.

Oops, don't forget to set cmd_fetch_size ( and cache_size) of QResultClass objects.

Add test declare-block-test.
It contains 3 cases which correspond to bugs introduced in 9.05.0100. They are all in declare/fetch mode.
1. Using block cursors.
2. Scroll BOF -> EOF -> BOF ...
3. When fetching a row "behind" the rowset by "fetch absolute", how many rows can be fetched?

CHAR causes a compilation error in some environment. Use char instead of CHAR.

Though I made it possible once to build test suite in a different place other than the source code is, it's not now. I would make it possible again.

Refine SQLGetData().
1. Unify midtemp and moneytemp.
2. Variable wconverted is no longer needed.
3. Pre_convert for PG_TYPE_BOOL is not needed.
4. It's better to take the same road even in case of pre_convet.

Correctly handle SQLGetData() when the target type is SQL_C_WCHAR and the buffer is small to get the whole data for some data types such as bool, date, time ... etc.

Make sure that EOF occurs when 'fetched < fetch requested'.

Don't forget to update num_total_read field of QResultClass record in case 'fetch 0 rows' after 'move'. This fixes the bug 'When fetching a row "behind" the rowset by "fetch absolute" only the first ones of the result set can be fetched.' reported by Bernhard Goebel.

Consider decimal_digits of SQL_TIME (and time field of PostgreSQL).

Fix the bug about MONEY type.

Correct the detection of eof in useDeclare/fetch mode. This change fixes the bug about fetching prior after scrolling next until eof reported by BenS.

Don't initialize next_in_cache or rowstart_in_cache info when scrolling (block) cursors in declare/fetch mode. This fixes the bug reported by John Kew.

Fix NULL dereferences which causes application crash. Patch by Tsunakawa, Takayuki.

Correct a cast.

Add support for VC14 build.

Clear the parsed result before executing simple query so that SQLRowCount() returns a proper result. This fix the bug reported by Walter Willmertinger.

change distfiles.

Prep Release 09.05.0100.

Prep Release 09.05.0100.
release note by Heikki Linnakangas.

Fix handling of return value in {? = CALL ... } syntax.

When sending the query to the server, need to use the number of parameters
calculated by build_libpq_bind_params(), which excludes the OUT parameter,
rather than the original number of parameters. The branch that executes
an already prepared statement got this right.

There was one test query for the CALL syntax already, but it was commented
out, because it doesn't work for unrelated issues. Add a simpler test case
that does work.

This was broken by commit e6d7ded, in version 09.03.0400. The code was
heavily refactored in commit e85fbb24, but that refactoring failed to fix
this issue.

Reported by Haribabu Kommi.

Let the server decide if SQL_LONGVARBINARY means a bytea or large object.

This reverts the behaviour of SQL_LONGVARBINARY parameters to the way it was
before commit 1ccd04c70b8037b268eefcaae09ce131c8c33b59. We can't reliably
guess in the driver whether the parameters should be sent to the server
as 'bytea' or as a large object, so send it as 'unknown' and let the server
decide.

Diagnosis and original patch by Hiroshi Inoue, I only added the comment.

Fix SQL_DATA_AT_EXEC processing for large objects.

Hiroshi Inoue identified another bug with large objects, with
ByteaAsLongVarBinary=1, but I'll deal with that separately.

Reported by Vadym Krevs.

Call dumpbin recursively so that indirectly linked dlls can be detected.
For example, when libintl-8.dll links libiconv-2.dll, the installer has to
install libiconv-2.dll as well.

Correct the precision of a timestamp representation.

Oops, PROCESSOR_ARCHITECTURE of 64bit OS is AMD64 not x64.

Remove an unrefereced variable.

It's not good to pass path name of files in system folder to candle.exe because the file redirection may occur. Instead we specify system folder's name at .wxs.

Remove useless statements.

If libpq needs different version of MSVC runtime from that psqlodbc
needs, install it also.

Improve the way to intall libpq related modules.
Instead of describing fixed items in a .wxs file, automatically detect
what modules to be installed.

Improve parsing of INSERT INTO statements to support @@identity

Patch by Christian Ullrich, some extra comments and regression test by me.

Set an error message when password is required to connect.

In previous versions, if a connection fails because the server requires a
password and no password was supplied, you got a "fe_sendauth: no password
supplied" message from SQLGetDiagField(). That got lost at some point, so
put it back, as "Password required".

Per report from "PSequel Support".

Remove obsolete references to pg_regress from README.txt.

The regression tests no longer use pg_regress, since commit 2002b71d.

Fix buffer overrun in constructing error message.

If the server returned an error longer than 4096 bytes, we would overrun
the buffer by two bytes. If you're unlucky, that could lead to a crash,
and it consistently did on Windows.

Per report from Andrus Moor.

Fix autoconf test for whether SQLColAttribute's argument is SQLLEN *.

There was a spurious < character, which made it fail with some compilers.
While we're at it, reformat, add a "checking for" message for that test,
make it a bit more robust by defining the function's body and using the
pointer too.

Per report from Mark Hellegers

Release critical section if SQLAllocHandle() fails.

Don't define strlcat prototype if the system has strlcat()

The #ifdef was wrong.

Hopefully this fixes the build failure on OS X Maverick that Alex Dunn
reported.

Silence compiler warning about unused variable

Fix description of error in comments.

I misunderstood the search_path and current_schema() issue.
current_schema() works fine if search_path contains more than one element.
It just returns the first one in that case. However, I set the search_path
to an invalid value, which just happened to look valid.

Fix segfault when "SELECT current_schema()" returns NULL.

current_schema() can be NULL, for example if search_path contains more
than one schema. CC_get_current_schema() segfaulted on that. Also fix all
the callers to expect that CC_get_current_schema() can return NULL.

Per report from Frank van der Aa.

Initialize parameter type of NULL parameters correctly.

When building the arrays of parameter types, formats, etc. for sending to
the server with PQexecParams(), the parameter types for NULL parameters
were not set. This caused "cache lookup failed for type XXX" errors.

Add test case for SQLBindParameter with NULL params.

Fix regression test to use correct CHECK macro

Fix typo in connection string used in regression test.

Fix minor memory leaks.

Per Coverity warnings.

Remove dead code in ColAttSet.

This is dead code because the caller of ColAttSet never passes 'rti' as
NULL.

Spotted by Coverity. Patch by Michael Paquier.

Remove dead code

Spotted by Coverity. Patch by Michael Paquier.

Remove dead code.

Pointed out by Coverity as 13523.

Michael Paquier

Remove some dead code.

gpps.h was removed back in 2004.

Refactor regression tests to use a custom program instead of pg_regress.

Using pg_regress was awkward, because it used a dummy .sql file for each
test that just launched the test program. Also, it added a dependency;
the driver doesn't require PostgreSQL sources to build, just libpq
headers, but the pgxs program is not usually shipped with libpq headers.

The new "runsuite" program runs all the test programs given on the command
line, compares the results with the expected output, and produces
TAP-compatible output. It can be run stand-alone, as the TAP output is
human-readable, or with the perl 'prove' program which gives more pretty
output.

Use _MSC_VER to check for existence of isinf/isnan.

Let's use this variant of the check that's also used in the main PostgreSQL
sources. It has been shown to work, so should be more robust.

Per suggestions from Michael Paquier and Tom Lane.

A better fix for isnan() and isinf() on Windows.

Newer versions of Visual Studio actually have isnan() and isinf() macros.
Silence compiler warnings about redefining them.

Fix typo in README.

Michael Paquier

Fix crash when a query is described, without binding params.

Add test case for the same.

Don't do premature execution.

Use protocol v3 Parse instead, even in UseServerSidePrepare=0 mode.
Remove the DisallowPremature config option.

Fix building on Windows systems without isinf()/isnan().

Michael Paquier

Remove unused variable.

Left over from the switch to PQconnectdbParams.

Cleanup after test, to silence Valgrind warnings about leaked memory.

Use PQconnectdbParams for connecting.

This makes it unnecessary to build a connection string, which was broken
anyway. (Try connecting with a username that contains a single quote and
a space).

Simplify the way the connection string is built.

Silence Coverity warning about unsafe use of strcpy

We know that pg_attribute.attnum value is an integer that fits in 16 bytes,
but Coverity doesn't. And of course it's nicer to not crash if the backend
is crazy and returns garbage..

Use fixed-width fields for SQLState values.

I'm hoping that this silences a Coverity warning, making it realize that
none of the strings in the array are longer than 5 bytes.

Silence Coverity warnings about derefencing NULL pointer.

Fix parsing of escaped quotes.

The driver-side SQL parsing code didn't recognize quotes and double-quotes
escaped by doubling them correctly. Also add a test case that fails without
this fix.

Disable LF <-> LF+CR conversion in the param-conversions test.

This silences a silly regression failure on Windows. There is a separate
test for LFConversion, so let's just rely on that.

Make boolsaschar test case insensitive to whether text maps to CHAR or WCHAR.

There's no need to print the type of the text column, it's not what
boolsaschar test case is testing.

This fixes regression failure on Windows with Unicode driver.

Use a larger buffer for error message.

This allows us to get a large NOTICE message. The driver limitation that the
comment in notice-test.c talks about has been lifted, remove comment.

Remove unneeded tests from configure script.

Teach deprecated-test regression test about keyset-type cursors.

Also use SQL_CURSOR_TYPE instead of SQL_ATTR_CURSOR_TYPE in the
SQLSetStmtOption calls. Although they both have the same value,
SQL_CURSOR_TYPE is what it used to be called back when SQLSetStmtOption
was not deprecated yet, so that's what a genuinely old application would
use.

The printf() output format of infinite/nan doubles varies across platforms.

Notably, it's different on Windows. Check for infinite/nan values explicitly
in the regression test, to avoid platform-dependent output.

Windows _snprintf is not a drop-in replacement for POSIX snprintf()

It handles truncation differently. Witnessed by failures in the
result-conversions regression test on Windows. Create a shim function that
uses _snprintf() to implement the POSIX truncation behavior (as long as you
don't run out of memory...)

Use correct handle when getting error message in regression test.

I hadn't noticed that the error message on procedure call didn't complain
about the invalid function. Also, on Windows, apparently the error message
isn't automatically copied from statement to connection handle, because
I got a diff.

Teach SQLDriverConnect that Port is no longer required.

This caused SQLDriverConnect to unnecessarily fail or open a connection
options dialog if no Port was specified.

Remove dead code.

Also move some stuff that's not used elsewhere into multibyte.c

Remove dead NULL-check.

Per Coverity warning, res-keyset is accessed earlier, so can't be NULL.

Don't crash on NULL result from CC_send_query

Per Coverity warning. There are probably dozens more cases like this that
it doesn't happen to warn aboute, but let's reduce the noise anyway.

Fix memory overrun bug with large keyset-driven result sets.

The address passed to memset() was bogus, resulting in clobbering random
piece of memory.

Found by Coverity. Add a somewhat contrived test case for this too.

Remove a few more unnecessary NULL-checks.

The only caller to QR_next_tuple always passed non-NULL.

Remove unnecessary null-check.

Per Coverity warning.

loadlib.c is only needed on Windows.

Remove dead function.

Use test -x rather than test -f to check the [i]odbc_config program.

I copied the odbc_config_dummy version from the Debian system, but forgot
to make the script executable. Configure failed, but it didn't complain
about odbc_config, but it claimed that "libpq-fe.h" could not be found.
Turns out that it tried to execute odbc_config --include-prefix, which
failed, leaving the variable empty. In subsequent tests, it ran gcc like
"gcc -I -I/usr/include/postgresql". With this fix, you'll get an error
about odbc_config.

Fix miscellaneous little issues spotted by Coverity.

A few were real bugs, although highly unlikely ones. Others were false
positives like dead code, but let's be tidy.

Remove unnecessary argument from schema_strcat1 and my_strcat1.

The length was always passed as SQL_NTS.

Fix test case to work with standards_conforming_strings=off.

With standards_conforming_strings=off, one \ in LIKE pattern becomes four.

You could get a segfault if the pattern contained a lot of \ characters.
Add test case for the bug.

Check return values of a few function calls.

While we're at it, fix spelling of the error messages.

Clang static analyzer complained.

Silence a few harmless warnings that clang static analyzer gave.

Fix bug in detection of E'' literals.

The code assigned to escape_in_literal local variable when it saw a quote
characeter after E. And checked the local variable on subsequent iterations.
But the function is called separately for each processed characters, so the
local variable was reset between each call. Move the flag to the QueryParse
struct so that it survives across calls.

Expand the 'quotes' test case to cover this.

Clang Static Analyzer warned that the escape_in_literal variable was set
but not used, which lead me into looking at this more closely.