Commit 103295c7 authored by François Gouget's avatar François Gouget Committed by Alexandre Julliard

Reset signals to SIG_DFL before starting a child process.

parent b4d3cb59
......@@ -772,7 +772,7 @@ static void do_trap( CONTEXT *context, int trap_code )
}
else
{
/* likely we get this because of a kill(SIGTRAP) on ourself,
/* likely we get this because of a kill(SIGTRAP) on ourself,
* so send a bp exception instead of a single step exception
*/
TRACE("Spurious single step trap => breakpoint simulation\n");
......@@ -1059,9 +1059,6 @@ BOOL SIGNAL_Init(void)
sigfillset( &all_sigs );
/* automatic child reaping to avoid zombies */
signal( SIGCHLD, SIG_IGN );
if (set_handler( SIGINT, have_sigaltstack, (void (*)())int_handler ) == -1) goto error;
if (set_handler( SIGFPE, have_sigaltstack, (void (*)())fpe_handler ) == -1) goto error;
if (set_handler( SIGSEGV, have_sigaltstack, (void (*)())segv_handler ) == -1) goto error;
......
......@@ -362,11 +362,6 @@ void SIGNAL_Unblock( void )
*/
BOOL SIGNAL_Init(void)
{
/* ignore SIGPIPE so that WINSOCK can get a EPIPE error instead */
signal( SIGPIPE, SIG_IGN );
/* automatic child reaping to avoid zombies */
signal( SIGCHLD, SIG_IGN );
sigfillset( &all_sigs );
if (set_handler( SIGINT, (void (*)())int_handler ) == -1) goto error;
......
......@@ -677,6 +677,8 @@ void CLIENT_InitThread(void)
/* ignore SIGPIPE so that we get a EPIPE error instead */
signal( SIGPIPE, SIG_IGN );
/* automatic child reaping to avoid zombies */
signal( SIGCHLD, SIG_IGN );
/* create the server->client communication pipes */
if (pipe( reply_pipe ) == -1) server_protocol_perror( "pipe" );
......
......@@ -25,6 +25,7 @@
#include <ctype.h>
#include <errno.h>
#include <fcntl.h>
#include <signal.h>
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
......@@ -861,6 +862,10 @@ static int fork_and_exec( const char *filename, char *cmdline,
char **envp = build_envp( env, extra_env );
close( fd[0] );
/* Reset signals that we previously set to SIG_IGN */
signal( SIGPIPE, SIG_DFL );
signal( SIGCHLD, SIG_DFL );
if (newdir) chdir(newdir);
if (argv && envp)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment