Fix reference to free'd plan name.

libpq_bind_and_exec fetched a pointer to the statement's plan name to a
local variable, but then called prepareParameters which free'd it, and
allocated a new plan name. The already-free'd name was then passed to
PQexecParams().

Spotted by valgrind

Fix read-after-free bug in error handler.

If a connection dies, CC_on_abort() calls PQfinish() which frees the error
message. handle_pgres_error better deal with the error message before
calling CC_on_abort().

Spotted by valgrind, on the "diagnostic" regression test.

Fix bug in determining the datatypes when binding.

In the big libpq patch, I accidentally removed the line where
current_exec_param was reset to 0. It's still required (which is a very
fiddly way to pass that value to ParseAndDescribeWithLibpq, but let's live
with it for now..)

Minor cleanup.

Remove local variable that was never used for anything real (just passed as
NULL to mylog/qlog). Replace with real query in the logs, and add some extra
logging.

Remove unused arguments.

Mark functions that are not used outside convert.c as static. Remove dead code.

Add test cases for conversions from string to date/time/timestamps.

Also improve a comment.

Add a few tests for conversions between binary and text.

Just noticed that it was not tested.

Remove long-since obsolete FAR/NEAR decorations from pointers.

We haven't supported a segmented memory model for ages, probably never.

Misc cleanup, mostly dead code removal.

Don't insist that Server, Username and Database options are set.

Libpq will use sensible defaults if they are not set. In particular, it's
very useful that if you don't specify a Server, on Unix systems we will now
try to connect to a local installation via Unix domain sockets. That was
difficult to configure otherwise.

Remove obsolete CancelAsFreeStmt option.

It became obsolete when support for ODBC versions < 3.51 was removed, as
it never did anything in 3.51 version.

Change URL to FAQ to point to new domain, odbc.postgresql.org.

The project's website was moved to odbc.postgresql.org.

Change links in the readmes accordingly.

Send proper datatypes to the server when binding parameters.

We used to always send 0 as the parameter type, which means that the server
should deduce the datatype from the context. That usually works well, but
we can do better. Sometimes there is ambiguity, like in "SELECT '555' > ?".
If the parameter is bound as a string, '6' would return true, but if it's
bound as an integer, it returns false.

When the application calls SQLBindParameter, it specifies the datatype, so
we should make use of that.

Add a test case for the "SELECT '555' > ?" case.

More fixes to building query with SQL_INTEGER and SQL_SMALLINT params.

Jeremy Faith pointed out that the code still incorrectly accepted '-' as
a valid integer. But a bigger issue with negative numbers is that even a
valid negative integer can lead to bogus queries. Consider "SELECT 0-?",
where the parameter is -123. It was expanded to "SELECT 0--123".

To fix, don't accept '-' as a valid integer, forcing it to be quoted. And
if the value is negative, put parens around it, so that the previous
example becomes "SELECT 0-(-123)".

Add test cases for these issues. While we're at it, improve the output of
the param-conversions-test to be more readable.

Fix passing username to the server.

In commit e85fbb2, as part of removing non-libpq code, I removed the code
that determined the username of the currently logged-in user, and send it
to the server during authentication. However, I accidentally also removed
the code that passed any explicitly-given username to the server. Oops.

Silence compiler warnings on 64-bit Windows build.

Be more careful about using size_t with variables that count offsets or
lengths of strings. In theory, a string longer than 2^31 could overflow an
int. I'm not sure if such a string can occur in practice, but let's be
tidy.

Fix Windows build.

In the previous commit, I missed a reference to the autocommit_public
field that was inside an #ifdef _HANDLE_ENLIST_IN_DTC_ section.

Don't reset autocommit attribute at SQLDriverConnect.

It should be possible to call SQLSetConnectAttr to disable autocommit
before establishing a connection. It was incorrectly reset when the
per-datasource options were read. To fix, move autocommit_public flag from
ConnInfo struct to Connection, like we've always done with login_timeout.

Also add a test case.

Reported by Ivan Zhang

Silence compiler warnings about variables being used uninitialized.

Fix various quoting issues in sending query parameters to server.

The code used to just assume that if the parameter's SQL type is SQL_INTEGER
or SQL_SMALLINT, it doesn't need quoting. While that's true for valid
integers values, there were no safeguards that the bound string is actually
valid input. The server will just throw an error on invalid input, but we
need to quote it correctly to send it to the server in the first place.
For example, if the string "123, 'inject'" is bound to an SQL_INTEGER param,
we need to quote it or the server will interpret it as part of the query.

Also add a test case for this.

Reported by Jeremy Faith.

