Re: bug#8938: make timeout and CTRL-C

[Pádraig Brady]

On 08/25/2011 11:35 AM, Pádraig Brady wrote:
> I'm committing this revert now...
>
> > From d00c73cf4d6402dbacc7cd587127a16a0e93724c Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?P=C3=A1draig=20Brady?=<address@hidden>
> Date: Thu, 25 Aug 2011 11:25:30 +0100
> Subject: [PATCH] timeout: revert signal propagation enhancement
>
> This effectively reverts the unreleased commit 5a647a05
>
> * src/timeout.c (main): Don't propagate signals from the monitored
> process, as on Linux /proc/sys/kernel/core_pattern could still
> handle them and cause false reports against `timeout`
> ---
>   src/timeout.c |    4 +++-
>   1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/src/timeout.c b/src/timeout.c
> index 6a37508..ae89942 100644
> --- a/src/timeout.c
> +++ b/src/timeout.c
> @@ -424,7 +424,9 @@ main (int argc, char **argv)
>             else if (WIFSIGNALED (status))
>               {
>                 int sig = WTERMSIG (status);
> -#if HAVE_SETRLIMIT&&  defined RLIMIT_CORE
> +/* The following is not used as one cannot disable processing
> +   by a filter in /proc/sys/kernel/core_pattern on Linux.  */
> +#if 0&&  HAVE_SETRLIMIT&&  defined RLIMIT_CORE
>                 if (!timed_out)
>                   {
>                     /* exit with the signal flag set, but avoid core files.  */

I found a more reliable way to disable core dumps,
so another attempt at this is attached.

cheers,
Pádraig.

timeout-signals.diff
Description: Text document