On Wed, Mar 13, 2019 at 04:35:48AM +, Matsumura, Ryo wrote:
> Hi Kurokawa-san
>
> I reviewd it. It's ok.
> I also confirm there is no same bug.
FYI, this was applied a few weeks ago:
Author: Michael Meskes
Date: Fri Mar 15 22:35:24 2019 +0100
Use corre
Hi Kurokawa-san
I reviewd it. It's ok.
I also confirm there is no same bug.
Regards
Ryo Matsumura
Dear Matsumura-san,
> I think that the 2nd argument of following ecpg_init() must be
> real_connection_name.
> Is it right?
Yes, I think it should be real_connection_name for raising correct error
message.
This is also an leak of my code and I attached a patch.
Best Regards,
Hayato Kuroda
Fu
Hi Kuroda-san
I think that the 2nd argument of following ecpg_init() must be
real_connection_name.
Is it right?
ECPGdeallocate(int lineno, int c, const char *connection_name, const char *name)
:
con = ecpg_get_connection(real_connection_name);
if (!ecpg_init(con, connection_name, lineno)
> I attached a simple bug-fixing patch.
I'm not happy with the situation, but don't see a better solution
either. Therefore I committed the change to get rid of the regression.
Thanks.
Michael
--
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Meskes at (Deb
Dear Rushabh, Michael,
I attached a simple bug-fixing patch.
Could you review it?
An added logic is:
1. Send a close statement to a backend process regardless of the existence
of a cursor.
2. If ecpg_do function returns false, raise “cursor is invalid” error.
3. Remove cursor from
Dear Rushabh, Michael,
Thank you for reporting.
I understood bugs had be embed in ecpg by me.
I start checking codes and investigating solutions and other errors.
Best Regards,
Hayato Kuroda
Fujitsu LIMITED
On Thu, Mar 7, 2019 at 9:56 AM Michael Meskes wrote:
> Hi,
>
> > Commit bd7c95f0c1a38becffceb3ea7234d57167f6d4bf add DECLARE
> > STATEMENT support to ECPG. This introduced the new rule
> > for EXEC SQL CLOSE cur and with that it gets transformed into
> > ECPGclose().
> >
> > Now prior to the abo
Hi,
> Commit bd7c95f0c1a38becffceb3ea7234d57167f6d4bf add DECLARE
> STATEMENT support to ECPG. This introduced the new rule
> for EXEC SQL CLOSE cur and with that it gets transformed into
> ECPGclose().
>
> Now prior to the above commit, someone can declare the
> cursor in the SQL statement and
