Re: Bug in PostgreSQL ODBC - in names with dollar

Thank you very much!

G.P.

From: pgsql-odbc-owner(at)postgresql(dot)org [mailto:pgsql-odbc-owner(at)postgresql(dot)org] On Behalf Of Inoue, Hiroshi
Sent: Tuesday, September 5, 2017 01:05
To: Georgy Pruss <gpruss(at)projects(dot)ksf(dot)kiev(dot)ua>
Cc: pgsql-odbc(at)postgresql(dot)org
Subject: Re: [ODBC] Bug in PostgreSQL ODBC - in names with dollar

Hi Georgy,

Sorry for the late reply.
We are planning to make a new release this weekend.

regards,
Hiroshi Inoue

On 2017/09/04 19:08, Georgy Pruss wrote:

Hi,

I’m sorry to bother you again, but I didn’t see any updates in the mailing list lately.

Unfortunately this problem with the dollar signs in names is quite sensitive to us (we’re porting a set of products to PostgreSQL), so we’d really like to know, even approximately, if the fix will be in the official odbc release and the time frame.

Any information on that would be greatly appreciated.

Thank you.

Georgy Pruss.

From: Georgy Pruss [mailto:gpruss(at)ksf(dot)kiev(dot)ua]
Sent: Wednesday, August 30, 2017 17:02
To: 'Inoue, Hiroshi' <mailto:h-inoue(at)dream(dot)email(dot)ne(dot)jp> <h-inoue(at)dream(dot)email(dot)ne(dot)jp>
Cc: pgsql-odbc(at)postgresql(dot)org <mailto:pgsql-odbc(at)postgresql(dot)org>
Subject: RE: [ODBC] Bug in PostgreSQL ODBC - in names with dollar

Thank you very much.

It works on Windows.

I hope it’ll be the same on Linux.

Can I ask when it’s in the official release of ODBC?

Thank you.

Georgy Pruss

From: pgsql-odbc-owner(at)postgresql(dot)org <mailto:pgsql-odbc-owner(at)postgresql(dot)org> [mailto:pgsql-odbc-owner(at)postgresql(dot)org] On Behalf Of Inoue, Hiroshi
Sent: Wednesday, August 30, 2017 01:39
To: Georgy Pruss <gpruss(at)ksf(dot)kiev(dot)ua <mailto:gpruss(at)ksf(dot)kiev(dot)ua> >
Cc: pgsql-odbc(at)postgresql(dot)org <mailto:pgsql-odbc(at)postgresql(dot)org>
Subject: Re: [ODBC] Bug in PostgreSQL ODBC - in names with dollar

Hi Georgy,

