Re: points and boxes - core dump - Mailing list pgsql-bugs

From Tom Lane
Subject Re: points and boxes - core dump
Date
Msg-id 26504.1021391014@sss.pgh.pa.us
Whole thread Raw
In response to points and boxes - core dump  (Ewald Geschwinde <webmaster@geschwinde.net>)
Responses Re: points and boxes - core dump
List pgsql-bugs
Ewald Geschwinde <webmaster@geschwinde.net> writes:
> When using the field koord instead of a static box the system will core
> dump:

Hmm, I get garbage answers for the second and later rows, rather than
a core dump.

> Does anybody have an idea why the problem occurs?

I think the guilty party is the pfree() in dist_pb.  Given this set of
data close_pb will return its input Point pointer, so the net result is
dist_pb tries to pfree the supplied constant.  A bad move :-(.

I am strongly inclined to rip out *all* the pfrees of someone else's
result in geo_ops.c, not only that one.  Given the current handling of
memory management they are a waste of cycles and code space, and after
seeing this example I am not inclined to trust them not to be pfreeing
something they shouldn't.

            regards, tom lane

pgsql-bugs by date:

Previous
From: Hannu Krosing
Date:
Subject: Re: [HACKERS] Bug #659: lower()/upper() bug on
Next
From: Tom Lane
Date:
Subject: Re: points and boxes - core dump