Re: "Attempt to free non-existent shared string"? (with patch)

I found a subtle problem with the lazydelete mechanism (which is used
to postpone the delete of a entry that may be getting iterated over).
I was using the HeKLEN slot to hold the hint, but the real HeKLEN is
needed later to call unsharepvn().  This means that only magical
hash entries can use the HeKLEN slot to hold flags.

Here's a tested patch against 5.00305 that fixes the problem.
The patch simply moves the LAZYDEL hint to become a SV-level private
flag.

perl 5.003_06: hv.c

Date: Fri, 20 Sep 1996 15:38:57 -0400
From: Gurusamy Sarathy <gsar@engin.umich.edu>
Subject: Re: "Attempt to free non-existent shared string"? (with patch)

I found a subtle problem with the lazydelete mechanism (which is used
to postpone the delete of a entry that may be getting iterated over).
I was using the HeKLEN slot to hold the hint, but the real HeKLEN is
needed later to call unsharepvn().  This means that only magical
hash entries can use the HeKLEN slot to hold flags.

Here's a tested patch against 5.00305 that fixes the problem.
The patch simply moves the LAZYDEL hint to become a SV-level private
flag.

Date: Mon, 30 Sep 1996 01:13:28 -0400
From: Spider Boardman <spider@Orb.Nashua.NH.US>
Subject: Re: pre extending hash? - need speed

The patch below (which is relative to perl5.001l) implements
"keys %hash = 50_000;" (or other integer-evaluable sizes) for
pre-sizing hashes.  I've only moved the patch forward from
when I first did it.  I'm sure the code in hv_ksplit could be
improved.

Dale's posting as patch (Was: Perl 5.003_5 make fails on NS3.2 - CURED)

Replace optimize="-g" by optimize="" since we're just trying to turn off
the optimizier.

Handle NeXT, POSIX, and setpgid in pp_sys.c and POSIX.

Add notes about MachTen 4.0.3 SYSV IPC.

Full LONG_MAX & co. patch over 5.003_05

This patch contains the changes I've collected for the various _MAX issues
since 5.003_05. No patches issued between 5.003_05 and this one should be
applied, use this one instead.

The effect is to remove the CHAR_* and I8_* constants (which are
ambiguous) and to explicitly cast all of the constants.

perl 5.003_06: global.sym

Restore the 5.003 gv_fullname() and gv_efullname() functions.
Provide new 3-arg forms gv_fullname3() and gv_efullname3().

Date: Mon, 30 Sep 1996 01:13:28 -0400
From: Spider Boardman <spider@Orb.Nashua.NH.US>
Subject: Re: pre extending hash? - need speed

The patch below (which is relative to perl5.001l) implements
"keys %hash = 50_000;" (or other integer-evaluable sizes) for
pre-sizing hashes.  I've only moved the patch forward from
when I first did it.  I'm sure the code in hv_ksplit could be
improved.

Date: Mon, 7 Oct 1996 22:03:00 +0300
From: Jarkko Hietaniemi <jhi@alpha.hut.fi>
Subject: LC_COLLATE.

Big patch to add, document, and test LC_COLLATE support.

added var lc_collate_active and func mem_collxfrm.

perl 5.003_06: ext/SDBM_File/sdbm/sdbm.h

Revert from Perl_malloc to Mymalloc for binary compatibility with
5.003.

Dale's posting as patch (Was: Perl 5.003_5 make fails on NS3.2 - CURED)

Handle NeXT, POSIX, and setpgid in pp_sys.c and POSIX.

LC_COLLATE.

Big patch to add, document, and test LC_COLLATE support.

setlocale() allowed one argument only,
call to perl_init_fold() (in util.c) if setlocale() succeeded.

LC_COLLATE.

Big patch to add, document, and test LC_COLLATE support.

enhanced setlocale() docs and introduced the one-argument variant doc.

perl 5.003_06: ext/Opcode/Opcode.pm

Date: Fri, 20 Sep 1996 12:59:21 +0200
From: Gisle Aas <aas@bergen.sn.no>
Subject: Re: Symbol.pm clobbers $_ at startup

The same kind of problem seem to be present in Opcode.pm:

Date: Fri, 20 Sep 1996 15:08:33 +0100 (BST)
From: "Joseph S. Myers" <jsm28@hermes.cam.ac.uk>
Subject: Pod typos, pod2man bugs, and miscellaneous installation comments

