[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [Bug 1061778] [NEW] signal mask not reset on exec
From: |
John Sullivan |
Subject: |
[Qemu-devel] [Bug 1061778] [NEW] signal mask not reset on exec |
Date: |
Thu, 04 Oct 2012 17:51:27 -0000 |
Public bug reported:
Seen in qemu-1.0 under 12.04, but AFAICT from current git it hasn't
changed.
./main-loop.c:qemu_signal_init blocks SIGALRM so it can be handled via
signalfd.
./net/tap.c:launch_script does not reset the signal mask before the
execv() call, and signal masks are inherited. So the script is run with
SIGALRM blocked (as can be seen in /proc/$$/status, "SigBlk:
0000000000002000"). One reasonable example of where this bites is an
interface up script that calls ping with a timeout to give things a
chance to settle down before continuing, but abort if this doesn't
happen within a reasonable time). Since ping uses SIGALRM for the
timeout, this now never terminates.
qemu-0.14 didn't block SIGALRM, so such scripts worked fine there.
** Affects: qemu
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1061778
Title:
signal mask not reset on exec
Status in QEMU:
New
Bug description:
Seen in qemu-1.0 under 12.04, but AFAICT from current git it hasn't
changed.
./main-loop.c:qemu_signal_init blocks SIGALRM so it can be handled via
signalfd.
./net/tap.c:launch_script does not reset the signal mask before the
execv() call, and signal masks are inherited. So the script is run
with SIGALRM blocked (as can be seen in /proc/$$/status, "SigBlk:
0000000000002000"). One reasonable example of where this bites is an
interface up script that calls ping with a timeout to give things a
chance to settle down before continuing, but abort if this doesn't
happen within a reasonable time). Since ping uses SIGALRM for the
timeout, this now never terminates.
qemu-0.14 didn't block SIGALRM, so such scripts worked fine there.
To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1061778/+subscriptions
- [Qemu-devel] [PATCH 0/5] qemu-ga: add isa-serial support to win32, Luiz Capitulino, 2012/10/31
- [Qemu-devel] [PATCH 3/5] qemu-ga: win32: GAChannel: store GAChannelMethod being used, Luiz Capitulino, 2012/10/31
- [Qemu-devel] [PATCH 1/5] configure: link qemu-ga.exe in default make target, Luiz Capitulino, 2012/10/31
- [Qemu-devel] [PATCH 2/5] qemu-ga: win32: add _ov suffix to current GSource functions, Luiz Capitulino, 2012/10/31
- [Qemu-devel] [PATCH 5/5] qemu-ga: win32: add isa-serial support, Luiz Capitulino, 2012/10/31
- [Qemu-devel] [PATCH 4/5] qemu-ga: win32: isolate virtio-serial specific code, Luiz Capitulino, 2012/10/31