From: | "Oleksiy Shchukin" <Oleksiy(dot)Shchukin(at)globallogic(dot)com> |
---|---|
To: | pgsql-bugs(at)postgresql(dot)org |
Subject: | BUG #4599: bugfix for contrib/dblink module |
Date: | 2008-12-31 10:14:58 |
Message-ID: | 200812311014.mBVAEwia032307@wwwmaster.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
The following bug has been logged online:
Bug reference: 4599
Logged by: Oleksiy Shchukin
Email address: Oleksiy(dot)Shchukin(at)globallogic(dot)com
PostgreSQL version: 8.3.5
Operating system: all
Description: bugfix for contrib/dblink module
Details:
Description
-----------
Function dblink_get_result(text connname [, bool fail_on_error]) from
contrib/dblink module ignores 2nd argument.
Reproduce how-to
----------------
I send bad SQL statement via dblink_send_query(text,text), and than try to
grab result with dblink_get_result(connname, /*fail_on_error=*/ false), the
bad sql statement error is issued on client side inspite of
'/*fail_on_error=*/ false' parameter is passed to dblink_get_result()
dblink.c fix
------------
In dblink.c:785 2nd argument for dblink_get_result(text,bool) is referenced
as 'PG_GETARG_BOOL(2)', must be 'PG_GETARG_BOOL(1)'.
I have already compiled and tested it, works fine for me.
Annotated dblink.c from HEAD
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/contrib/dblink/dblink.c?anno
tate=1.76) states that commiter is Joe, please, pass him the bug fix.
Great thanks for your excellect work! :)
Oleksiy
From | Date | Subject | |
---|---|---|---|
Next Message | Richard Evans | 2008-12-31 11:49:24 | Re: BUG #3818: Cross compilation problems |
Previous Message | Alvaro Herrera | 2008-12-30 23:53:24 | Re: BUG #3818: Cross compilation problems |
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2008-12-31 10:46:42 | Re: version() output vs. 32/64 bits |
Previous Message | Simon Riggs | 2008-12-31 10:12:40 | Re: Hot standby and b-tree killed items |