Here is a patch for various typos and other defects in the Perl
5.003_05 pods, including the pods embedded in library modules.

perl 5.003_06: dev/null

Perl 5.003_05 compiles on DYNIX/ptx 4.0 (v4.1.3), and passes all tests.
The only change needed is in "ext/NDBM_File/Makefile.PL" - on this system,
ndbm is actually contained in the libc library, and must be linked against
-lc when compiling.  (this is for dynamic ELF executables, I didn't compile
statically)

Pod typos, pod2man bugs, and miscellaneous installation comments

Here is a patch for various typos and other defects in the Perl
5.003_05 pods, including the pods embedded in library modules.

Updated to IO-1.12.

Updated to IO-1.12.

perl 5.003_06: dump.c embed.h gv.c run.c

Restore the 5.003 gv_fullname() and gv_efullname() functions.
Provide new 3-arg forms gv_fullname3() and gv_efullname3().

Re: pre extending hash? - need speed

The patch below (which is relative to perl5.001l) implements
"keys %hash = 50_000;" (or other integer-evaluable sizes) for
pre-sizing hashes.  I've only moved the patch forward from
when I first did it.  I'm sure the code in hv_ksplit could be
improved.

VMS patches to 5.003_05

perl 5.003_06: configure

Date: Mon, 30 Sep 1996 14:01:05 +0100
From: Sven Verdoolaege <skimo@breughel.ufsia.ac.be>

Warn the user of case-insensitive file systems that they may have
accidentally gotten 'configure' instead of 'Configure'.

Pod typos, pod2man bugs, and miscellaneous installation comments

Here is a patch for various typos and other defects in the Perl
5.003_05 pods, including the pods embedded in library modules.

Changed Larry's address to larry@wall.org.

perl 5.003_06: MANIFEST

Date: Sun, 22 Sep 1996 00:59:56 +0200
From: Gisle Aas <aas@aas.no>
Subject: More standard library test scripts

This is a collection of test scripts for the standard library modules.
Some of the tests does not pass unless some of the patches I have sent
out are applied.

Date: Sat, 28 Sep 1996 15:11:06 +0200
From: Andreas Koenig <k@anna.in-berlin.de>
Subject: Dale's posting as patch (Was: Perl 5.003_5 make fails on NS3.2 - CURED)

Handle NeXT, POSIX, and setpgid in pp_sys.c and POSIX.

Date: Mon, 7 Oct 1996 22:03:00 +0300
From: Jarkko Hietaniemi <jhi@alpha.hut.fi>
Subject: LC_COLLATE.

Big patch to add, document, and test LC_COLLATE support.

added perli18n.pod.

LC_COLLATE.

Big patch to add, document, and test LC_COLLATE support.

added LC_COLLATE doc.

perl 5.003_06: Configure

Add -Wl,rpath option for irix* to find the installed shared
libperl.so

Add /shlib to libpth.  It is used by Digital Unix 4.0.

Date: Mon, 30 Sep 1996 14:01:05 +0100
From: Sven Verdoolaege <skimo@breughel.ufsia.ac.be>

Detect Cygnus Win32, or at least don't let Configure get fooled
into thinking it's OS/2.

Updated for 5.003_06.

Change to subversion 6.

perl 5.003_05: [patch introduction and re-organisation]

# This is my patch perl5.003_05.pat to perl5.003_04
# The full description is below.
# Please execute the following commands before applying this patch.
# (You can feed this patch to 'sh' to do so.)
#  Andy Dougherty               <doughera@lafcol.lafayette.edu>

# We'll create a new test, but patch won't automatically make it
# executable.
touch t/io/read.t
chmod +x t/io/read.t

exit 0

This is patch perl5.003_05.pat to perl version 5.003_04.
This takes you from 5.003_04 to 5.003_05.

To apply this patch, run the above commands,
cd to your perl source directory and then type

        patch -p1 -N < perl5.003_05.pat

The changes are described after each /^Index:/ line below.  This is
designed so you can examine each change with a command such as

        csplit -k perl5.003_05.pat '/^Index:/' '{99}'

Patch and enjoy,

    Andy Dougherty              doughera@lafcol.lafayette.edu
    Dept. of Physics
    Lafayette College, Easton PA 18042

