In perl.git, the branch smoke-me/linestr_sv has been created
<http://perl5.git.perl.org/perl.git/commitdiff/5d3f331a5e4001d074f3f065e71e024cf59a8418?hp=0000000000000000000000000000000000000000>
at 5d3f331a5e4001d074f3f065e71e024cf59a8418 (commit)
- Log -----------------------------------------------------------------
commit 5d3f331a5e4001d074f3f065e71e024cf59a8418
Author: Nicholas Clark <[email protected]>
Date: Sun Feb 26 00:22:41 2012 +0100
In S_parse_body(), don't "leak" linestr_sv until global destruction.
This commit ensures that linestr_sv is properly cleaned up, if allocated.
The local variable linestr_sv was added by commit 009d90df4e17a415 in 2007,
to replace use of PL_linestr in S_parse_body(). However, that commit didn't
add any code to free linestr_sv at the end of S_parse_body(), meaning that
the SV sticks around until global destruction.
Subsequent code simplification possible by the removal of suidperl reveals
that linestr_sv is only needed for the '-x' option, so it's safe to avoid
allocating it up front. Additionally, during '-x' processing, Perl_sv_gets()
will upgrade the target SV to SVt_PV and allocate the string buffer as
needed,
so there's no need to pre-upgrade or pre-allocate the SV in S_parse_body().
This slightly reduces the amount of code.
M perl.c
commit 03e6b0f8aa279960cd8dd831f34ac3fea5310754
Author: Nicholas Clark <[email protected]>
Date: Sat Feb 25 23:50:49 2012 +0100
Remove all the never used parameters from the macro validate_suid()
Several parameters are unused in either remaining variant of the
validate_suid() macro. The two variants which used the extra parameters
were removed with suidperl by commit cc69b689ee7c2745 in Jan 2009.
M perl.c
-----------------------------------------------------------------------
--
Perl5 Master Repository
