Mailing-List: contact cygwin-developers-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-developers-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-developers/>
List-Post: <mailto:cygwin-developers AT cygwin DOT com>
List-Help: <mailto:cygwin-developers-help AT cygwin DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-developers-owner AT cygwin DOT com
Delivered-To: mailing list cygwin-developers AT cygwin DOT com
Message-ID: <3E7F5A9C.4050804@hekimian.com>
Date: Mon, 24 Mar 2003 14:21:00 -0500
X-Sybari-Trust: 35e8fa5b 36b09be0 04609a3e 00000109
From: Joe Buehler <jbuehler AT hekimian DOT com>
Reply-To: jbuehler AT hekimian DOT com
Organization: Spirent Communications, Inc.
User-Agent: Mozilla/5.0 (Windows; U; WinNT4.0; en-US; rv:1.3) Gecko/20030312
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: cygwin-developers AT cygwin DOT com
Subject: second try: [PATCH] finishing spawn / vfork signal inheritance
References: <3E79FA1B DOT 60302 AT hekimian DOT com>
In-Reply-To: <3E79FA1B.60302@hekimian.com>
X-Enigmail-Version: 0.73.1.0
X-Enigmail-Supports: pgp-inline, pgp-mime
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

If someone with CVS access would commit this, I would appreciate it.
Currently, spawn() and vfork() do not pass signal handler settings
on to the child process, and this breaks ksh93.  This patch fixes
the problem (in conjunction with a change cgf already committed).

2003-03-20  Joe Buehler  <jhpb AT draco DOT hekimian DOT com>

     * spawn.cc: (spawn_guts): copy signal handler info to spawned child.

Index: spawn.cc
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/spawn.cc,v
retrieving revision 1.120
diff -u -r1.120 spawn.cc
--- spawn.cc    13 Feb 2003 02:52:41 -0000    1.120
+++ spawn.cc    20 Mar 2003 17:26:55 -0000
@@ -756,6 +756,8 @@
      }
        child->dwProcessId = pi.dwProcessId;
        child->hProcess = pi.hProcess;
+      /* make child inherit our signal settings */
+      child->copysigs (myself);
        child.remember ();
        strcpy (child->progname, real_path);
        /* FIXME: This introduces an unreferenced, open handle into the child.
-- 
Joe Buehler