VMS 5.003_05 Update.

perl 5.003_05: utils/perldoc.PL

More choices in the pager war.  Unfortunately, we can't rely on
all users agreeing with the Sysadmin's choice, nor can we
assign a default preference order, since opinions vary.  If the
user doesn't have $ENV{PAGER} set, we do want to pick up one that
at least works, so we'll try whatever Configure found.

Fix typo $Config{'has_sockets'} ought to be $Config{'d_socket'};

updated h2xs

Changes:
a) Docs and examples for -x updated;
b) Path to xxxx.h would not be changed to /usr/include/xxxx.h
unless this file exists (outside of VMS, I'm afraid to make an error
there). - Useful with -x option, when the file may be eaten via -I
inside -F.
c) .h file would be scanned only if needed.
d) typemap would be generated (with T_PTROBJ).
e) Documentation (=list) for autogenerated guys would be
included into POD.
f) duplicated XSUBs would not be generated;
g) arguments to XSUBs being arrays are recognized (note that
xsubpp would probably choke on such guys).

-x option requires C-Scan-0.3 (releases a couple of minutes ago to
    ftp://ftp.math.ohio-state.edu/pub/users/ilya/perl
should propagate to CPAN soon).

perl 5.003_05: unixish.h

Change comment style so that IBM's picky xlc compiler doesn't
think we've mistakenly tried to nest comments.

Re: UNIVERSAL::class busted

yes, but I also noticed that this does not check that the reference
is an object, so the patch should be

Pack Patch (was Re: 5.002 - pack/unpack does not do "I" right)

perl 5.003_05: t/op/inc.t util.c

One last LONG & co. fix (yet another cut'n'paste error) and a few
minor cleanups. Nothing crucial.

Perl 5.003.03: race condition in t/lib/io_sock.t

io_sock.t works by forking a subprocess it can communicate with.
It has the subprocess wait for the main process by sleeping 10
seconds or until an alarm arrives.

With my setup, the alarm signal arrives *before* the child
has a chance to ignore the alarm signal.

I fixed this by moving the "$SIG{ALRM} = sub {};" up before the
fork. It does not hurt to have the parent ignore alarms, too.

perl 5.003_05: t/io/read.t

Clear any buffer space exposed by by read().
This is almost certainly a bug-fix.

Minor potential bug in AV creation

Fix more spots where we had PerlIO_stderr() and should have had
Perl_debug_log instead.

I wasn't the one who originated this patch.  But it looks like it
would improve the safety of AV creation.

Remove potentially incorrect casts on PerlIO_set_ptrcnt.
'ptr' is already STDCHAR, which is supposed to be the type of
char used in stdio.h, so we shouldn't have to cast it.

perl 5.003_05: pp_sys.c

Clear any buffer space exposed by by read().
This is almost certainly a bug-fix.

Undef and then re-define my_chsize from Perl_my_chsize to
just plain chsize if this system HAS_CHSIZE.  This probably only
applies to SCO.  This shows the perils of having internal
functions with the same name as external library functions :-).

Use CLK_TCK if HZ is not available.

perl 5.003_05: pp_hot.c

Date: Thu, 05 Sep 1996 00:25:28 -0400
From: Gurusamy Sarathy <gsar@engin.umich.edu>
Subject: minor misc. cleanup

This patch makes some minor cleanups to the sources.  No change
in functionality whatsoever.

Date: Thu, 29 Aug 1996 15:14:51 +0200
From: Sven Verdoolaege <skimo@breughel.ufsia.ac.be>
Subject: more TIEHANDLE

This adds support for a READLINE method.

perl 5.003_05: pp.c

Date: Fri, 23 Aug 1996 17:22:40 -0400 (EDT)
From: Chip Salzenberg <salzench@nielsenmedia.com>
Subject: Minor integer speedups in mathematics

This patch provides minor speedups by using integer math and SVt_IV
values when performing bitwise operations and modulus.

Date: Tue, 3 Sep 1996 17:49:22 -0400 (EDT)
From: Kenneth Albanowski <kjahds@kjahds.com>
Subject: Pack Patch (was Re: 5.002 - pack/unpack does not do "I" right)

(double)auint cast added for call to sv_setnv().

perl 5.003_05: pod/perlvar.pod

