develooper Front page | perl.perl5.porters | Postings from April 2016

[perl #127981] [PATCH] Deep recursion warning with%Module::CoreList::version

Thread Previous | Thread Next
From:
bulk88 via RT
Date:
April 28, 2016 14:06
Subject:
[perl #127981] [PATCH] Deep recursion warning with%Module::CoreList::version
Message ID:
rt-4.0.18-24447-1461852357-602.127981-15-0@perl.org
On Mon Apr 25 00:29:23 2016, davem wrote:
> +1
> 
> Its due to a tied lookup recursively looking up parent perl versions
> (i.e. working its way back through the chain of deltas between perl
> versions), then caching the result.
> 
> In the tests the warning only appears occasionally during smoking due to
> hash key randomisation affecting the order perl versions are scanned.
> 
> Post-5.24 we really ought to fix it properly (iterate rather than
> recursive tie lookups) otherwise at some point we're going to blow the C
> stack.

For the record I already encountered C stack recursion exhaustion bugs before with TieHashDelta https://rt.perl.org/Ticket/Display.html?id=121974 and fixed in http://perl5.git.perl.org/perl.git/commit/42484eedeb4abcee1d893010daa67c1a42ec6424 .

More recursion instead of looping/tailcalling bugs dont surprise me.

-- 
bulk88 ~ bulk88 at hotmail.com

---
via perlbug:  queue: perl5 status: open
https://rt.perl.org/Ticket/Display.html?id=127981

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About