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.

Correctly set returned value to an empty string in "cvt_null_date" mode.

Spotted by clang compiler warning. Also add test case for this.

Refactor and add comments for the modes ResolveOneParam can operate in.

ResolveOneParam can work in four different modes that are mutually
exclusive. Put the mode in an enum field in the QueryBuild struct,
separately from the additional flags that can be set. Add comments to
explain what each mode does.

Refactor the way CVT_SPECIAL_CHARS works with the query buffer along the
way. It was always a bit awkward, but with this refactoring it would've
become more so, as the mode would've needed to be another argument to it.

More miscellaneous cleanup.

* Simplify the way the ignore-round-trip-time config option works. (No need
  for the caller of CC_send_query_append to pass it as argument,
  CC_send_query_append can look at the options in the connection object
  itself)
* remove unnecessary local variable used for "fetch" constant.
* remove unused FLGP_BUILDING_PREPARE_STATEMENT flag.
* remove unused parameter, and check for stmt==NULL, from QB_Initialize.
* inline process_statements function into its only caller.

Miscellaneous cleanup.

* Mark functions used only in same file as static.
* Remove unused 'encoding' argument from msgtowstr and wstrtoms (all
  callers passed NULL).
* Silence remaining compiler warnings on 64-bit Windows about assigning
  size_t to int without cast.
* Remove superfluous FORMAT_INT4/FORMAT_UINT4 defines.
* Remove useless dollar_quote argument from findTag (it was always '$'), and
  add comment.
* Turn OID_NAME into a #define, to avoid a useless sprintf.
* Turn the mapFuncs array into structs, for clarity, and add comment.

Remove "Cancel as FreeStmt" from docs, too.

Check for PQsetSingleRowMode function in configure test.

We require libpq 9.2 (or newer), where PQsetSingleRowMode was added. Check
for that in configure, so that you get a nice error message if you try
build with anything older.

Add expected output for running on a database initialized with C locale.

Remove default for Port attribute.

The default is to use whatever the libpq's default is. Makes no difference
under normal circumstances, as 5432 is the libpq default too, but it can
be changed if you make a custom build of libpq.

While at it, fix a typo in error message.

Don't put default Servername and Port in the odbc.ini file.

This way, the driver will pick up the defaults from the libpq environment
variables, which makes more sense as we rely on those for the 'sampletables'
test case anyway.

Expand colattributes test case a bit.

Test SQL_DESC_TYPE_NAME. There was an old report in the pgfoundry bug
tracker that it didn't work correctly with interval types, (#1010745, from
Farid Zidan on 2009-12-30), but it seems to work now.

Improve print_diag() regression test helper to print all diagnostic records.

Previously, it only printed the first record. This is academic at the
moment, as the driver never reports more than one record AFAICS, but let's
be prepared.

Silence regression failure of diagnostic test.

Now that I fixed the bug in the error handler, we're not losing the libpq's
message about lost connect anymore. It overwrites the previous FATAL error,
which is unfortunate, but it's been long-standing behaviour that if multiple
errors are returned as result for a query, only the last one is kept.

Fix references to uninitialized memory.

Some variables that were supposed to be set by sscanf() were accessed before
checking the return value of sscanf().

Spotted by valgrind on the result-conversions regression test.

Fix uninitialized memory access.

Introduced by the big libpq patch. Before that, the keyset array was
allocated - and zeroed - in another function.

Spotted by Valgrind.

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.