Add more test cases for ODBC catalog functions.

Michael Paquier

Add test/Makefile to .gitignore.

It's generated from test/Makefile.in now.

Use the PG_CONFIG setting from ./configure cmd line in regression tests.

If compiling without libpq (configure --without-libpq), then you still need
to have pg_config in path to build the regression tests, or specify
PG_CONFIG at the make command line.

Replace NOT_USE_LIBPQ with USE_LIBPQ #define.

This avoids the cumbersome double-negatives, "#ifndef NOT_USE_LIBPQ",
making the code easier to read.

Use the same CFLAGS/LDFLAGS to build regression tests as used to build driver.

Add missing function declarations.

These are not actually used outside of pgtypes.c, but let's be consistent
and declare them in pgtypes.h anyway.

Michael Paquier

Use "function(void)" instead of "function()" to declare 0-arg functions.

"function(void)" is the correct syntax in a function declaration, although
in practice compilers accept the latter too. It's OK in a function
definition, but change those too for consistency.

Michael Paquier and me.

Remove a couple of unused functions, and fix a typeo.

More whitespace cleanup.

Make whitespace and indentation more consistent in SQL_API function
declarations.

Whitespace and indentation fixes.

This also changed the file permissions stored in git for a couple of header
files, removing the executable bit. Didn't realize they were different from
all the others, but apparently my editor decided to change them. Now
they're the same as all the other files, which seems good.

Michael Paquier and me.

Pass the argument to isalpha/isspace etc. routines as unsigned char.

Per the C standard, the routine should be passed an int, with a value that's
representable as an unsigned char or EOF. Passing a signed char is wrong,
because a negative value is not representable as an unsigned char.
Unfortunately no compiler warns about that.

Fix implicit casts between SQLCHAR and char. Don't rely on -Wno-pointer-sign.

Prep release 09.03.0210.

Add a .gitignore file to ignore test/results dir.

Michael Paquier

Use "exit(1);" instead of "return;" to exit from main().

Per Michael Paquier, at least some compiler on OS X didn't like it.

Remove some dead code.

Pass the content of POSTGRES_RESOURCE_VERSION variable to the resource
compiler correctly so that FileVersion and ProductVersion are properly set.

Silence another compiler warning.

conn_settings cannot be NULL here.

Silence compiler warning.

Michael Paquier

Record the result of Build in configuration.xml.

It's preferable to leave PRODUCTCODE history.

Add test case for CTE queries (WITH ...)

Tests for the bug fixed by commit b547553233b52fa5a84c24801c468530d8dd9a18.
None of the existing test cases covered that, so seems good to add one. This
test case gives a different error message than the original one, but it's
the same underlying issue.

Fix encoding of release-7.3.html

The Content-type meta tag claimed it to be US-ASCII, but there was in fact
one non-ASCII character in there. Change the claimed encoding to UTF-8, and
fix the special character to the right byte sequence in UTF-8.

Convert line-endings from CT+LF to LF.

All the other files were using LF, so let's be consistent.

Fix locking in SC_set_prepared

Commit 2f4f8eabf6ce685c38d69da3e98132b77d38d188 added ENTER/LEAVE_CRIT_CS
calls in SC_set_prepared, as a belt-and-suspenders fix for the race condition
already fixed by commit 7065a7af196bb09f80beddce82fa728c8a6dfcba. But it
didn't take a NULL connection into account.

Update EXTRA_DIST in Makefile.am

Files for the new lfconversion test case were missing. Also, move bindcol
in the right place to keep the list in alphabetical order.

SSL verify[-(ca|full)] is avaiable since 8.4. There seems no need to
check it. Also there's no need to call lt_dlopen currently.

Currently PQConnectdbParams() not yet used anyway.

change version number.

change version number.

change version number.

Ooops missing change.

Forget win_md5.c.

Prep release ver 09.03.0200.

Prep release ver 09.03.0200.

Additional modules, and etc.

winresrc.h instead of winres.h.

Change to supply non-NULL parameters for SC_set_error().

It's safe to lock SC_set_prepared() itself.