Be explicit about $/="" matching empty lines, that is, lines
with no spaces or tabs.

Change blank lines to empty lines.

More (and less!) 425traps

Here's documentation on the change in split's behavior between Perl 4
and Perl 5.

Large integer traps

Precedence

warn STDERR

Change blank lines to empty lines.

more TIEHANDLE

This adds support for a READLINE method.

Document correct C<use POSIX ":wait_h";> usage.

perl 5.003_05: pod/perlfunc.pod

Document correct C<use POSIX ":wait_h";> usage.

Add notes about POSIX [gs]etpgrp.

perl 5.003_05: dev/null

New script to check for common errors in pods.  This is not
normally called during the perl build process, but you can
use it with  B<make check>.

perl 5.003_05: pod/Makefile

Remove trailing spaces in pods.
Include a call to the checkpods script in the Makefile (though it's
not ordinarily used by users).

perl 5.003_05: perlsdio.h

Don't supply redundant parameters for PerlIO_open and PerlIO_fdopen.

Include PerlIO_reopen and PerlIO_getname.

s/FILE_(CNT|PTR)_LVALUE/STDIO_(CNT|PTR)_LVALUE to fix a typo.
This had prevented SV_FAST_FGETS from working anywhere.

Include PerlIO_canset_cnt.  I'm not sure how this is supposed to
differ from STDIO_CNT_LVALUE.

perl 5.003_05: perlio.c

Eliminate potential "signed vs. unsigned" warning

Add PerlIO_reopen and PerlIO_cgetname functions.

perl 5.003_05: perl.h

One last LONG & co. fix (yet another cut'n'paste error) and a few
minor cleanups. Nothing crucial.

Make a little more C++-friendly for IBM's CSET++ compiler.

Enhance detection of [gs]etpgrp() variants.  Some systems have
BSD-style behavior for one and POSIX for the other.  Use
[gs]etpgid() instead, whenever possible.

Make floating point constants Locale-friendly.

Re: \ ( @array ) busted for lexical @array (once more)

Add _STDIO_LOADED (VMS) to list of guard symbols.

perl 5.003_05: mg.h

Replace the oft-repeated mg_ptr incantation with
the simple MgPVKEY macro.

Rename MgPVKEY to MgPV (to match with HePV elsewhere).  Add
additional parens around the "mg".

Integerize mg.c; eliminate warning on C< local($)) >

Ok, here's a tested patch for the debugger problem.
I was missing the fact that DB::dbline magic is actually
uppercase (which means hv_store_ent() etc., will pass SV keys
to the vtbl_dbline handlers).

Replace the oft-repeated mg_ptr incantation with
the simple MgPVKEY macro.

Rename MgPVKEY to MgPV (to match with HePV elsewhere).  Add
additional parens around the "mg".

(lines near 584) Part of VMS changes.  I don't know what this did.

This patch converts magic variables ($!, $^E, etc.) to use integers
(C<sv_setiv>) instead of floats.  It also eliminates a warning from
C< local($)) >, via a hack similar to $!.

perl 5.003_05: makedepend.SH

This patch eliminates "\|" in sed patterns in makedepend.SH, since
they're not really needed anyway in this one case.

