"Hiroshi Inoue" <Inoue(at)tpf(dot)co(dot)jp> writes:
> As far as I see,freeObject() has a fundamental problem.
> Probably it couldn't free multiple references safely.
Yes, that code will have a problem with node trees that have
circular references or multiple links to the same node.
Both of which are cases that we can currently generate :-(.
Of course, circular links will also break copyfuncs, equalfuncs,
and printfuncs. We have a known bug with the backend crashing
when -d is enabled for certain queries, because of infinite
recursion in printfuncs.
For the moment, the only thing I need the freefuncs code for is
to free nodetrees that have been created by stringToNode. AFAICT
that routine is incapable of creating circular links or multiple
links, so it should work.
Eventually it would be nice to have a better solution.
regards, tom lane
In response to
pgsql-hackers by date
|Next:||From: Adriaan Joubert||Date: 2000-01-30 14:43:18|
|Subject: Re: [HACKERS] Bit strings|
|Previous:||From: Hiroshi Inoue||Date: 2000-01-30 06:05:13|
|Subject: RE: [HACKERS] freefuncs.c is never called from anywhere!?|