Date: Wed, 1 Mar 2006 16:40:06 GMT From: Matthias Meyser <Meyser@xenet.de> To: freebsd-ports-bugs@FreeBSD.org Subject: Re: ports/93981: ePerl coredumps as CGI Handler when used with Perl 5.8 Message-ID: <200603011640.k21Ge6r2070585@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
The following reply was made to PR ports/93981; it has been noted by GNATS.
From: Matthias Meyser <Meyser@xenet.de>
To: Anton Berezin <tobez@tobez.org>, Matthias Meyser <Meyser@xenet.de>,
FreeBSD-gnats-submit@FreeBSD.org
Cc:
Subject: Re: ports/93981: ePerl coredumps as CGI Handler when used with Perl 5.8
Date: Wed, 1 Mar 2006 17:29:09 +0100
On Wed, Mar 01, 2006 at 05:05:33PM +0100, Anton Berezin wrote:
> On Wed, Mar 01, 2006 at 03:24:03PM +0100, Matthias Meyser wrote:
>
> > In "eperl_perl5.h" the function "fwrite" is redefined iby a macro to use "PerlIO_write".
> > "FILE *" parameter (fwrite) ist used as "PerlIO *" parameter (PerlIO_write).
> > This does not work. It may have worked with older Perl versions.
> >
> > Using the original fwrite works with Perl 5.8.8 (tested) and schould work with older
> > Versions of Perl (not tested).
>
> I am not sure that using the stdio's fwrite is safe, when the rest of
> embedded perl (any perl code, basically) is using PerlIO. There might
> be some interesting effects with regard to buffering.
Thats right. But the whole thing uses stdio and only the fwrite function
ist redefined to use PerlIO. Using stdio handles in PerlIO is definitivly broken.
I agree that this fix may cause problems in other places. (threading)
But its less "worse" than the original code and makes eperl work again. :)
Best thing would be to rewrite the code to use PerlIO in all places.
CU
matthias
--
Matthias Meyser | XeNET Gesellschaft fuer Informations- und
Telefon: +49-5323-94018 | Kommunikationssysteme mbH
Fax: +49-5323-94014 | Burgstaetter Strasse 6
Email: Meyser@xenet.de | 38678 Clausthal-Zellerfeld
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200603011640.k21Ge6r2070585>