syslog.pl `use Socket' lossage

syslog.pl tries but fails to use
Socket.pm, the problem is that use doesn't return a true value.  This
module should be recast in terms of Sys::Syslog, of course.

perl 5.003_05: lib/open3.pl

The I/O directions on the dad_wtr and kid_rdr were backwards.
IO/Open3.pm didn't have this error.

perl 5.003_05: lib/Math/Complex.pm

There was a mistake in the sqrt routine in lib/Math/Complex.pm that
gave wrong answers when the magnitude of the imaginary part of the
argument exceeded the magnitude of the real part.  Line 69 had too
many sqrt($y)'s.  Further, expressions were re-arranged so that
calls to the expensive real sqrt() routine were reduced from 4 to 2
in this case.

Cwd::fastcwd in File::Find

Is there a good reason why File::Find uses Cwd::fastcwd instead of
Cwd:cwd when fastcwd isn't as portable?
[In particular, fastcwd() doesn't work on AFS.]

perl 5.003_05: lib/ExtUtils/MakeMaker.pm

Updated to MakeMaker-5.38.

Updated to MakeMaker-5.39 to allow CFLAGS in hint files.

Updated to MakeMaker-5.38.

Updated to MakeMaker-5.38.

perl 5.003_05: lib/ExtUtils/Install.pm

Updated to MakeMaker-5.38.

Fix for VMS utime.

perl 5.003_05: lib/AnyDBM_File.pm

AnyDBM_File (modifying ISA does not work as expected)
Now behaves as documented:  Modifying ISA works to select
order in which *DB* modules are tried.  The default is still
the same.

Add helpful "die" message to end of AnyDBM_File.  Previously
it would return a 0, and the failure would eventually show up
somewhere else in the script and be hard to track down.  It is
a failure if perl can't open AnyDBM_File.  The test regression
suite is supposed to indicate this as a failure too.

Re: installperl feature request (was: Re: Upgrade 4.0x to 5.001m)

Fix installperl so that the -n option really only prints commands.
(previously, it would still do the mkdirs.)

an "ignore installed" option, so that it doesn't bother to check
to see if the target already exists (an optimization that I
*don't* want it to do when I do #1 above)

perl 5.003_05: hv.c

Date: Thu, 05 Sep 1996 00:25:28 -0400
From: Gurusamy Sarathy <gsar@engin.umich.edu>
Subject: minor misc. cleanup

This patch makes some minor cleanups to the sources.  No change
in functionality whatsoever.

Date: Thu, 05 Sep 1996 02:52:21 -0400
From: Gurusamy Sarathy <gsar@engin.umich.edu>

Subject: debugger problems--another patch (was Re: 5.003_04)

I have tried to avoid copying of hash keys that are passed to
magical hashes, but it seems that copying may be unavoidable
since the hv_*_ent() functions could be PADTMPs (and other
SVs that may get reused) as keys.

VMS dynamic %ENV fix

perl 5.003_05: hints/sunos_4_1.sh

Describe solution for the __lib_version problem with acc on
SunOS.

perl 5.003_05: hints/solaris_2.sh

Catch GNU ld even though it doesn't identify itself as a GNU tool.
Thanks to Tim Pierce <twpierce@midway.uchicago.edu>.

perl 5.003_05: hints/sco.sh

Turn off optimization for stock cc.  This appears to
prevent miniperl core dumps.

perl 5.003_05: hints/hpux.sh

Add note about possible gcc GR3 warning message.

Remove [gs]etpgrp workaround.  Configure & perl.h should handle
this now.

perl 5.003_05: hints/convexos.sh

Remove [gs]etpgrp workaround.  Configure & perl.h should handle
this now.

Make a little more C++-friendly for IBM's CSET++ compiler.

Patch to inhibit autosplit on Socket.pm

This patch inhibits production and use of a completely useless
auto/Socket/autosplit.ix.

Try to work around "bad free" in dbmclose().

perl 5.003_05: ext/ODBM_File/ODBM_File.xs

Attempt to correct for "Bad free" in Ultrix and HPUX versions of
odbm.

FileHandle::DESTROY for fd 0

This fixes FileHandle::DESTROY when called on stdin.

No AutoLoader for Fcntl

Just like Socket, Fcntl doesn't need splitting and AutoLoading.

Update to DB_File 1.03.

Newer CPerl-mode

perl 5.003_05: config_h.SH

Updated to match Configure.

Changed the DLSYM_NEEDS_UNDERSCORE comment to
/**/ to conform to metaconfig style.

Updated to match newest config_h.SH.

turbidity in av.[ch]

Some unclean code that I noticed today.

VMS 5.003_05 Update.

Updated to match Configure.

perl 5.003_05: Makefile.SH

A new heredoc tag in Makefile.SH needs to be quoted.

shrpenv stuff moved to Configure.

perl 5.003_05: MANIFEST

Add new test t/io/read.t.

Add new hints files for ODBM_File for ultrix and hpux.

Add new pod checker script.

perl 5.003_05: INSTALL

Correct libperl5 -> libperl typo.

Describe MakeMaker's Warning (will try anyway) messages.

More info on where and how to send reports.

Add info on non-Unix ports.

perl 5.003_05: Configure

Allow command line or hint-file overrides of $afs.

Allow trailing spaces in nm output for HPUX10.

Check for newer BIND 'search' directives in /etc/resolv.conf as well
as older 'domain' directive.

Enhance detection of [gs]etpgrp() variants.  Some systems have
BSD-style behavior for one and POSIX for the other.  Use
[gs]etpgid() instead, whenever possible.

Include -s in the -h summary of available options.

Allow command-line override of $afs.

Handle trailing spaces in nm-output on HPUX10.

Set shrpenv for handling LD_RUN_PATH, if needed. (This used to
be in Makefile.SH.  Now it's available for other modules too.)

When using shared libperl, avoid LD_RUN_PATH if possible by adding
correct ld flags.  Currently, Solaris and NetBSD get the correct
-R $archlibexp/CORE, and Linux gets its
-Wl,-rpath,$archlibexp/CORE flag.  Other contributions are
welcome.

Updated for 5.003_05.

Change to subversion 5.

perl 5.003_04: [patch introduction and re-organisation]

# The full description is below.
# Please execute the following commands before applying this patch.
# (You can feed this patch to 'sh' to do so.)
#  Andy Dougherty               <doughera@lafcol.lafayette.edu>

rm -f hints/dnix.sh
rm -f os2/notes

# We'll create a new test, but patch won't automatically make it
# executable.
touch t/op/gv.t
chmod +x t/op/gv.t

exit 0

This is patch perl5.003_04.pat to perl version 5.003_03.
This takes you from 5.003_03 to 5.003_04.

To apply this patch, run the above commands,
cd to your perl source directory and then type

        patch -p1 -N < perl5.003_04.pat

The changes are described after each /^Index/ line below.  This is
designed so you can examine each change with a command such as

        csplit -k perl5.003_04.pat '/^Index:/' '{99}'

Patch and enjoy,

    Andy Dougherty              doughera@lafcol.lafayette.edu
    Dept. of Physics
    Lafayette College, Easton PA 18042

perl 5.003_04: utils/perldoc.PL

Use col -x to filter out half-line feeds (ESC-9) from
HP-UX nroff -man output.  (col -x isn't portable -- SunOS
doesn't support the -x option.)

Make leading =head NAME item a paragraph so pod2man finds it.

Patch for LONG_MAX & co.

    __DIE__ (with patch)

sv_2pv() might call croak() (which is not prepared to handle that
when it calls sv_2pv(), itself).  Likewise for warn() (but under
slightly more esoteric circumstances--mg_get() in sv_2pv() might
trigger a call to warn()).

PERL_BADLANG is examined by default before issuing a warning during
    internationalization.

lval substr() coredumps with refs (with patch)

substr() in lvalue context interacts in buggy fashion with SVs that
are !SvOK.  This manifests itself with lexicals that have a REFCNT of
1, since these are merely "cleared in place" by setting SvOK_off.

substr() coredumps with a target that is a ref, when it is used in
an lvalue context.
The patch below corrects the problem by stringifying the reference
first (and emitting a warning when appropriate).

FAKE typeglobs seriously busted (with patch)

Handling of fake typeglobs (scalars that are really globs
in disguise) is seriously busted since 5.002 (it wasn't
so in 5.001n).

The problem is that mg_get() on a glob calls gv_efullname()
which might coerce its first arg to a string.

Re: truncate with file name does not work (with patch)

The prototype for truncate was changed so that perl won't die
with C<use strict;> when the first arg is a bareword (filehandle).
I think it was Tom (as in "tchrist") who brought this up.

Here's a patch that undoes the damage, makes it work with
C<use strict;>, and adds to the testsuite.

The "not implemented" branch is missing a "\n".

Track SVs for destruction when -DPURIFY

This patch removes a few obvious redundancies in the source.

This patch changes neither behavior nor performance.  However, it does
reduce code size and improve maintainability by combining some common
code in gv_fullname() and gv_efullname().

When checking for memory leaks, I compiled Perl with "-DPURIFY".
Although that flag improves the leak checking, it also breaks
destruction of global objects, because SVs aren't kept in captive
arenas any more.

This patch rectifies the problem by providing an alternative
method for keeping track of SVs when Perl is compiled for Purify.
It has no effect on normal operation.

Add comment about assert(len >=0) when len is unsigned anyway.

Patch for LONG_MAX & co.

This patch changes neither behavior nor performance.  However, it does
reduce code size and improve maintainability by combining some common
code in gv_fullname() and gv_efullname().