Minor cleanup of the way binary parameter format is chosen.

Rather than have the caller guess whether ResolveOneParam chooses to use
the binary format, add an explicit return value for it.

Fix building with MSDTC=no.

Silence compiler warning about unused variable.

Use libpq for everything.

Instead of speaking the frontend/backend protocol directly, use libpq's
functions for executing queries. This makes it libpq a hard dependency, but
removes direct dependencies to SSL and SSPI, and a lot of related code.

Add a test case for error handling.

The current behavior isn't actually quite correct with some configurations,
but seems better to add the regression test now anyway, even though the
errors_2.out basically memorizes the bogus behavior. It should be removed
when the bug is fixed.

Remove spurious semicolon from the end of default ExtraSysTablePrefixes.

The code that forms a new connection string from current settings doesn't
escape it correctly, so that the connection string looks like
"...;C2=dd_;;CX=...". Of course, the code that forms the connection string
should be fixed, but that's a much bigger job.

On second thoughts, treat a bookmark as a 32-bit integer, not SQLULEN.

This seems more reliable.

Add a test case for SQLBulkOperations(SQL_FETCH_BY_BOOKMARK), although it's
disabled because it doesn't seem to work correctly (fetches wrong rows).
I don't have the energy to actually fix that now.

Fix confusion on whether a bookmark is 4 or 8 bytes long.

In some places, we used SQLULEN to refer to a bookmark, and in others,
UInt4. But SQLULEN is 8 bytes long on some platforms, which caused all
kinds of strange issues.

Also add a test case for SQLBulkOperations. This revealed the bug.

Add test for fetching deleted/updated rows in an updateable cursor.

Add test case for SQLSetCursorName and SQLGetCursorName.

The test case also contains tests for UPDATE ... WHERE CURRENT OF, but
it's disabled because the driver doesn't actually support it. I wrote the
test before I realized that, and didn't want to throw it away, in case we
ever want to add support for it.

It also contains an #ifdef'd out test for cursor names that contain quotes.
The driver doesn't handle that correctly, but I didn't want to fix it right
now. No sane application does that anyway.

Enhance the error-rollback test case.

This covers some cases that I found to contain bugs in a patch that I'm
working on. Current master is correct, but extra coverage is good.

Modify test case to call SQLPutData twice.

For increased test coverage.

Fix SQL_DESC_OCTET_LENGTH for 'unknown' type columns.

This makes 'unknown' to behave the same as 'varchar' without typmod.

Also add a regression test for it.

This addresses the bug reported originally on the pgsql-bugs mailing list as
bug #11767, by Keyur Govande.

Add test cases for:

* getting a bookmark with SQLBindCol,
* unbinding columns with SQLBindCol and SQLFreeStmt(SQL_UNBIND), and
* the procedure call "{ CALL ... }", and date, time, and timestamp escape
  syntaxes.

Misc cleanup.

Fix some misc issues pointed out by clang's static analyzer, scan-build.
Mostly dead assignments.

Add tests for Nan, Infinity, -Infinity float values.

Silence compiler warning about unused variable.

odbc.ini and odbcinst.ini are now generated, add them to .gitignore

Michael Paquier

Add a Makefile target to run the tests with different config options.

Make the regression test output independent of some settings.

Namely, UseServerSidePrepare, UseDeclareFetch and Protocol.

Run the odbc[inst].ini generation script in a more Makefile fashion.

The expected output didn't match what I got on my laptop. Not sure why,
or if it worked for anyone, but in any case it's more straightforward to
let make run the script directly, rather than through pg_regress.

In the passing, allow passing extra command line options to odbcini-gen.sh,
which will be appended to the generated odbc.ini. This makes it handy to
run the regressions with extra non-default options.

Add more test cases.

This adds coverage for SQLConnect, SQLRowCount, and moving a cursor with
SQLFetchScroll, and for bookmarks.

Remove some remnants of old protocol support

Another round of -Wformat fixes.

Fix test case for big-endian systems.

Silence more compiler warnings in -Wformat mode, in regression test.

Silence compiler warnings in -Wformat mode.

Passing SQLINTEGER/SQLUINTEGER sprintf "%d", "%ld" modifiers gave a warning
on some systems (at least on my 32-bit Debian system) .

Allow building test suite in a different place other than the source code is.
Change the test suite so that it works not only with unixODBC but also with iodbc.

Avoid buffer overrun by makeKeepaliveConnectString().

Fix Windows compilation with libpq enabled, but without SSPI

GetUserNameEx function requires secur32.dll. It is now referenced in the
code, regardless of SSPI, so we need to link with secur32.dll.

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.