s3:nmbd: as the sig_term() handler only sets a flag we don't need to block SIGTERM
authorStefan Metzmacher <metze@samba.org>
Fri, 23 Jan 2009 09:15:00 +0000 (10:15 +0100)
committerStefan Metzmacher <metze@samba.org>
Fri, 23 Jan 2009 09:46:59 +0000 (10:46 +0100)
The arguments of commit d98bea900ee694cdba83149620c65bd7f8765f26
are no longer valid.

metze

source3/nmbd/asyncdns.c
source3/nmbd/nmbd.c
source3/nmbd/nmbd_packets.c

index baa88bc44b5be24d6a392329eb8cbef78f897d17..0736a66fb8a9bc570aa050230ee3b01d90e2b4a0 100644 (file)
@@ -210,9 +210,6 @@ void run_dns_queue(void)
        if (fd_in == -1)
                return;
 
-        /* Allow SIGTERM to kill us. */
-        BlockSignals(False, SIGTERM);
-
        if (!process_exists_by_pid(child_pid)) {
                close(fd_in);
                close(fd_out);
@@ -224,12 +221,9 @@ void run_dns_queue(void)
        if (!NT_STATUS_IS_OK(status)) {
                DEBUG(0, ("read from child failed: %s\n", nt_errstr(status)));
                fd_in = -1;
-                BlockSignals(True, SIGTERM);
                return;
        }
 
-        BlockSignals(True, SIGTERM);
-
        namerec = add_dns_result(&r.name, r.result);
 
        if (dns_current) {
@@ -338,14 +332,8 @@ bool queue_dns_query(struct packet_struct *p,struct nmb_name *question)
 
        DEBUG(3,("DNS search for %s - ", nmb_namestr(question)));
 
-        /* Unblock TERM signal so we can be killed in DNS lookup. */
-        BlockSignals(False, SIGTERM);
-
        dns_ip.s_addr = interpret_addr(qname);
 
-        /* Re-block TERM signal. */
-        BlockSignals(True, SIGTERM);
-
        namerec = add_dns_result(question, dns_ip);
        if(namerec == NULL) {
                send_wins_name_query_response(NAM_ERR, p, NULL);
index 870b9a1f30d90db663b407db71d917cee70ca090..1b26c0a5ada8e01878141ab2b584b1fce12e1fb4 100644 (file)
@@ -969,9 +969,6 @@ static bool open_sockets(bool isdaemon, int port)
                exit(1);
        }
 
-       /* We can only take signals in the select. */
-       BlockSignals( True, SIGTERM );
-
        TALLOC_FREE(frame);
        process();
 
index 489034899e653249e11c0fe3ad026acc9f80595f..f69845b346c4b6c9586ba6cee76ba6a2bd150cfa 100644 (file)
@@ -1794,16 +1794,8 @@ bool listen_for_packets(bool run_election)
                                         &r_fds, &w_fds, &timeout, &maxfd);
        }
 
-       /* Prepare for the select - allow certain signals. */
-
-       BlockSignals(False, SIGTERM);
-
        selrtn = sys_select(maxfd+1,&r_fds,&w_fds,NULL,&timeout);
 
-       /* We can only take signals when we are in the select - block them again here. */
-
-       BlockSignals(True, SIGTERM);
-
        if (run_events(nmbd_event_context(), selrtn, &r_fds, &w_fds)) {
                return False;
        }