Format if-statement with commented out "then"-block differently.

To silence a Coverity warning that the semicolon might be misplaced.

Silence compiler warning that bmp variable might be used uninitialized.

It wasn't actually true, but apparently not all compiler versions were smart
enough to realize it.

Silence warning about sscanf format vs var type mismatch on 32-bit systems.

Mark these two structs as const.

It's a good habit to mark constants as such. But I'm also hoping that this
would silence Coverity from complaining that ver2str and ver3str might
not fit in 6 bytes.

Restructure < 7.4 protocol code a bit.

This avoids accessing unitialized bitmap[0]. It was harmless, as the
uninitialize value that was read was not used for anything, but Coverity
complained about it.

Initialize field to silence a Coverity false positive.

Coverity doesn't like the fact that the struct is passed to
SC_set_rowset_start without initializing rowset_start field. It's OK,
SC_set_rowset_start doesn't actually use the 'incr' value calculated from
the field if the 3rd arg is FALSE, so this is a false positive, but seems
like a good idea to be tidy here anyway.

Remove spurious 'break'.

Found by Coverity scan.

Add NULL-check, to silence a Coverity problem.

The problem is that if the loop above doesn't find a result set with
command tag "FETCH ...", res is left to point to an already-freed result
set, and we'll go ahead and call QR_set_withhold on the already-freed
struct. That can't happen in practice, because the backend does always
return a result set with "FETCH ..." command tag when a FETCH statement is
issued, but better safe than sorry.

Remove useless null-check.

We reference the same pointer without null-check elsewhere in the function,
so if it's ever NULL, we'll crash anyway. (but looking at the callers, it
never is NULL)

Found by Coverity scan.

Don't crash if getpwuid() returns NULL.

Found by Coverity scan.

Handle realloc failures gracefully.

If realloc fails and returns NULL, don't forget the old array. Also check
for overflow of num_Stmts, which is only a 16-bit int

Remove unused variable, to silence compiler warning.

Silence compiler warnings about unused msg_truncated variables.

BTW, the error and notice handling could be made much smarter wrt.
truncation. The caller of handle_notice/error_message knows the message size
in advance, so it could easily pass it down and we could allocate a buffer
of the right size, instead of using a fixed size buffer that can be too
small.

Add parens, per compiler warning's suggestion.

Silence warning about losing const in "const char *" to "char *" assignment.

Call shared library destructor at dlclose() or end of program.

For some reason, the destructor was not being called when compiled with gcc.
Not sure why, but I'm assuming it was an oversight. This silences a compiler
warning complaining that finalize_global_cs() function was unused.

Silence compiler warning by casting away a const.

Use FORMAT_ULEN to print SQLULENs.

Silences a compiler warning.

Fix typo in format string, which fixes passing an interval as param.

Compiler warned about this.

Silence compiler warning about unhandled enum values in switch.

Silence compiler warnings about variables set but not used.

Use -Wno-pointer-sign to silence a lot of warnings.

We liberally assign between SQLCHAR * and char *, which generates a lot of
compiler warnings about different pointer signedness. Seems like a lost
cause to add casts everywhere (and it might not be good for robustness
anyway, because it might mask a bug where the datatype of one side is
changed to a struct or something in the future), so just silence the
warnings.

Remove commented-out if-check, and related unused variable.

This silences a compiler warning about the unused variable.

Check BUILD_LEGACY_64_BIT_MODE instead of BUILD_REAL_64_BIT_MODE.

Since unixODBC version 2.2.13, the #define that controls whether SQLLEN
and SQLULEN are 32-bits or 64-bits wide is called BUILD_LEGACY_64_BIT_MODE,
not BUILD_REAL_64_BIT_MODE (the default changed to 64-bits in 2.2.13).
2.2.13 was released in 2006, so it's probably safe to assume that psqlodbc
is being built against that or higher.

Check SIZEOF_LONG_INT when deciding whether SQLROWSETSIZE exists.

For some reason SIZEOF_VOID_P is 0 on my system. But unixodbc's sqltypes.h
header file uses SIZEOF_LONG_INT to decide whether SQLROWSETSIZE exists or
not, so we should use the same.

If one of 'Database', 'Port', or 'Servername' is missing, make error message
more specific.

Fix reference-after-free bug.

'refcnt' is a member variable, so it mustn't be referenced after
"delete this;"

Found by Coverity scan.

Allocate right amount of space in malloc().

