From a9b12b22acac113a46f2ecc5ba29e055f259da18 Mon Sep 17 00:00:00 2001 From: David Herrmann Date: Mon, 5 Nov 2012 14:52:25 +0100 Subject: [PATCH] uterm: vt: avoid blocking SIGUSR1/2 twice eloop does already block signals that we register callbacks for so we do not have to block it manually here. Note that the signal-handlers are registered _before_ doing the backend initialization so we also avoid the previously fixed race-condition. Signed-off-by: David Herrmann --- src/uterm_vt.c | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/src/uterm_vt.c b/src/uterm_vt.c index 26ea99f..d35855e 100644 --- a/src/uterm_vt.c +++ b/src/uterm_vt.c @@ -316,7 +316,6 @@ static int real_open(struct uterm_vt *vt, const char *vt_for_seat0) struct vt_mode mode; struct vt_stat vts; int ret; - sigset_t mask; log_debug("open vt %p", vt); @@ -361,16 +360,6 @@ static int real_open(struct uterm_vt *vt, const char *vt_for_seat0) goto err_reset; } - sigemptyset(&mask); - sigaddset(&mask, SIGUSR1); - sigaddset(&mask, SIGUSR2); - ret = sigprocmask(SIG_BLOCK, &mask, NULL); - if (ret) { - log_error("cannot block SIGUSR1/2 (%d): %m", errno); - ret = -EFAULT; - goto err_text; - } - memset(&mode, 0, sizeof(mode)); mode.mode = VT_PROCESS; mode.acqsig = SIGUSR1;