Re: pgsql: Remove "fmgr.h" include in cube contrib --- caused crash on a Ge - Mailing list pgsql-committers

From Bruce Momjian
Subject Re: pgsql: Remove "fmgr.h" include in cube contrib --- caused crash on a Ge
Date
Msg-id 201109011807.p81I7Bs15141@momjian.us
Whole thread Raw
In response to Re: pgsql: Remove "fmgr.h" include in cube contrib --- caused crash on a Ge  (Jeremy Drake <pgsql@jdrake.com>)
Responses Re: pgsql: Remove "fmgr.h" include in cube contrib --- caused crash on a Ge
List pgsql-committers
Jeremy Drake wrote:
> On Thu, 1 Sep 2011, Bruce Momjian wrote:
>
> > Tom Lane wrote:
> > > Bruce Momjian <bruce@momjian.us> writes:
> > > > Remove "fmgr.h" include in cube contrib --- caused crash on a Gentoo
> > > > builfarm member.
> > >
> > > mongoose is still crashing, so it must have been some other aspect of
> > > commit 4bd7333 that caused this.
> >
> > Agreed.  Let me look some more.  Odd this succeeds:
> >
> >     okapi           Gentoo 1.12.14 icc 11.1.072 x86_64
> >
> > but this fails:
> >
> >     mongoose     Gentoo 1.6.14 icc 9.0.032 i686
> >
> > The backtrace:
> >
> >     http://www.pgbuildfarm.org/cgi-bin/show_log.pl?nm=mongoose&dt=2011-09-01%2013%3A45%3A01
> >
> > shows it failing on this line:
> >
> >     size = offsetof(NDBOX, x[0]) +sizeof(double) * 2;
> >
> > so I wonder if this is some compiler bug.  offsetof is:
> >
> >     ((long) &((type *)0)->field)
> >
> > and the struct is:
> >
> >     typedef struct NDBOX
> >     {
> >         int32       vl_len_;        /* varlena header (do not touch directly!) */
> >         unsigned int dim;
> >         double      x[1];
> >     } NDBOX;
> >
> > That "x" is quite a common symbol.  Is there any way to get access to
> > this machine?  Should I just revert it all and see what happens?
> >
> >
>
> I am the owner of both mongoose and okapi.  Let me know if there's
> anything you want me to try.

Thanks.  I would either like to email you patches to test or get ssh
access so I can compile it myself.

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + It's impossible for everything to be true. +

pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: Re: pgsql: libpq compiles various pgport files like ecpg does, and needs si
Next
From: Bruce Momjian
Date:
Subject: pgsql: Improve method of avoiding fcinfo compile errors.