diff options
author | Nick Ing-Simmons <nik@tiuk.ti.com> | 2003-01-22 21:03:43 +0000 |
---|---|---|
committer | Nick Ing-Simmons <nik@tiuk.ti.com> | 2003-01-22 21:03:43 +0000 |
commit | 157b749675929e393a7c0bb5f4006743737e22f3 (patch) | |
tree | 415c67495148bb77f32762b3a68700dc14bbfc3a /t | |
parent | c9bca74aca217023baf0f921dcffaaa072a83cf3 (diff) | |
parent | fa6f41cfedcd9c19e472d10292b6d4367aa3b9b0 (diff) | |
download | perl-157b749675929e393a7c0bb5f4006743737e22f3.tar.gz |
Integrate mainline
p4raw-id: //depot/perlio@18570
Diffstat (limited to 't')
-rwxr-xr-x | t/op/array.t | 27 | ||||
-rwxr-xr-x | t/op/ref.t | 41 |
2 files changed, 57 insertions, 11 deletions
diff --git a/t/op/array.t b/t/op/array.t index 472e02cd35..8f2f1a9510 100755 --- a/t/op/array.t +++ b/t/op/array.t @@ -1,6 +1,12 @@ #!./perl -print "1..72\n"; + +BEGIN { + chdir 't' if -d 't'; + @INC = '../lib'; +} + +print "1..73\n"; # # @foo, @bar, and @ary are also used from tie-stdarray after tie-ing them @@ -247,3 +253,22 @@ sub tary { @tary = (0..50); tary(); + + +require './test.pl'; + +# bugid #15439 - clearing an array calls destructors which may try +# to modify the array - caused 'Attempt to free unreferenced scalar' + +my $got = runperl ( + prog => q{ + sub X::DESTROY { @a = () } + @a = (bless {}, 'X'); + @a = (); + }, + stderr => 1 + ); + +$got =~ s/\n/ /g; +print "# $got\nnot " unless $got eq ''; +print "ok 73\n"; diff --git a/t/op/ref.t b/t/op/ref.t index 1205a7a824..9470efa69a 100755 --- a/t/op/ref.t +++ b/t/op/ref.t @@ -5,7 +5,7 @@ BEGIN { @INC = qw(. ../lib); } -print "1..63\n"; +print "1..65\n"; require 'test.pl'; @@ -296,23 +296,44 @@ $a = $a->[1]; print "not " unless $a == 2; print "ok 55\n"; -sub x::DESTROY {print "ok ", 55 + shift->[0], "\n"} -{ my $a1 = bless [4],"x"; - my $a2 = bless [3],"x"; - { my $a3 = bless [2],"x"; - my $a4 = bless [1],"x"; - 567; +# This test used to coredump. The BEGIN block is important as it causes the +# op that created the constant reference to be freed. Hence the only +# reference to the constant string "pass" is in $a. The hack that made +# sure $a = $a->[1] would work didn't work with references to constants. + +my $test = 56; + +foreach my $lexical ('', 'my $a; ') { + my $expect = "pass\n"; + my $result = runperl (switches => ['-wl'], stderr => 1, + prog => $lexical . 'BEGIN {$a = \q{pass}}; $a = $$a; print $a'); + + if ($? == 0 and $result eq $expect) { + print "ok $test\n"; + } else { + print "not ok $test # \$? = $?\n"; + print "# expected ", _qq ($expect), ", got ", _qq ($result), "\n"; } + $test++; } +sub x::DESTROY {print "ok ", $test + shift->[0], "\n"} +{ my $a1 = bless [3],"x"; + my $a2 = bless [2],"x"; + { my $a3 = bless [1],"x"; + my $a4 = bless [0],"x"; + 567; + } +} +$test+=4; my $result = runperl (switches=>['-l'], prog=> 'print 1; print qq-*$\*-;print 1;'); my $expect = "1\n*\n*\n1\n"; if ($result eq $expect) { - print "ok 60\n"; + print "ok $test\n"; } else { - print "not ok 60\n"; + print "not ok $test\n"; foreach ($expect, $result) { s/\n/\\n/gs; } @@ -321,7 +342,7 @@ if ($result eq $expect) { # test global destruction -my $test = 61; +++$test; my $test1 = $test + 1; my $test2 = $test + 2; |