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