Andrew Dunstan wrote:
> [hours of digging later]
> 
> On FBSD at least, this is failing in the function Perl_do_taint(). I 
> can't see anything too terrible in the source for this, but to dig 
> further I'd have to build another perl with debugging turned on. Anyway, 
> it turns out that this actually isn't called at all if Perl is 
> configured to use its own malloc() routines instead of those supplied by 
> the system, as is the case on Fedora, for example, but not FBSD.
> 
> So we have a bit of a dilemma. We know that this initialization is 
> required by the Perl API, and we know that some platforms fail without 
> it, and we also know that this fails on some platforms which use the 
> system's malloc() for perl.
> 
> I think we need to float a bug upstream to the perl guys on this, but as 
> a holding position I suggest that we alter the #ifdef test to avoid 
> calling PERL_SYS_INIT3() where MYMALLOC is defined. It's ugly, but I 
> can't think of another simple way around it (and we've done worse things 
> to accommodate platform weirdness ;-) )
> 
> BTW, it's not caused by passing NULL as the third argument of 
> PERL_SYS_INIT3() - changing that was the first thing I tried.

I think with an extensive C comment that is acceptable at this point, at
least until we hear from the perl guys.  I would think projects would
have hit this issue too.

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

  + If your life is a hard drive, Christ can be your backup. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to