Skip to content

Commit

Permalink
Merge pull request #2022 from okurz/feature/driver_log_blocking
Browse files Browse the repository at this point in the history
Remove unnecessary but potentially racy signalblocker log lines
  • Loading branch information
mergify[bot] committed Apr 11, 2022
2 parents dbe823d + 2e88642 commit 112ac9e
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions signalblocker.pm
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ use POSIX ':signal_h';

sub new ($class, @args) {
# block signals
bmwqemu::diag('Blocking SIGCHLD and SIGTERM');
my %old_sig = %SIG;
$SIG{TERM} = 'IGNORE';
$SIG{INT} = 'IGNORE';
Expand All @@ -32,7 +31,6 @@ sub new ($class, @args) {

sub DESTROY ($self) {
# set back signal handling to default to be able to terminate properly
bmwqemu::diag('Unblocking SIGCHLD and SIGTERM');
die "Could not unblock SIGCHLD and SIGTERM\n" unless defined sigprocmask(SIG_UNBLOCK, $self->{_sigset}, undef);
%SIG = %{$self->{_old_sig}};
}
Expand Down

0 comments on commit 112ac9e

Please sign in to comment.