Re: another ecpg crash - Mailing list pgsql-hackers

From Martijn van Oosterhout
Subject Re: another ecpg crash
Date
Msg-id 20080511091252.GA14240@svana.org
Whole thread Raw
In response to Re: another ecpg crash  (Euler Taveira de Oliveira <euler@timbira.com>)
Responses Re: another ecpg crash  (Euler Taveira de Oliveira <euler@timbira.com>)
List pgsql-hackers
On Sun, May 11, 2008 at 02:19:05AM -0300, Euler Taveira de Oliveira wrote:
> Alvaro Herrera wrote:
>
> >Huh, isn't the test backwards?
> >
> In which way? I use a simple one but whatever test that uses 'exec sql
> include foo' and foo.h doesn't exist, it will crash.

I think he means specifically this line in the diff:

> !                     /* there are some cases (i.e. file not found)
> !                      * that the input is not available */
> !                     if (!yyin)
> !                             fclose(yyin);

This will close the file *only* if yyin is NULL, which probably isn't
what is meant.

Have a nice day,
--
Martijn van Oosterhout   <kleptog@svana.org>   http://svana.org/kleptog/
> Please line up in a tree and maintain the heap invariant while
> boarding. Thank you for flying nlogn airlines.

pgsql-hackers by date:

Previous
From: Nikhils
Date:
Subject: Re: [PATCHES] [badalex@gmail.com: Re: [BUGS] Problem identifying constraints which should not be inherited]
Next
From: Tino Wildenhain
Date:
Subject: Re: Setting a pre-existing index as a primary key