chiark / gitweb /
ask-password: reset signal mask after we are done
authorLennart Poettering <lennart@poettering.net>
Wed, 16 Mar 2011 01:55:27 +0000 (02:55 +0100)
committerLennart Poettering <lennart@poettering.net>
Wed, 16 Mar 2011 01:55:27 +0000 (02:55 +0100)
TODO
src/ask-password-api.c

diff --git a/TODO b/TODO
index d377bc3..307d023 100644 (file)
--- a/TODO
+++ b/TODO
@@ -26,6 +26,8 @@ F15:
 
 * recreate private socket on SIGUSR2
 
+* serialize condition execution information
+
 Features:
 
 * optionally create watched directories in .path units
index 0b2e9ad..9c3dad9 100644 (file)
@@ -271,11 +271,15 @@ int ask_password_agent(
         FILE *f = NULL;
         char *socket_name = NULL;
         int socket_fd = -1, signal_fd = -1;
-        sigset_t mask;
+        sigset_t mask, oldmask;
         struct pollfd pollfd[_FD_MAX];
 
         assert(_passphrases);
 
+        assert_se(sigemptyset(&mask) == 0);
+        sigset_add_many(&mask, SIGINT, SIGTERM, -1);
+        assert_se(sigprocmask(SIG_BLOCK, &mask, &oldmask) == 0);
+
         mkdir_p("/dev/.run/systemd/ask-password", 0755);
 
         if ((fd = mkostemp(temp, O_CLOEXEC|O_CREAT|O_WRONLY)) < 0) {
@@ -294,10 +298,6 @@ int ask_password_agent(
 
         fd = -1;
 
-        assert_se(sigemptyset(&mask) == 0);
-        sigset_add_many(&mask, SIGINT, SIGTERM, -1);
-        assert_se(sigprocmask(SIG_SETMASK, &mask, NULL) == 0);
-
         if ((signal_fd = signalfd(-1, &mask, SFD_NONBLOCK|SFD_CLOEXEC)) < 0) {
                 log_error("signalfd(): %m");
                 r = -errno;
@@ -493,6 +493,8 @@ finish:
         if (final[0])
                 unlink(final);
 
+        assert_se(sigprocmask(SIG_SETMASK, &oldmask, NULL) == 0);
+
         return r;
 }