Could you try the git at git.postgresql.org/git/postgresql.git ?
(web page https://git.postgresql.org/gitweb/?p=psqlodbc.git;a=summary ).

Or could you please try the test drivers 9.6.0415 on Windows at
http://www.ne.jp/asahi/inocchichichi/entrance/psqlodbc/ ?

regards,
Hiroshi Inoue
.

On 2017/08/30 2:49, Georgy Pruss wrote:

Hi,

I’m sorry, I really tried to register at pgsql-odbc list at postgresql.org, using different email addresses, but it didn’t’ work.

I saw that you may deal with this problem.

When UseServerSidePrepare is set to 1 (which is default, I guess), any names with $ in them cause error:

“The # of binded parameters < the # of parameter markers”

It’s both on Windows and Linux.

It seems that PG tries to use un-paired ‘$’ symbols as parameter marks for the server-side prepare/execute feature, even if there’s no PREPARE or EXECUTE statements.

I believe, PG should not parse any statements for $-parameters outside PREPARE/EXECUTE.

Simple ODBC program:

$ cat test_odbc.c

// gcc -I unixODBC-2.3.4/include -I unixODBC-2.3.4 -L libs -lodbc test_odbc.c -o test_odbc

// test_odbc 0|1

#include <stdio.h>

#include <time.h>

#include <sql.h>

#include <sqlext.h>

int main(int ac, char* av[])

{

SQLHENV env;

SQLHDBC dbc;

SQLRETURN ret;

SQLCHAR outstr[1024];

SQLSMALLINT outstrlen;

SQLHSTMT stmt;

if( ac!=2 || av[1][1]!='\0' || !(av[1][0]=='0' || av[1][0]=='1')) return 1;

SQLAllocHandle(SQL_HANDLE_ENV, SQL_NULL_HANDLE, &env);

SQLSetEnvAttr(env, SQL_ATTR_ODBC_VERSION, (void*)SQL_OV_ODBC3, 0);

SQLAllocHandle(SQL_HANDLE_DBC, env, &dbc);

char conn[512];

sprintf( conn, "DSN=PGDriver;UseServerSidePrepare=%s", av[1] );

printf( "Connection: %s\n", conn );

ret = SQLDriverConnect(dbc, NULL, conn, SQL_NTS,

outstr, sizeof(outstr), &outstrlen, SQL_DRIVER_COMPLETE);

if(!SQL_SUCCEEDED(ret))

{

fprintf(stderr, "Failed to connect\n");

}

else

{

printf("Returned connection string was:\n\t%s\n", outstr);

char sql[512];

sprintf( sql, "create table T$_x_%u (f integer)", (int)time(0) );

printf( "Exec: %s\n", sql );

SQLAllocHandle(SQL_HANDLE_STMT, dbc, &stmt);

ret = SQLExecDirect(stmt, sql, SQL_NTS);

if(SQL_SUCCEEDED(ret))

printf("ok\n");

else

printf("error\n");

SQLFreeHandle(SQL_HANDLE_STMT,stmt);

SQLDisconnect(dbc);

}

SQLFreeHandle(SQL_HANDLE_DBC, dbc);

SQLFreeHandle(SQL_HANDLE_ENV, env);

return 0;

}

$ test_odbc 0

Connection: DSN=PGDriver;UseServerSidePrepare=0

Returned connection string was:

DSN=PGDriver;DATABASE=…;SERVER=…;PORT=5432;UID=…;PWD=…;SSLmode=disable;ReadOnly=0;Protocol=7.4;FakeOidIndex=0;ShowOidColumn=0;RowVersioning=0;ShowSystemTables=0;=Fetch=100;UnknownSizes=0;MaxVarcharSize=255;MaxLongVarcharSize=8190;Debug=0;CommLog=0;UseDeclareFetch=0;TextAsLongVarchar=1;UnknownsAsLongVarchar=0;BoolsAsChar=1;Parse=0;ExtraSysTablePrefixes=;LFConversion=0;UpdatableCursors=1;TrueIsMinus1=0;BI=0;ByteaAsLongVarBinary=1;UseServerSidePrepare=0;LowerCaseIdentifier=0;

Exec: create table T$_x_1504022913 (f integer)

ok

$ test_odbc 1

Connection: DSN=PGDriver;UseServerSidePrepare=1

Returned connection string was:

DSN=PGDriver;DATABASE=…;SERVER=…;PORT=5432;UID=…;PWD=…;SSLmode=disable;ReadOnly=0;Protocol=7.4;FakeOidIndex=0;ShowOidColumn=0;RowVersioning=0;ShowSystemTables=0;=Fetch=100;UnknownSizes=0;MaxVarcharSize=255;MaxLongVarcharSize=8190;Debug=0;CommLog=0;UseDeclareFetch=0;TextAsLongVarchar=1;UnknownsAsLongVarchar=0;BoolsAsChar=1;Parse=0;ExtraSysTablePrefixes=;LFConversion=0;UpdatableCursors=1;TrueIsMinus1=0;BI=0;ByteaAsLongVarBinary=1;UseServerSidePrepare=1;LowerCaseIdentifier=0;

Exec: create table T$_x_1504022916 (f integer)

error

Actually, probably any statement with ‘$’ in table/field name causes this error.

PGDriver - /usr/pgsql-9.6/lib/psqlodbcw.so

Driver Info

DBMS Name: PostgreSQL

DBMS Version: 9.6.3

PostgreSQL 9.6.3 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-18), 64-bit

This is probably the same error as in https://www.postgresql.org/message-id/09ce3d91ce364987964726aa50a16155%40neosys.si <http://TOC.ASG.COM:8080/?dmVyPTEuMDAxJiY4NWFkOWEwNjFkMTgzOGRiYj01OUE1NzQ2NV8yMTIyOF8xMzEzN18xJiZkZTU4YWZhNmRjMzY5YTk9MTIyMyYmdXJsPWh0dHBzJTNBJTJGJTJGd3d3JTJFcG9zdGdyZXNxbCUyRW9yZyUyRm1lc3NhZ2UtaWQlMkYwOWNlM2Q5MWNlMzY0OTg3OTY0NzI2YWE1MGExNjE1NSU0MG5lb3N5cyUyRXNp>

Thank you.

Georgy Pruss.