Re: Re: [COMMITTERS] pgsql: Initialise perl library as documented in perl API.

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: Re: [COMMITTERS] pgsql: Initialise perl library as documented in perl API.
Дата
Msg-id 200906051353.n55Drrd18136@momjian.us
обсуждение исходный текст
Ответ на Re: Re: [COMMITTERS] pgsql: Initialise perl library as documented in perl API.  (Andrew Dunstan <andrew@dunslane.net>)
Список pgsql-hackers
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  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: Re: PostgreSQL Developer meeting minutes up
Следующее
От: Tom Lane
Дата:
Сообщение: Re: Re: [COMMITTERS] pgsql: Initialise perl library as documented in perl API.