fix coredump after 64k-deep recursion
Hugo van der Sanden [Tue, 3 Sep 2002 14:34:20 +0000 (15:34 +0100)]
Subject: Re: debugging coredump with -DD
Message-Id: <200209031334.g83DYLN09709@crypt.compulink.co.uk>

p4raw-id: //depot/perl@17835

cop.h
t/op/recurse.t

diff --git a/cop.h b/cop.h
index 81733e9..0732a04 100644 (file)
--- a/cop.h
+++ b/cop.h
@@ -111,7 +111,7 @@ struct block_sub {
     AV *       savearray;
 #endif /* USE_5005THREADS */
     AV *       argarray;
-    U16                olddepth;
+    long       olddepth;
     U8         hasargs;
     U8         lval;           /* XXX merge lval and hasargs? */
     SV **      oldcurpad;
@@ -119,7 +119,7 @@ struct block_sub {
 
 #define PUSHSUB(cx)                                                    \
        cx->blk_sub.cv = cv;                                            \
-       cx->blk_sub.olddepth = (U16)CvDEPTH(cv);                        \
+       cx->blk_sub.olddepth = CvDEPTH(cv);                             \
        cx->blk_sub.hasargs = hasargs;                                  \
        cx->blk_sub.lval = PL_op->op_private &                          \
                              (OPpLVAL_INTRO|OPpENTERSUB_INARGS);
index 374813c..9d00640 100755 (executable)
@@ -8,7 +8,7 @@ BEGIN {
     chdir 't' if -d 't';
     @INC = qw(. ../lib);
     require "test.pl";
-    plan(tests => 26);
+    plan(tests => 28);
 }
 
 use strict;
@@ -112,6 +112,9 @@ is(takeuchi($x, $y, $z), $z + 1, "takeuchi($x, $y, $z) == $z + 1");
     is(sillysum(1000), 1000*1001/2, "recursive sum of 1..1000");
 }
 
-
-
-
+# check ok for recursion depth > 65536
+is(runperl(
+       nolib => 1,
+       prog => q{$d=0; $e=1; sub c { ++$d; if ($d > 66000) { $e=0 } else { c(); c() unless $d % 32768 } --$d } c(); exit $e},
+), '', "64K deep recursion - no output expected");
+is($?, 0, "64K deep recursion - no coredump expected");