Skip Menu |
 

This queue is for tickets about the IO-Async-Loop-Epoll CPAN distribution.

Report information
The Basics
Id: 126282
Status: patched
Priority: 0/
Queue: IO-Async-Loop-Epoll

People
Owner: Nobody in particular
Requestors: TEAM [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.19
Fixed in: (no value)

Attachments


Subject: Crash during global destruction when cleaning up signals
Download (untitled) / with headers
text/plain 264b
This one has been mentioned before but latest CPAN still has the issue, so I figured worth raising here: (in cleanup) (in cleanup) at .../IO/Async/Loop/Epoll.pm line 415 during global destruction. caused by calling `->orig` on undef, see attached patch.
Subject: 2018-08-16-undefined-signal.patch
diff --git a/lib/IO/Async/Loop/Epoll.pm b/lib/IO/Async/Loop/Epoll.pm index 5f4fbb4..99eb0f8 100644 --- a/lib/IO/Async/Loop/Epoll.pm +++ b/lib/IO/Async/Loop/Epoll.pm @@ -412,7 +412,7 @@ sub unwatch_signal # When we saved the original value, we might have got an undef. But %SIG # doesn't like having undef assigned back in, so we need to translate - $SIG{$signal} = $self->{signals}{$signal}->orig || 'DEFAULT'; + $SIG{$signal} = ($self->{signals}{$signal} && $self->{signals}{$signal}->orig) || 'DEFAULT'; delete $self->{signals}{$signal};
Thanks. Will be in 0.20 -- Paul Evans


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.