The next release allows minor version up from 09.03.0100.

Fix compilation errors caused by the commit 3116f70ab51d0218ceff241fd1323f436bb4fd60.

The 2dn argument of SC_set_prepared() is int(enum) not BOOL.

Fix the judgement condition of minor or major upgrade.

Add locking to SQLFreeStmt and SQLFreeHandle.

This fixes a race condition, where SQLFreeStmt is called while the
connection is busy executing another statement. SC_set_prepared would see
that the connection is busy (CONN_EXECUTING), and not issue a DEALLOCATE
statement to free the prepared statement in the backend, leaking it.

Refactor utf8_to_ucs_lf.

A macro is difficult to debug, so turn it into a regular function.

Also, add a new test case for LF->CR+LF conversion, to test the bug that
Nils Gösche reported (which was already fixed).

Revert "When LF->CR+LF conversion causes an buffer truncation, supress the conversion (in case of unicode)."
This reverts commit 3666c87c1440862bde2e6b8f43ee585deed70d31.

The driver takes SQL_C_LONG to mean SQLINTEGER rather than "long".

The regression test was failing on the s390x architecture because of that.
It's big-endian, with sizeof(long) == 8.

Add regression test for SQLBindCol.

We were already doing SQLBindCol as part of the positioned-update test, but
seems good to have one explicitly for it.

Fix cursors test case on big-endian systems.

The SQL_CURSOR_COMMIT_BEHAVIOR property is a SQLUSMALLINT, not SQLUINTEGER.
On a little-endian system, you wouldn't notice, provided that the target
variable was initialized to 0 before the SQLGetInfo call.

Per report from Christoph Berg that the cursors test was failing on mipsel
and other big-endian architectures.

MSBuild instead of nmake to build binaries using 10.0 or later version of VC.
See readme_winbuild.txt or winbuild/readme.txt about the use.

Set TCP keepalive by default.

Fix out-of-bounds write when Bind request packet size is a multiple of 4096.

Also add a test case to reproduce that bug on unpatched version.

Report and analysis by Patrice Dardoize.

Fix off-by-one error in parsing ExtraSysTablePrefixes and table types.

I fixed these already once, but it was still off-by-one. Since we've been
burned by this twice now, refactor the code to make it more readable. Also
add a regression test case for the table types. The prefixes code is more
difficult to test since it's only used when connected to a pre-7.3 server.

Per report from Pavel Raiskup with help from Coverity static code analyzer.

Remember "Description" DSN attribute

DSN attribute "Description" was not being copied from the input during DSN
creation in ConfigDSN(). This attribute is now copied into connection info
struct (while parsing input attributes) so that it is later saved (either in
File DSN or user/system DSN entry in Windows registry).

Patch and diagnosis by Nikhil Deshpande

Update link to MSDN appendix on ODCB scalar functions.

Mike Landl

Update GUI to edit configuration.xml.
Added items corrsponding to setvcvars elements.

The environment variable env:ProgramFiles(x86) in PowerShell must be enbraced.

The value x64 for PROCESSOR_ARCHITECTURE on Windows is unappropriate.

increment of build number.

Fixed some typo.

adjustment of the path name.

change name libintl.dll.

Forget connexp.h.

increment of build number.

Stop using inet_pton() when it's unavailable on Windows.

Suppress Windows Compile warnings: C4018 signed/unsigned mismatch by default.

Forget modification.

/bin/sh of standard equipment.
problem in an environment of /usr/local/bin/bash.

Ooops, test makefile (not gmake) is broken.

Fix the bug introduced by
  commit d293c1bfbab69660683b07fd1fe3e0cc3cad570e
    Replace calls to my_strcat and my_strcat with snprintf_add.

Note that my_strcat1() does nothing when the 4th parameter doesn't
have a valid value.

fixed build error in WiX v3.8.
Per report ljwilson@digitalav.com.

Fixed not recognized by FreeBSD.

Prep release 09.03.0100.

Adding a split document.

Arrangement change history.

add some contents and adjustment.
Per report Pavel Raiskup.

change history(prep release)

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.