Open p5pRT opened 9 years ago
A full description would probably be longer than this code snippet:
$ perl -w sub foo { die } sub bar { goto &foo }
$SIG{__DIE__} = \&bar; die; Deep recursion on subroutine "main::foo" at - line 2. Segmentation fault
Segfault aside\, I did not expect the die() in foo() to trigger the __DIE__ handler; according to perlvar(1)\, the handler "is explicitly disabled *during* the call"\, from which we haven't returned yet (even though we have technically left the subroutine).
This SEGV with a nested die handler doesn't seem severity low to me. It seems to be similar to Lukas Mai's fix for 'don't fatalize warnings during unwinding (#123398)'\, which is not even applied yet. -- Reini Urban
The RT System itself - Status changed from 'new' to 'open'
"Reini Urban via RT" \perlbug\-followup@​perl\.org writes:
This SEGV with a nested die handler doesn't seem severity low to me. It seems to be similar to Lukas Mai's fix for 'don't fatalize warnings during unwinding (#123398)'\, which is not even applied yet.
You appear to be looking at an outdated copy of the perl source\, it was applied by TonyC on 2015-07-08:
http://perl5.git.perl.org/perl.git/commitdiff/46b27d2f2c37f40dde845f9b4743975c69f2cc27
If you think it should be backported to maint-5.22\, feel free to suggest it.
-- "I use RMS as a guide in the same way that a boat captain would use a lighthouse. It's good to know where it is\, but you generally don't want to find yourself in the same spot." - Tollef Fog Heen
Die handler testcases added in the attached patch. do not apply\, fix it first. or I'll do I'm faster
-- Reini Urban
On Aug 12\, 2015\, at 3:10 PM\, Dagfinn Ilmari MannsĂ„ker \ilmari@​ilmari\.org wrote:
"Reini Urban via RT" \perlbug\-followup@​perl\.org writes:
This SEGV with a nested die handler doesn't seem severity low to me. It seems to be similar to Lukas Mai's fix for 'don't fatalize warnings during unwinding (#123398)'\, which is not even applied yet.
You appear to be looking at an outdated copy of the perl source\, it was applied by TonyC on 2015-07-08:
http://perl5.git.perl.org/perl.git/commitdiff/46b27d2f2c37f40dde845f9b4743975c69f2cc27
I only merge blead monthly so I missed that. Good that it is in finally. Should have been in for your 5.22.0 already. Fixed Feb 12\, it was in my 5.22 in March.
If you think it should be backported to maint-5.22\, feel free to suggest it.
Sure it must. Itâs on the summary list of the maint-5.20 votings already\, havenât seen it on maint-5.22\, but I have no authority to suggest anything there.
* Reini Urban \reini\.urban@​gmail\.com [2015-08-15T17:15:28]
On Aug 12\, 2015\, at 3:10 PM\, Dagfinn Ilmari MannsĂ„ker \ilmari@​ilmari\.org wrote:
If you think it should be backported to maint-5.22\, feel free to suggest it.
Sure it must. Itâs on the summary list of the maint-5.20 votings already\, havenât seen it on maint-5.22\, but I have no authority to suggest anything there.
Filing a ticket (or somewhat less reliably\, sending a normal p5p mail) to get attention on things needing backporting is a good idea. The response may well be\, "Oh\, yes\, of course\, just nobody had caught that one yet."
-- rjbs
On Sat Aug 15 14:36:13 2015\, perl.p5p@rjbs.manxome.org wrote:
* Reini Urban \reini\.urban@​gmail\.com [2015-08-15T17:15:28]
On Aug 12\, 2015\, at 3:10 PM\, Dagfinn Ilmari MannsĂ„ker \ilmari@​ilmari\.org wrote:
If you think it should be backported to maint-5.22\, feel free to suggest it.
Sure it must. Itâs on the summary list of the maint-5.20 votings already\, havenât seen it on maint-5.22\, but I have no authority to suggest anything there.
Filing a ticket (or somewhat less reliably\, sending a normal p5p mail) to get attention on things needing backporting is a good idea. The response may well be\, "Oh\, yes\, of course\, just nobody had caught that one yet."
Thanks. It's now in the voting file for maint-5.22\, which I've just started working on:
http://perl5.git.perl.org/perl.git/blob/a5e72c3582fa6693395860fcaf2d1124e146a43e:/votes-5.22.xml#l83
(I've also listed #123878 in the hope that it can be fixed in time\, but there is no fix to backport yet.)
Migrated from rt.perl.org#123878 (status was 'open')
Searchable as RT123878$