This happened to be just harmless waste of memory, because
sizeof(ParameterInfoClass) > sizeof(ParameterImplClass), but it was clearly
wrong.

Found by Coverity scan.

memset arguments were reversed.

Found by Coverity scan.

Fix typo in comment.

Fix references to 'notice.txt'.

notice.txt doesn't exist anymore, the copyright and license information is
in readme.txt now. Update references in comments.

Remove all CVS keywords.

Fixed EN_Destructor provided. Per Michael Kocherov.
patch by Inoue-san.

Protect shared connection list when making cleanup tasks in EN_Destructor
using the patch provided by Michael Kocherov.

adjustment build spec.

Prep release 09.01.0200

add the change history.

Take care of an environment variable PGKRBSRVNAME.

add the change history.

Fix the bug that when the show OID column option is enabled, SQLColumns() returns "oid" column info even when the column name parameter which is different from "oid" is specified (bug report from Seifert, Jan-Peter).

follow of fixed.

Fix the bug which causes a segfault in SQLSpecialColumns when table name is null string (bug report by Terrence Enger).

add change history.

missing end character.

Remove columns info of dropped tables ASAP.

Fix the bug reported by Alexandre(alepaes@aldeiadigital.com.br).
1. Don't discard the result of unnamed statements for later SQLDescribeCol or SQLColAttribute calls.
2. Set rowstart_in_cache properly when closing eof cursors.

Fix a compilation error etc when MULTITHREAD support is disabled.

Improve the handling of ARRAY type.

Fix a bug about reference count handling for columns info (Bug report by B.Goebel).

Add a missing constant.

Add a necessary break in a switch statement etc.

Use int instead of size_t when condition >=0 is used.

Prep release.

Prep release note.

1. Don't clear the columns cache info when they are referenced.
2. Correct the column size of interval types.

Prep release 09.01.0100.

Prep release 09.01.0100.

add the change history.

Improve the handling of BYTEA type.

add the change history.

Clear col_info(columns info) cache ehen DROP/ALTER TABLE is called
so that later SQLDescribeCol() etc work properly.

Fix memory leaks in case memory overflow.

add the change history.

Before dropping the statement, sync and discard the response from the server for the pending extended query.

Adjust the operations after closing eof-detected cursors.

Revise the handling of decimal point.

Fix the bug that the first fetch operation doesn't work properly when
the operation is SQL_FETCH_ABSOLUTE or SQL_FETCH_LAST.

Fix the bug that PostgreSQL's function calls in queries cause a
crash on SQL Server linked servers.

SQLStatistics() sets 'D' for the collation column when an reverse index is used.

Prep release the minor version.

Some bug fix notes.

Correct the behavior of SQLSetConnectAttr() for the driver specific options.

Fix compilation errors 2.

Fix compilation errors in case ODBC 2.5.

Add some driver specific options for SQLSetConnectAttr().

Use SOCK_get_next_n_bytes() instead of SOCK_get_next_byte().

Take the environment variable into account.

Handle *with or without oids* correctly.

Now SQLDescribeCol() can detect the changes of column type or size.

Don't clear the parsed plan in SC_recycle_statement().

Fix the bug that strings are copied to a null pointer.

Cosmetic changes.

Let SC_forget_unnamed() clear the result in case the statement is not
executed yet. It would fix the bug reported by Silvio Brandani.

Remove ssl_verify_available() because it's no longer needed.

Add the change history.

Remove a meaningless if clause.

Prep release 09.00.0300.

Add change history.

Resolve the trouble introduced by the change
 *Return 0 for the column size when the size of numeric items are unknown*
reported by Marco Gaiarin.

Arrangement of a history.

Convert large integer strings properly.

Fix the bug that Access autonumber fields are not detected in 9.0.0200 reported
 by Arnaud Lesauvage.

add some changes.

Change to read and skip the rest of result data when out of memory occurs while reading tuples so that we can continue to use the connection.

Applied a patch by Taizo Ito and correct the length of BIT type in Postgres.

addition of a change history.

Initialize flags member in QResultClass before calling QR_set_rowstart_in_cache() per report from Arno Moore.

Divide SC_returns_rows() macro into several categories and make SQLResultNumCols(), SQLDescribeCol() or SQLColAttribute() available for insert/update/delete .. returning statements.

Fix the bug introduced by the previous change reprted by Adrien de Croy.

Improved a mylog output.